On Thu, Dec 2, 2021 at 4:55 PM Martin Liška wrote:
>
> On 12/2/21 15:22, Martin Liška wrote:
> > On 12/2/21 15:11, Richard Biener wrote:
> >> 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?
> >
> > Yeah, it's typo, should be 'v != 0 ?
tstrap on x86_64-linux-gnu and survives
regression tests.
Ready to be installed?
Thanks,
MartinFrom d311b2370e6ab7a7aa36d666311a98d4e2ae8b35 Mon Sep 17 00:00:00 2001
From: Martin Liska
Date: Mon, 29 Nov 2021 14:46:47 +0100
Subject: [PATCH] Fix --help -Q output
PR middle-end/103438
gcc/ChangeL
On 12/2/21 15:11, Richard Biener wrote:
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?
Yeah, it's typo, should be 'v != 0 ? ...'.
Martin
On Thu, Dec 2, 2021 at 3:07 PM Martin Liška 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.
7 +0100
Subject: [PATCH] Fix --help -Q output
PR middle-end/103438
gcc/ChangeLog:
* config/s390/s390.c (s390_valid_target_attribute_inner_p):
Use new enum CLVC_INTEGER.
* opt-functions.awk: Use new CLVC_INTEGER.
* opts-common.c (set_option): Likewise.
(option_enabled): Return -1,0,1 f
On Mon, Nov 29, 2021 at 4:16 PM Martin Liška wrote:
>
> There are cases where a default option value is -1 and
> auto-detection happens in e.g. target.
>
> Do not print these options. Leads to the following diff:
>
> - -fdelete-null-pointer-checks [enabled]
> + -fdelete-null-poin
There are cases where a default option value is -1 and
auto-detection happens in e.g. target.
Do not print these options. Leads to the following diff:
- -fdelete-null-pointer-checks [enabled]
+ -fdelete-null-pointer-checks
@@ -332 +332 @@
- -fleading-underscor
