[PATCH, AArch64] Fix configure test for AArch64 dwarf2 debug_line

Richard Earnshaw rearnsha@arm.com
Thu Aug 22 15:03:00 GMT 2013


On 22/08/13 15:26, Julian Brown wrote:
> Hi,
> 
> This patch fixes what I assume to be a simple omission from the
> configure test for debug_line assembler support in gcc/configure. This
> tripped an obscure corner-case test in our build infrastructure, which
> is fixed by the attached patch, but I don't otherwise have a way of
> testing the patch at present.
> 
> OK to apply?
> 
> Thanks,
> 
> Julian
> 
> ChangeLog
> 
>     gcc/
>     * configure.ac: Add aarch64 to list of arches which use "nop" in
>     debug_line test.
>     * configure: Regenerate.
> 
> 

This is OK, though it seems to me that, given how old gas 2.11 is now,
it might be better to have a 'default' for this test and then catch the
systems that are different.

R.

> aarch64-debug-linux-configure-fix-1.diff
> 
> 
> Index: gcc/configure.ac
> ===================================================================
> --- gcc/configure.ac	(revision 201919)
> +++ gcc/configure.ac	(working copy)
> @@ -4219,8 +4219,9 @@ esac
>  # ??? Once 2.11 is released, probably need to add first known working
>  # version to the per-target configury.
>  case "$cpu_type" in
> -  alpha | arm | avr | bfin | cris | i386 | m32c | m68k | microblaze | mips \
> -  | pa | rs6000 | score | sparc | spu | tilegx | tilepro | xstormy16 | xtensa)
> +  aarch64 | alpha | arm | avr | bfin | cris | i386 | m32c | m68k | microblaze
> +  | mips | pa | rs6000 | score | sparc | spu | tilegx | tilepro | xstormy16
> +  | xtensa)
>      insn="nop"
>      ;;
>    ia64 | s390)
> 




More information about the Gcc-patches mailing list