On Mon, 2026-03-09 at 14:37 +0100, Jakub Jelinek wrote:
> On Mon, Mar 09, 2026 at 11:59:14AM +0530, Avinash Jayakar wrote:
> > Pinging this patch. Actually the need for this enhancement was
> > observed
> > from PR106536 where,
> >
> > int compare2(unsigned long long a, unsigned long long b)
> > {
> > return (a > b ? 1 : (a < b ? -1 : 0));
> > }
> > this was not being recognized as spaceship but on the other hand
> >
> > int compare2(unsigned long long a, unsigned long long b)
> > {
> > return (a == b ? 0 : (a < b ? -1 : 1));
> >
> >
> > }
> > this one was. And I think identifying it as spaceship generates
> > better
> > code than if not.
>
> Is this a regression? If not, it feels to me more like stage1
> material.
Right, this is not a regression, just enhancement. Got it, will ping
again once stage1 time period starts.
Thanks and regards,
Avinash Jayakar
>
> Jakub
