================
@@ -804,6 +804,13 @@ def setUpCommands(cls):
             )
         return commands
 
+    def getDebugInfoSetupCommands(self):
+        if self.getDebugInfo() == "native-pdb":
----------------
DavidSpickett wrote:

That's if we want to go Pavels' way. I don't really mind.

The debug info category way is easier for me as a bot owner for sure. We could 
accept it as a temporary implementation, with all the usual risk that temporary 
implementations carry.

> particularly if the end goal is to remove the non-native plugin completely.

Also do we have a support plan for this? For example would we ship 22.x with 
both but one is the fallback, then 23.x removes the fallback, or can we just 
remove one of them? Idk how much chaos we expect to cause with that.

Because if it's going to need multiple release cycles to remove one then our 
"temporary" stuff becomes at least a year, and the dotest setting option is 
more acceptable.

https://github.com/llvm/llvm-project/pull/149305
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to