[Lldb-commits] [lldb] [llvm] [llvm][lldb] Add check for incorrect target features (PR #180901)

[David Spickett via lldb-commits]

================
@@ -452,7 +477,9 @@ class Target {
                                          StringRef Features) const {
     if (!MCSubtargetInfoCtorFn)
       return nullptr;
-    return MCSubtargetInfoCtorFn(TheTriple, CPU, Features);
+    if (!isSignedFeatureList(Features))
----------------
DavidSpickett wrote:

As this is changing a key function in LLVM, I would have put this change in its own PR so that it's more clear to LLVM reviewers what they're looking at.

That said, hopefully we get their attention anyway, I just want to make sure that happens at some point.

https://github.com/llvm/llvm-project/pull/180901