Re: [PATCH] Fix combiner move_deaths DEBUG_INSN handling (PR target/84614)

2018-03-02 Thread Jeff Law
On 03/01/2018 04:13 PM, Jakub Jelinek wrote: > Hi! > > prev_real_insn doesn't skip over DEBUG_INSNs, on aarch64 on this > testcase we get both -fcompare-debug failure and wrong-code with -g. > > Instead of adding yet another two: > do > insn = prev_real_insn (insn); > while (D

Re: [PATCH] Fix combiner move_deaths DEBUG_INSN handling (PR target/84614)

2018-03-01 Thread Segher Boessenkool
Hi Jakub, On Fri, Mar 02, 2018 at 12:13:51AM +0100, Jakub Jelinek wrote: > prev_real_insn doesn't skip over DEBUG_INSNs, on aarch64 on this > testcase we get both -fcompare-debug failure and wrong-code with -g. > > Instead of adding yet another two: > do > insn = prev_real_insn (ins