Re: [PATCH] Fix up remove_partial_avx_dependency (PR target/89474)

2019-02-26 Thread Richard Biener
On Mon, Feb 25, 2019 at 11:53 PM Jakub Jelinek wrote: > > Hi! > > The following patch fixes two issues in the new rpad pass. > One is that the insertion at the start of a basic block didn't work properly > if the basic block didn't contain any non-NOTE/non-DEBUG_INSN instructions. > next_nonnote_n

Re: [PATCH] Fix up remove_partial_avx_dependency (PR target/89474)

2019-02-26 Thread Jakub Jelinek
Hi! Honza, you've reviewed H.J.'s patch, could you please review this follow-up as well? Thanks. On Mon, Feb 25, 2019 at 02:57:39PM -0800, H.J. Lu wrote: > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > > > 2019-02-25 Jakub Jelinek > > > > PR target/89474 >

Re: [PATCH] Fix up remove_partial_avx_dependency (PR target/89474)

2019-02-25 Thread H.J. Lu
On Mon, Feb 25, 2019 at 2:53 PM Jakub Jelinek wrote: > > Hi! > > The following patch fixes two issues in the new rpad pass. > One is that the insertion at the start of a basic block didn't work properly > if the basic block didn't contain any non-NOTE/non-DEBUG_INSN instructions. > next_nonnote_no

[PATCH] Fix up remove_partial_avx_dependency (PR target/89474)

2019-02-25 Thread Jakub Jelinek
Hi! The following patch fixes two issues in the new rpad pass. One is that the insertion at the start of a basic block didn't work properly if the basic block didn't contain any non-NOTE/non-DEBUG_INSN instructions. next_nonnote_nondebug_insn hapilly turns through into another basic block and the