On 12-10-03 9:01 AM, Jakub Jelinek wrote:
Hi!While bootstrapping the PR54519 patches last night, I've noticed a comparison failure on i686-linux. The problem was -g vs. -g0 swapping two instructions in tree-inline.o, caused by find_modifiable_mems scanning [head, tail) instead of [head, tail] sequence of instructions. Without -g0 on tree-inline.c we had a MEM load as last insn in a bb (i.e. tail) which wasn't processed by find_modifiable_mems, while with -g there were some DEBUG_INSNs after it and find_mems was called on it and triggerring the optimization. Fixed by scanning also the tail insn, the same way as other spots in sched-deps.c handle [head, tail] walk. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2012-10-03 Jakub Jelinek <ja...@redhat.com> PR rtl-optimization/54792 * sched-deps.c (find_modifiable_mems): Scan also TAIL insn.
Yes. Thanks, Jakub.
