[PATCH] D122983: [C11/C2x] Change the behavior of the implicit function declaration warning

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 21 04:15:16 PDT 2022

aaron.ballman added a comment.

In D122983#3463600 <https://reviews.llvm.org/D122983#3463600>, @nemanjai wrote:

> In D122983#3463569 <https://reviews.llvm.org/D122983#3463569>, @xbolva00 wrote:
>> But your link shows failures in compiler-rt/, not in llvm test-suite
> It shows both. But the `compiler-rt` ones have subsequently been fixed while the `test-suite` ones are still failing as of https://lab.llvm.org/buildbot/#/builders/105/builds/24314.

Thanks for pointing this out -- I'm baffled what's going on with that bot, because 1) other bots running the test suite are fine, and 2) the test suite was already fixed to disable the diagnostics (which is what fixed things for #1):


(and so on). It's almost as if something on that particular bot is overriding the cmake flags (or perhaps using CFLAGS instead of CPPFLAGS)... Here's the command line that's failing for bison's symtab.c:

RunSafely.sh detected a failure with these command-line arguments:  --show-errors -t /home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/test/sandbox/build/tools/timeit 500 /dev/null Output/symtab.llvm.o.compile /home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/stage1.install/bin/clang -I/home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/test/sandbox/build/MultiSource/Benchmarks/Prolangs-C/bison -I/home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/test/test-suite/MultiSource/Benchmarks/Prolangs-C/bison -I/home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/test/test-suite/include -I../../../../include -D_GNU_SOURCE -D__STDC_LIMIT_MACROS -DNDEBUG -O3 -ffp-contract=off -fomit-frame-pointer -c /home/buildbots/ppc64le-clang-lnt-test/clang-ppc64le-lnt/test/test-suite/MultiSource/Benchmarks/Prolangs-C/bison/symtab.c -o Output/symtab.llvm.o

Note how it's missing the warning flag from the CMakeLists.txt file.

So I have no idea what's going on (I can't even really run the tests; I'm on Windows and test-suite requires perl to run).

In D122983#3463561 <https://reviews.llvm.org/D122983#3463561>, @MaskRay wrote:

> Adding a global `-Wno-implicit-function-declaration` looks good. @Meinersbur

+1 to this idea, but I'm worried it won't fix that particular bot.

  rG LLVM Github Monorepo



More information about the cfe-commits mailing list