[clang] [Clang] Allow LibBuiltins with placeholder return types to be called (PR #101702)

via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 2 10:10:03 PDT 2024


================
@@ -4895,7 +4895,7 @@ void Sema::InstantiateFunctionDefinition(SourceLocation PointOfInstantiation,
   // Never implicitly instantiate a builtin; we don't actually need a function
   // body.
   if (Function->getBuiltinID() && TSK == TSK_ImplicitInstantiation &&
-      !DefinitionRequired)
+      !DefinitionRequired && !Function->getReturnType()->isUndeducedType())
----------------
Sirraide wrote:

I think this ends up affecting more builtins than just the ones we care about here. I’m not sure that’s a good idea, but I’m also not too familiar w/ builtins.

https://github.com/llvm/llvm-project/pull/101702


More information about the cfe-commits mailing list