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

Kyrill Tkachov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 4 07:59:53 PST 2021


ktkachov added inline comments.


================
Comment at: clang/lib/Basic/Targets/AArch64.cpp:532-533
       HasFlagM = true;
+    if (Feature == "+frint3264")
+      HasFRInt3264 = true;
   }
----------------
I don't think this flag should exist as it's inconsistent with GCC. In GCC these instructions are enabled through -march=armv8.5-a . Generally, these intrinsics should use the same target as the assembly instructions?


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

https://reviews.llvm.org/D97949



More information about the llvm-commits mailing list