From 5b384c3015100ad815f4d994d7ef35cc947db711 Mon Sep 17 00:00:00 2001 From: David Spickett Date: Fri, 11 Apr 2025 13:14:16 +0000 Subject: [PATCH] [lldb][test] Adjust TestTargetReadInstructionsFlavor skipIfs Original in https://github.com/llvm/llvm-project/pull/134626 was written as if it was "this or this" but it's "this and this". So the test ran on AArch64 Linux, because Linux is not Windows. Split out the Windows check to fix that. --- .../TestTargetReadInstructionsFlavor.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lldb/test/API/python_api/target/read-instructions-flavor/TestTargetReadInstructionsFlavor.py b/lldb/test/API/python_api/target/read-instructions-flavor/TestTargetReadInstructionsFlavor.py index 40edc57df21c..12805985798d 100644 --- a/lldb/test/API/python_api/target/read-instructions-flavor/TestTargetReadInstructionsFlavor.py +++ b/lldb/test/API/python_api/target/read-instructions-flavor/TestTargetReadInstructionsFlavor.py @@ -7,7 +7,8 @@ from lldbsuite.test.lldbtest import * class TargetReadInstructionsFlavor(TestBase): - @skipIf(archs=no_match(["x86_64", "x86", "i386"]), oslist=["windows"]) + @skipIfWindows + @skipIf(archs=no_match(["x86_64", "x86", "i386"])) def test_read_instructions_with_flavor(self): self.build() executable = self.getBuildArtifact("a.out")