Skip to content

Conversation

Nerixyz
Copy link
Contributor

@Nerixyz Nerixyz commented Oct 7, 2025

Fixes the test failure from #161678 (comment).

@Nerixyz Nerixyz requested a review from JDevlieghere as a code owner October 7, 2025 17:53
@llvmbot llvmbot added the lldb label Oct 7, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 7, 2025

@llvm/pr-subscribers-lldb

Author: nerix (Nerixyz)

Changes

Fixes the test failure from #161678 (comment).


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

1 Files Affected:

  • (modified) lldb/test/Shell/SymbolFile/NativePDB/c-calling-conventions.cpp (+1-1)
diff --git a/lldb/test/Shell/SymbolFile/NativePDB/c-calling-conventions.cpp b/lldb/test/Shell/SymbolFile/NativePDB/c-calling-conventions.cpp
index 569eaace1baef..d1d0bb08dfec4 100644
--- a/lldb/test/Shell/SymbolFile/NativePDB/c-calling-conventions.cpp
+++ b/lldb/test/Shell/SymbolFile/NativePDB/c-calling-conventions.cpp
@@ -1,5 +1,5 @@
 // clang-format off
-// REQUIRES: lld, x86
+// REQUIRES: lld, (target-x86 || target-x86_64)
 
 // RUN: %build --compiler=clang-cl --arch=32 --nodefaultlib --output=%t-32.exe %s
 // RUN: lldb-test symbols %t-32.exe | FileCheck --check-prefixes CHECK-32,CHECK-BOTH %s

@Nerixyz Nerixyz merged commit c16d252 into llvm:main Oct 7, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants