================
@@ -2380,7 +2380,7 @@ FunctionDecl *Sema::CreateBuiltin(IdentifierInfo *II, 
QualType Type,
   }
 
   FunctionDecl *New = FunctionDecl::Create(Context, Parent, Loc, Loc, II, Type,
-                                           /*TInfo=*/nullptr, SC_Extern,
+                                           /*TInfo=*/nullptr, SC_None,
----------------
AaronBallman wrote:

This change looks suspicious to me -- this is creating a builtin function, 
which should be modeled as an extern. In C++, you seem to be relying on the 
language linkage being sufficient for that, but C has no notion of language 
linkage and so I worry that the declaration for the builtin will be incorrectly 
handled in C.

https://github.com/llvm/llvm-project/pull/93913
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to