bug-bison
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Bison 3.5 - y.tab.c generates compiler warnings


From: Akim Demaille
Subject: Re: Bison 3.5 - y.tab.c generates compiler warnings
Date: Tue, 31 Dec 2019 12:06:11 +0100


> Le 29 déc. 2019 à 09:09, Akim Demaille <address@hidden> a écrit :
> 
> Hi guys,
> 
>> Le 19 déc. 2019 à 07:22, Akim Demaille <address@hidden> a écrit :
>> 
>> But so far I was unable to reproduce that warning 
>> (https://travis-ci.org/akimd/bison/builds/626034753), and if we want to 
>> commit avoiding this warning, we should also add it to the set of warnings 
>> we check.
> 
> I am still unable to reproduce it on Bison's test suite.  I tried three other 
> architectures on Travis (PPC64le, ARM64 and s390x), I also tried GCC's 
> -funsigned-char, but I still cannot reproduce it.
> 
> I wish I had a CI that could check that case.

Well, I'm installing this anyway.  Andy, it would be great if you could run the 
full test suite, I guess yacc.c is not the only place where we have to take 
care of this.

Cheers!

commit 78bb152a63f711af65364881c434af4c198e1ee0
Author: Akim Demaille <address@hidden>
Date:   Tue Dec 17 06:39:09 2019 +0100

   tests: also check -Wchar-subscripts

   GCC's -Wchar-subscripts may report issues on platforms where char is
   unsigned.  Unfortunately the current CI does not reproduce the
   problem.  But that would allow contributors to report issues if the
   warning appears somewhere.

   See 139d0655947c87f90af08718618feaaca0e558d7.
   Problem reported by Andy Fiddaman in:
   https://lists.gnu.org/r/bug-bison/2019-12/msg00021.html

   * configure.ac (warn_common): Add -Wchar-subscripts.

diff --git a/configure.ac b/configure.ac
index 7a80eac1..240ca3d5 100644
--- a/configure.ac
+++ b/configure.ac
@@ -100,7 +100,8 @@ if test "$enable_gcc_warnings" = yes; then
  # -Wno-tautological-constant-out-of-range-compare for Clang 3.3 and
  # 3.4 on GNU/Linux that choke on intprops.h's INT_MULTIPLY_WRAPV,
  # etc.
-  warn_common='-Wall -Wextra -Wcast-align
+  warn_common='-Wall -Wextra
+    -Wcast-align -Wchar-subscripts
    -fparse-all-comments -Wdocumentation
    -Wformat -Wimplicit-fallthrough -Wnull-dereference
    -Wno-sign-compare -Wno-tautological-constant-out-of-range-compare




reply via email to

[Prev in Thread] Current Thread [Next in Thread]