[PATCH] D97949: [AArch64] Add missing intrinsics for vrnd

[JinGu Kang via Phabricator via llvm-commits]

jaykang10 marked an inline comment as not done.
jaykang10 added inline comments.


================
Comment at: clang/include/clang/Basic/arm_neon.td:1213
 
+let ArchGuard = "__ARM_ARCH >= 8 && defined(__aarch64__) && defined(__ARM_FEATURE_FRINT3264)" in {
+def FRINT32X_S32 : SInst<"vrnd32x", "..", "fQf">;
----------------
ktkachov wrote:
> Clang should use the same feature guard as GCC here, as it is mandated by ACLE: __ARM_FEATURE_FRINT
> See the relevant section in https://developer.arm.com/documentation/101028/0012/8--Data-processing-intrinsics
Thanks for letting know!!! That was what I want to know.


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

https://reviews.llvm.org/D97949