[PATCH] D137073: [clang] Fix inline builtin functions of an __asm__ renamed function with symbol prefixes
Martin Storsjö via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 2 13:25:46 PDT 2022
This revision was automatically updated to reflect the committed changes.
Closed by commit rG9b3834ef67e3: [clang] Fix inline builtin functions of an __asm__ renamed function with symbol… (authored by mstorsjo).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137073/new/
https://reviews.llvm.org/D137073
Files:
clang/lib/CodeGen/CGExpr.cpp
clang/test/CodeGen/inline-builtin-asm-name.c
Index: clang/test/CodeGen/inline-builtin-asm-name.c
===================================================================
--- /dev/null
+++ clang/test/CodeGen/inline-builtin-asm-name.c
@@ -0,0 +1,32 @@
+// RUN: %clang_cc1 -triple i686-windows-gnu -emit-llvm -o - %s -disable-llvm-optzns | FileCheck %s
+
+// CHECK: call i32 @"\01_asm_func_name.inline"
+
+// CHECK: declare dso_local i32 @"\01_asm_func_name"(ptr noundef, i32 noundef, ptr noundef, ptr noundef)
+
+// CHECK: define internal i32 @"\01_asm_func_name.inline"
+
+// CHECK: call i32 @__mingw_vsnprintf
+
+// CHECK: declare dso_local i32 @__mingw_vsnprintf
+
+typedef unsigned int size_t;
+
+int __mingw_vsnprintf(char *_DstBuf, size_t _MaxCount, const char *_Format, __builtin_va_list _ArgList);
+
+// For the real use case, "_asm_func_name" is actually "___mingw_vsnprintf", but it's renamed in the testcase for disambiguation.
+int vsnprintf(char *__stream, size_t __n, const char *__format, __builtin_va_list __local_argv) __asm__("_asm_func_name");
+
+extern __inline__ __attribute__((__always_inline__, __gnu_inline__))
+int vsnprintf(char *__stream, size_t __n, const char *__format, __builtin_va_list __local_argv)
+{
+ return __mingw_vsnprintf(__stream, __n, __format, __local_argv);
+}
+
+void call(const char* fmt, ...) {
+ char buf[200];
+ __builtin_va_list ap;
+ __builtin_va_start(ap, fmt);
+ vsnprintf(buf, sizeof(buf), fmt, ap);
+ __builtin_va_end(ap);
+}
Index: clang/lib/CodeGen/CGExpr.cpp
===================================================================
--- clang/lib/CodeGen/CGExpr.cpp
+++ clang/lib/CodeGen/CGExpr.cpp
@@ -5014,8 +5014,7 @@
std::string NoBuiltinFD = ("no-builtin-" + FD->getName()).str();
std::string NoBuiltins = "no-builtins";
- auto *A = FD->getAttr<AsmLabelAttr>();
- StringRef Ident = A ? A->getLabel() : FD->getName();
+ StringRef Ident = CGF.CGM.getMangledName(GD);
std::string FDInlineName = (Ident + ".inline").str();
bool IsPredefinedLibFunction =
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D137073.472739.patch
Type: text/x-patch
Size: 1994 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221102/561445c5/attachment.bin>
More information about the cfe-commits
mailing list