================ @@ -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