[PATCH] D133800: [Clang 15.0.1] Downgrade implicit int and implicit function declaration to warning only

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 14 04:25:30 PDT 2022


aaron.ballman added a comment.

In D133800#3788950 <https://reviews.llvm.org/D133800#3788950>, @nikic wrote:

> I tested this patch as well and saw the same failures as @mgorny.

Thank you, I think I've got those patched up, but with one difference:

>   <snip>
>   diff --git a/clang/test/CodeGen/neon-crypto.c b/clang/test/CodeGen/neon-crypto.c
>   index 916886574416..1b29b1a53096 100644
>   --- a/clang/test/CodeGen/neon-crypto.c
>   +++ b/clang/test/CodeGen/neon-crypto.c
>   @@ -14,7 +14,7 @@
>    
>    uint8x16_t test_vaeseq_u8(uint8x16_t data, uint8x16_t key) {
>      // CHECK-LABEL: @test_vaeseq_u8
>   -  // CHECK-NO-CRYPTO: error: call to undeclared function 'vaeseq_u8'
>   +  // CHECK-NO-CRYPTO: warning: call to undeclared function 'vaeseq_u8'
>      return vaeseq_u8(data, key);
>      // CHECK: call <16 x i8> @llvm.{{arm.neon|aarch64.crypto}}.aese(<16 x i8> %data, <16 x i8> %key)
>    }

Because this was the only `error` in the file, I changed the `RUN` line to remove the `not` in front of `%clang_cc1`, as that seemed like it would be necessary to get the test to pass as well.


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

https://reviews.llvm.org/D133800



More information about the cfe-commits mailing list