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

LGTM, but it might be good to wait with committing until next week, so people 
in the US have a chance to take a look as well.



================
Comment at: clang/lib/Driver/ToolChains/Arch/AArch64.cpp:143
     MtuneLowerCase = llvm::sys::getHostCPUName();
-  if (MtuneLowerCase == "cyclone") {
+  if (MtuneLowerCase == "cyclone" || MtuneLowerCase.find("apple") == 0) {
     Features.push_back("+zcm");
----------------
It might be slightly more obvious to use MtuneLowerCAse.StartsWith("apple")


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

https://reviews.llvm.org/D70779



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to