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):

https://github.com/llvm/llvm-test-suite/blob/main/MultiSource/Benchmarks/Prolangs-C/archie-client/CMakeLists.txt#L2
https://github.com/llvm/llvm-test-suite/blob/main/MultiSource/Benchmarks/Prolangs-C/bison/CMakeLists.txt#L1

(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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122983/new/

https://reviews.llvm.org/D122983

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to