------- Comment #15 from ddaney at avtrex dot com 2007-09-05 21:15 ------- Subject: Re: internal compiler error: in print_operand_reloc, at config/mips/mips.c:5579
richard at codesourcery dot com wrote: > ------- Comment #14 from richard at codesourcery dot com 2007-09-05 21:08 > ------- > Subject: Re: internal compiler error: in print_operand_reloc, at > config/mips/mips.c:5579 > > "ddaney at avtrex dot com" <[EMAIL PROTECTED]> writes: >>> I've not forced -EB because that fails for -EL >>> multilibs, and we want this test to work for both anyway.) >>> >> Are you sure? On the trunk I tested on mipsel-linux with -EB and it >> seems to work. For a compile only test it should work on little-endian >> system. > > I mean that it fails if you explicitly test -EL multilibs. E.g. on > mipsisa64-elf, I normally test "mips-sim{,-msoft-float}{-EB,-EL}", and > the compiler will complain at having both -EL and -EB on the command line. > We could make mips.exp skip the test when -EL is forced, but it seems > more useful to run it regardless. Or we could use target selectors to > select between two dg-mips-options lines depending on whether -EL has > been added to the multilib flags. However, it seems odd to test > big-endian (only) on little-endian systems when -EL is not explicitly > given, but test little-endian when -EL _is_ explicitly (and redundantly) > given. > My concern was that the bug only occurs for me with -EB, so running the test case with -EL would be pointless. I understand that you do most of your testing on the simulator, but for someone doing testing on little endian hardware there is no coverage without the -EB. David Daney -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33256