Skip to content

Conversation

@Nerixyz
Copy link
Contributor

@Nerixyz Nerixyz commented Oct 21, 2025

The test builds files for Windows, so the target has to be required. I didn't add this in #163733.

Fixes the failure from #163733 (comment).

@Nerixyz Nerixyz requested a review from JDevlieghere as a code owner October 21, 2025 12:18
@llvmbot llvmbot added the lldb label Oct 21, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 21, 2025

@llvm/pr-subscribers-lldb

Author: nerix (Nerixyz)

Changes

The test builds files for Windows, so the target has to be required.

Fixes the failure from #163733 (comment).


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

1 Files Affected:

  • (modified) lldb/test/Shell/SymbolFile/NativePDB/func-symbols.test (+1-1)
diff --git a/lldb/test/Shell/SymbolFile/NativePDB/func-symbols.test b/lldb/test/Shell/SymbolFile/NativePDB/func-symbols.test
index 33ffad6b2f20f..ee96fa8fd5c33 100644
--- a/lldb/test/Shell/SymbolFile/NativePDB/func-symbols.test
+++ b/lldb/test/Shell/SymbolFile/NativePDB/func-symbols.test
@@ -1,4 +1,4 @@
-# REQUIRES: lld
+# REQUIRES: lld, target-windows
 
 # Test that functions have the correct types.
 # This uses the same input as SymbolFile/PDB/func-symbols.test. However, DIA 

@Nerixyz Nerixyz enabled auto-merge (squash) October 21, 2025 12:22
@Nerixyz Nerixyz merged commit d9556d3 into llvm:main Oct 21, 2025
9 of 11 checks passed
Lukacma pushed a commit to Lukacma/llvm-project that referenced this pull request Oct 29, 2025
…lvm#164406)

The test builds files for Windows, so the target has to be required. I
didn't add this in llvm#163733.

Fixes the failure from
llvm#163733 (comment).
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.

2 participants