[Lldb-commits] [lldb] [lldb] Consolidating platform support checks in tests. (PR #184656)

[Jonas Devlieghere via lldb-commits]

================
@@ -59,7 +59,10 @@ class VariablesTest : public ::testing::Test {
   std::optional<llvm::sys::fs::TempFile> core;
   std::optional<llvm::sys::fs::TempFile> binary;
 
-  void CreateDebugger() { debugger = lldb::SBDebugger::Create(); }
+  void CreateDebugger() {
+    debugger = lldb::SBDebugger::Create();
+    SKIP_UNLESS_PLATFORM_SUPPORTED(debugger, "X86");
----------------
JDevlieghere wrote:

Isn't `GetBuildConfiguration` static? Maybe I'm missing something but I wouldn't expect us to need a debugger to check the supported targets.

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