Re: [PATCH] phiopt: Remove unnecessary checks from spaceship_replacement [PR106506]

2022-08-12 Thread Richard Biener via Gcc-patches
On Thu, 11 Aug 2022, Jakub Jelinek wrote: > Hi! > > Those 2 checks were just me trying to be extra careful, the > (phires & 1) == phires and variants it is folded to of course make only sense > for the -1/0/1/2 result spaceship, for -1/0/1 one can just use comparisons of > phires. We only floati

[PATCH] phiopt: Remove unnecessary checks from spaceship_replacement [PR106506]

2022-08-12 Thread Jakub Jelinek via Gcc-patches
Hi! Those 2 checks were just me trying to be extra careful, the (phires & 1) == phires and variants it is folded to of course make only sense for the -1/0/1/2 result spaceship, for -1/0/1 one can just use comparisons of phires. We only floating point spaceship if nans aren't honored, so the 2 cas