tmatheson accepted this revision.
tmatheson added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: clang/lib/Basic/Targets/AArch64.cpp:420
+    Builder.defineMacro("__ARM_FEATURE_RCPC", "3");
+  else if (HasRCPC)
     Builder.defineMacro("__ARM_FEATURE_RCPC", "1");
----------------
I didn't realise we hadn't implemented RPCP2 here. If we aren't aware of anyone 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153130

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D153130: [Clang][AA... Tomas Matheson via Phabricator via cfe-commits

Reply via email to