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

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 21 04:52:06 PDT 2022


fhahn added a comment.

In D122983#3464331 <https://reviews.llvm.org/D122983#3464331>, @xbolva00 wrote:

> In D122983#3464315 <https://reviews.llvm.org/D122983#3464315>, @aaron.ballman wrote:
>
>> 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.
>
> You probably need to fix Makefiles as well. I believe there were similar mysterious errors with -ffp-contract changes. Try to look at history of llvm test-suite.
>
> cc @fhahn if he can remember more

Depending on how many tests need updating in llvm-test-suite, I think it would be better to only disable the warning per-test, after verifying the warning works as expected.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122983



More information about the cfe-commits mailing list