Skip to content

Conversation

@bulbazord
Copy link
Member

Reverts #166769

Darwin platforms prefix symbols with _, other platforms don't necessarily.

@llvmbot
Copy link
Member

llvmbot commented Nov 7, 2025

@llvm/pr-subscribers-lldb

Author: Alex Langford (bulbazord)

Changes

Reverts llvm/llvm-project#166769

Darwin platforms prefix symbols with _, other platforms don't necessarily.


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

1 Files Affected:

  • (modified) lldb/test/Shell/Unwind/Inputs/call-asm.c (+2-1)
diff --git a/lldb/test/Shell/Unwind/Inputs/call-asm.c b/lldb/test/Shell/Unwind/Inputs/call-asm.c
index 30d0069504172..b154c1ac1385d 100644
--- a/lldb/test/Shell/Unwind/Inputs/call-asm.c
+++ b/lldb/test/Shell/Unwind/Inputs/call-asm.c
@@ -1,2 +1,3 @@
-int asm_main();
+int asm_main() asm("asm_main");
+
 int main() { return asm_main(); }

@bulbazord bulbazord enabled auto-merge (squash) November 7, 2025 17:31
@bulbazord bulbazord merged commit 9cca883 into main Nov 7, 2025
10 of 11 checks passed
@bulbazord bulbazord deleted the revert-166769-lldb_gnu_asm_call branch November 7, 2025 17:38
vinay-deshmukh pushed a commit to vinay-deshmukh/llvm-project that referenced this pull request Nov 8, 2025
…ly call" (llvm#166970)

Reverts llvm#166769

Darwin platforms prefix symbols with `_`, other platforms don't
necessarily.
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