[lldb] Restrict JITLoaderGDB test to native Linux environments

[weliveindetail]

This test used to work on non-Linux platforms that could run simple ELF objects in a JIT session. However, there is a risk that this will become too unstable for CI, so let's limit it to what we actually need.

[llvmbot]

@llvm/pr-subscribers-lldb

Author: Stefan Gränitz (weliveindetail)

Changes

This test used to work on non-Linux platforms that could run simple ELF objects in a JIT session. However, there is a risk that this will become too unstable for CI, so let's limit it to what we actually need.

---

Full diff: https://github.com/llvm/llvm-project/pull/172292.diff

1 Files Affected:

• (modified) lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test (+2-3)

diff --git a/lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test b/lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test
index 9a972f1f1ece7..e2adc3a434798 100644
--- a/lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test
+++ b/lldb/test/Shell/Breakpoint/jit-loader_jitlink_elf.test
@@ -1,9 +1,8 @@
-# REQUIRES: target-x86_64
-# XFAIL: system-windows
+# REQUIRES: target-x86_64, system-linux, native
 
 # JITLink is the Orc-specific JIT linker implementation.
 #
-# RUN: %clangxx -g -S -emit-llvm -fPIC --target=x86_64-unknown-unknown-elf \
+# RUN: %clangxx -g -S -emit-llvm -fPIC \
 # RUN:          -o %t.ll %p/Inputs/jitbp.cpp
 # RUN: %lldb -b -o 'settings set plugin.jit-loader.gdb.enable on' -o 'b jitbp' \
 # RUN:          -o 'run --jit-linker=jitlink %t.ll' lli | FileCheck %s

[github-actions]

🐧 Linux x64 Test Results

• 33243 tests passed
• 503 tests skipped

✅ The build succeeded and all tests passed.

[charles-zablit]

Should we do the same for the former jit-loader_rtdyld_elf.test (which is now an API test)? I understand that this is what you suggest here: #170333 (comment)

[weliveindetail]

Should we do the same for the former jit-loader_rtdyld_elf.test (which is now an API test)?

You can consider it if you keep seeing flakes, yes