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

JinGu Kang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 4 08:23:45 PST 2021


jaykang10 added inline comments.


================
Comment at: clang/lib/Basic/Targets/AArch64.cpp:532-533
       HasFlagM = true;
+    if (Feature == "+frint3264")
+      HasFRInt3264 = true;
   }
----------------
ktkachov wrote:
> 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?
You are right! I will update it.


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

https://reviews.llvm.org/D97949



More information about the llvm-commits mailing list