[PATCH] D98264: [AArch64] Implement __rndr, __rndrrs intrinsics

Stelios Ioannou via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 12 07:04:36 PST 2021


stelios-arm added inline comments.


================
Comment at: clang/test/CodeGen/arm_acle.c:908
+#if __ARM_64BIT_STATE && defined(__ARM_FEATURE_RNG)
+// AArch64-v8_3-LABEL: @test_rndr(
+// AArch64-v8_3-NEXT:  entry:
----------------
SjoerdMeijer wrote:
> Not sure if I am surprised that this works.... This is (re)using tag `AArch64-v8_3` and for the other tests that use this and don't have RNG set, I was expecting FileCheck to complain about this, not sure if I am missing something though.
It turns out that this test is disabled (https://github.com/llvm/llvm-project/commit/6fcd4e080f09c9765d6e0ea03b1da91669c8509a). I am going to remove the changes in this file. 


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

https://reviews.llvm.org/D98264



More information about the cfe-commits mailing list