Re: [PATCH] Fix RTL DCE with separate shrink wrapped epilogues (PR rtl-optimization/83985)

2018-01-26 Thread Richard Biener
On Thu, 25 Jan 2018, Segher Boessenkool wrote: > On Thu, Jan 25, 2018 at 11:20:33PM +0100, Jakub Jelinek wrote: > > Hi! > > > > The r241060 change added the second hunk in this patch which the patch is > > reverting. The problem is that not deleting some unmarked insns in > >

Re: [PATCH] Fix RTL DCE with separate shrink wrapped epilogues (PR rtl-optimization/83985)

2018-01-25 Thread Segher Boessenkool
On Thu, Jan 25, 2018 at 11:20:33PM +0100, Jakub Jelinek wrote: > Hi! > > The r241060 change added the second hunk in this patch which the patch is > reverting. The problem is that not deleting some unmarked insns in > delete_unmarked_insns is done in a wrong place, it causes indeed not to >

[PATCH] Fix RTL DCE with separate shrink wrapped epilogues (PR rtl-optimization/83985)

2018-01-25 Thread Jakub Jelinek
Hi! The r241060 change added the second hunk in this patch which the patch is reverting. The problem is that not deleting some unmarked insns in delete_unmarked_insns is done in a wrong place, it causes indeed not to delete the instruction we don't want to DCE, but the instructions that are