[PATCH] Fix --help -Q output

[Richard Biener]

On Thu, Dec 2, 2021 at 3:07 PM Martin Liška <mliska@suse.cz> wrote:
>
> On 11/30/21 10:33, Richard Biener wrote:
> > and the "docs" say
> >
> >    /* The switch is enabled when FLAG_VAR is nonzero.  */
> >    CLVC_BOOLEAN,
>
> That's bogus, because real meaning of CLVC_BOOLEAN is that it holds an integer value.
> It can be just true/false for a simple flag, can be 0,1,2 for things like flag_lifetime_dse,
> or it can be an arbitrary integer value for things like params.
>
> That said, I would suggest removing that..
>
> Thoughts about the patch?

+           return v > 0 ? (v < 0 ? -1 : 1) : 0;

the ?:s look odd to me, the above is equivalent to v >  0 ? 1 : 0, no?
Did you mean to make v > 0 v != 0?

> Martin