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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
