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) >
