Re: [PATCH] Don't assert for IFN_COND_{MIN, MAX} in vect_transform_reduction

2024-04-30 Thread Hongtao Liu
On Tue, Apr 30, 2024 at 3:38 PM Jakub Jelinek wrote: > > On Tue, Apr 30, 2024 at 09:30:00AM +0200, Richard Biener wrote: > > On Mon, Apr 29, 2024 at 5:30 PM H.J. Lu wrote: > > > > > > On Mon, Apr 29, 2024 at 6:47 AM liuhongt wrote: > > > > > > > > The Fortran standard does not specify what the r

Re: [PATCH] Don't assert for IFN_COND_{MIN, MAX} in vect_transform_reduction

2024-04-30 Thread Jakub Jelinek
On Tue, Apr 30, 2024 at 09:30:00AM +0200, Richard Biener wrote: > On Mon, Apr 29, 2024 at 5:30 PM H.J. Lu wrote: > > > > On Mon, Apr 29, 2024 at 6:47 AM liuhongt wrote: > > > > > > The Fortran standard does not specify what the result of the MAX > > > and MIN intrinsics are if one of the argument

Re: [PATCH] Don't assert for IFN_COND_{MIN, MAX} in vect_transform_reduction

2024-04-30 Thread Richard Biener
On Mon, Apr 29, 2024 at 5:30 PM H.J. Lu wrote: > > On Mon, Apr 29, 2024 at 6:47 AM liuhongt wrote: > > > > The Fortran standard does not specify what the result of the MAX > > and MIN intrinsics are if one of the arguments is a NaN. So it > > should be ok to tranform reduction for IFN_COND_MIN wi

Re: [PATCH] Don't assert for IFN_COND_{MIN, MAX} in vect_transform_reduction

2024-04-29 Thread H.J. Lu
On Mon, Apr 29, 2024 at 6:47 AM liuhongt wrote: > > The Fortran standard does not specify what the result of the MAX > and MIN intrinsics are if one of the arguments is a NaN. So it > should be ok to tranform reduction for IFN_COND_MIN with vectorized > COND_MIN and REDUC_MIN. The commit subject

[PATCH] Don't assert for IFN_COND_{MIN, MAX} in vect_transform_reduction

2024-04-29 Thread liuhongt
The Fortran standard does not specify what the result of the MAX and MIN intrinsics are if one of the arguments is a NaN. So it should be ok to tranform reduction for IFN_COND_MIN with vectorized COND_MIN and REDUC_MIN. Bootstrapped and regtested on x86_64-pc-linux-gnu{-m32,}. Ok for trunk and bac