[PATCH] D59879: [ARM][CMSE] Add commandline option and feature macro

[Todd Snider via Phabricator via cfe-commits]

snidertm added inline comments.


================
Comment at: lib/Basic/Targets/ARM.cpp:438
+    } else if (Feature == "+8msecext") {
+      if ((CPUProfile != "M" && CPUProfile != "B") || ArchVersion != 8) {
+        Diags.Report(diag::err_target_unsupported_mcmse) << CPU;
----------------
How does CPUProfile get a value of "B"? I thought any Cortex-M processor would set CPUProfile to "M". Is CMSE available on a processor besides Cortex-m33?


Repository:
  rC Clang

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

https://reviews.llvm.org/D59879