Re: [PATCH] Invalidate combiner's cached last value upon insn removal (PR rtl-optimization/79388, PR rtl-optimization/79450)

2017-02-13 Thread Segher Boessenkool
Hi Jakub, On Fri, Feb 10, 2017 at 08:50:36PM +0100, Jakub Jelinek wrote: > On the following testcases combiner during notes distribution sees a > REG_DEAD note and decides to remove the setter thereof. That register > has remembered a last value on that insn though, and it is a pseudo set >

Re: [PATCH] Invalidate combiner's cached last value upon insn removal (PR rtl-optimization/79388, PR rtl-optimization/79450)

2017-02-13 Thread Bernd Schmidt
On 02/10/2017 08:50 PM, Jakub Jelinek wrote: 2017-02-10 Jakub Jelinek PR rtl-optimization/79388 PR rtl-optimization/79450 * combine.c (distribute_notes): When removing TEM_INSN for which corresponding dest has last value recorded, invalidate

[PATCH] Invalidate combiner's cached last value upon insn removal (PR rtl-optimization/79388, PR rtl-optimization/79450)

2017-02-10 Thread Jakub Jelinek
Hi! On the following testcases combiner during notes distribution sees a REG_DEAD note and decides to remove the setter thereof. That register has remembered a last value on that insn though, and it is a pseudo set multiple times. Later on we combine some further insns into pseudo = pseudo op