[Lldb-commits] [lldb] [lldb] Disable arm64e tests under ASan (PR #189052)

[via lldb-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: Jonas Devlieghere (JDevlieghere)

<details>
<summary>Changes</summary>

Technically ASan is supported, but we need an arm64e sanitizer runtime. It's still enabled when the whole test suite runs as arm64e, assuming that you need arm64e runtimes regardless.

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


1 Files Affected:

- (modified) lldb/packages/Python/lldbsuite/test/decorators.py (+7) 


``````````diff
diff --git a/lldb/packages/Python/lldbsuite/test/decorators.py b/lldb/packages/Python/lldbsuite/test/decorators.py
index 02c30f96d85c5..8fd38c62f7b16 100644
--- a/lldb/packages/Python/lldbsuite/test/decorators.py
+++ b/lldb/packages/Python/lldbsuite/test/decorators.py
@@ -1196,6 +1196,7 @@ def does_compiler_support_flag():
 
     return skipTestIfFn(does_compiler_support_flag)
 
+
 def skipIfAsan(func):
     """Skip this test if the environment is set up to run LLDB *itself* under ASAN."""
     return skipTestIfFn(is_running_under_asan)(func)
@@ -1345,6 +1346,12 @@ def can_build_and_run_arm64e():
         if not configuration.arm64e_debugserver:
             return "debugserver not built with arm64e support"
 
+        # Technically ASan is supported, but we need an arm64e sanitizer
+        # runtime and we assume that's not the case unless we run the whole
+        # test suite as arm64e.
+        if is_running_under_asan():
+            return "Sanitizer runtime may not support arm64e"
+
         return None
 
     return skipTestIfFn(can_build_and_run_arm64e)(func)

``````````

</details>


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