Re: [PATCH] Improve verification of loop->latch in verify_loop_structure

[Tom de Vries]

On 26/11/15 13:15, Richard Biener wrote:

On Thu, 26 Nov 2015, Alan Lawrence wrote:

This caused an ICE compiling value.c from gdb on
aarch64-none-linux-gnu; the testcase, after preprocessing on aarch64,
ICEs on both aarch64 and x86_64, but is about 1MB - I'm working on
reducing that down to something small enough to post...

$ ./gcc/xgcc -B ./gcc -O2 -g value.c
../../binutils-gdb/gdb/value.c: In function ‘show_convenience’:
../../binutils-gdb/gdb/value.c:2615:1: error: loop 3’s latch is missing
../../binutils-gdb/gdb/value.c:2615:1: internal compiler error: in
verify_loop_structure, at cfgloop.c:1669
0x71e653 verify_loop_structure()
         /work/alalaw01/src2/gcc/gcc/cfgloop.c:1669
0x97c6ae checking_verify_loop_structure
         /work/alalaw01/src2/gcc/gcc/cfgloop.h:325
0x97c6ae loop_optimizer_init(unsigned int)
         /work/alalaw01/src2/gcc/gcc/loop-init.c:106
0x97c78a rtl_loop_init
         /work/alalaw01/src2/gcc/gcc/loop-init.c:398
0x97c78a execute
         /work/alalaw01/src2/gcc/gcc/loop-init.c:425

See also PR68549 for why I think this happens "by design".  Thus
I think we need to revert the checking when
LOOPS_MAY_HAVE_MULTIPLE_LATCHES for now.

I'll revert the whole patch for now.

Thanks,
- Tom