xbolva00 added a comment.

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

> This is still causing failures when building `test-suite`:
> https://lab.llvm.org/buildbot/#/builders/105/builds/24292
>
> Why don't we just turn off this warning for the entire `test-suite` since I 
> don't expect we want to modify the tests (as some of them have licenses that 
> probably don't allow us to modify them)?
> We could do something like this:
>
>   diff --git a/CMakeLists.txt b/CMakeLists.txt
>   index 8d76a45c7..032281ecf 100644
>   --- a/CMakeLists.txt
>   +++ b/CMakeLists.txt
>   @@ -47,6 +47,7 @@ set(LLVM_CODESIGNING_IDENTITY "" CACHE STRING
>      "Sign executables and dylibs with the given identity or skip if empty 
> (Darwin Only)")
>    
>    add_definitions(-DNDEBUG)
>   +add_definitions(-Wno-implicit-function-declaration)
>    option(TEST_SUITE_SUPPRESS_WARNINGS "Suppress all warnings" ON)
>    if(${TEST_SUITE_SUPPRESS_WARNINGS})
>      add_definitions(-w)

But your link shows failures in compiler-rt/, not in llvm test-suite


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