Re: [PATCH][v3] rtl-optimization/105231 - distribute_notes and REG_EH_REGION

2022-04-21 Thread Richard Biener via Gcc-patches
On Thu, 21 Apr 2022, Richard Biener wrote: > On Thu, 21 Apr 2022, Richard Biener wrote: > > > On Wed, 20 Apr 2022, Segher Boessenkool wrote: > > > > > Hi! > > > > > > This looks great :-) > > > > > > On Wed, Apr 20, 2022 at 03:52:33PM +0200, Richard Biener wrote: > > > > The following mitigate

Re: [PATCH][v3] rtl-optimization/105231 - distribute_notes and REG_EH_REGION

2022-04-21 Thread Richard Biener via Gcc-patches
On Thu, 21 Apr 2022, Richard Biener wrote: > On Wed, 20 Apr 2022, Segher Boessenkool wrote: > > > Hi! > > > > This looks great :-) > > > > On Wed, Apr 20, 2022 at 03:52:33PM +0200, Richard Biener wrote: > > > The following mitigates a problem in combine distribute_notes which > > > places an or

Re: [PATCH][v3] rtl-optimization/105231 - distribute_notes and REG_EH_REGION

2022-04-21 Thread Richard Biener via Gcc-patches
On Wed, 20 Apr 2022, Segher Boessenkool wrote: > Hi! > > This looks great :-) > > On Wed, Apr 20, 2022 at 03:52:33PM +0200, Richard Biener wrote: > > The following mitigates a problem in combine distribute_notes which > > places an original REG_EH_REGION based on only may_trap_p which is > > goo

Re: [PATCH][v3] rtl-optimization/105231 - distribute_notes and REG_EH_REGION

2022-04-20 Thread Segher Boessenkool
Hi! This looks great :-) On Wed, Apr 20, 2022 at 03:52:33PM +0200, Richard Biener wrote: > The following mitigates a problem in combine distribute_notes which > places an original REG_EH_REGION based on only may_trap_p which is > good to test whether a non-call insn can possibly throw but not if

[PATCH][v3] rtl-optimization/105231 - distribute_notes and REG_EH_REGION

2022-04-20 Thread Richard Biener via Gcc-patches
The following mitigates a problem in combine distribute_notes which places an original REG_EH_REGION based on only may_trap_p which is good to test whether a non-call insn can possibly throw but not if actually it does or we care. That's something we decided at RTL expansion time where we possibly