[PATCH] D118199: [AArch64] ACLE feature macro for Armv8.8-A MOPS

[Momchil Velikov via Phabricator via cfe-commits]

chill added inline comments.


================
Comment at: clang/lib/Basic/Targets/AArch64.cpp:229
                                                 MacroBuilder &Builder) const {
+  // FIXME: this does not handle the case where MOPS is disabled using +nomops
+  Builder.defineMacro("__ARM_FEATURE_MOPS", "1");
----------------
What's the deal with `"+nomops"` ? This FIXME sort of contradicts with an earlier comment


> Add support for +nomops




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118199