[PATCH] D15281: [ARMv8-M] [8/9] Add new system registers to ARMv8-M Baseline/Mainline

[Tim Northover via llvm-commits]

t.p.northover added inline comments.

================
Comment at: lib/Target/ARM/ARMISelDAGToDAG.cpp:3765
@@ +3764,3 @@
+      Flags = Flags.drop_front(3);
+      if (Flags.startswith("_"))
+        Flags = Flags.drop_front();
----------------
bsmith wrote:
> t.p.northover wrote:
> > Won't this allow "basepri_maxns"? Seems a bit inconsistent with the others.
> It will reconstruct Reg = basepri, Flags = max{_ns,} into Reg = basepri_max, Flags = {ns,}, the call to getMClassRegisterMask afterwards will check whether or not the ns flags are allowed.
But what if the second '_' is skipped? It looks like we'd treat Flags == "max_ns" and Flags == "maxns" equivalently, which seems like an unnecessary quirk.


Repository:
  rL LLVM

http://reviews.llvm.org/D15281