[PATCH] D58488: [WebAssembly] Default to a reasonable signature in WebAssemblyAddMissingPrototypes

[Phabricator via Phabricator via llvm-commits]

This revision was not accepted when it landed; it landed in state "Needs Review".
This revision was automatically updated to reflect the committed changes.
Closed by commit rL354544: [WebAssembly] Default to something reasonable in WebAssemblyAddMissingPrototypes (authored by sbc, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D58488?vs=187718&id=187719#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58488/new/

https://reviews.llvm.org/D58488

Files:
  llvm/trunk/lib/Target/WebAssembly/WebAssemblyAddMissingPrototypes.cpp
  llvm/trunk/test/CodeGen/WebAssembly/add-prototypes.ll


Index: llvm/trunk/test/CodeGen/WebAssembly/add-prototypes.ll
===================================================================
--- llvm/trunk/test/CodeGen/WebAssembly/add-prototypes.ll
+++ llvm/trunk/test/CodeGen/WebAssembly/add-prototypes.ll
@@ -58,6 +58,9 @@
 
 declare void @func_param(i64 (...)*)
 
+; CHECK: declare void @func_not_called()
+declare void @func_not_called(...) #1
+
 ; CHECK: declare extern_weak i64 @foo(i32)
 declare extern_weak i64 @foo(...) #1
 
Index: llvm/trunk/lib/Target/WebAssembly/WebAssemblyAddMissingPrototypes.cpp
===================================================================
--- llvm/trunk/lib/Target/WebAssembly/WebAssemblyAddMissingPrototypes.cpp
+++ llvm/trunk/lib/Target/WebAssembly/WebAssemblyAddMissingPrototypes.cpp
@@ -86,7 +86,6 @@
     // Create a function prototype based on the first call site (first bitcast)
     // that we find.
     FunctionType *NewType = nullptr;
-    Function *NewF = nullptr;
     for (Use &U : F.uses()) {
       LLVM_DEBUG(dbgs() << "prototype-less use: " << F.getName() << "\n");
       LLVM_DEBUG(dbgs() << *U.getUser() << "\n");
@@ -97,10 +96,6 @@
             // Create a new function with the correct type
             NewType = DestType;
             LLVM_DEBUG(dbgs() << "found function type: " << *NewType << "\n");
-            NewF = Function::Create(NewType, F.getLinkage(), F.getName() + ".fixed_sig");
-            NewF->setAttributes(F.getAttributes());
-            NewF->removeFnAttr("no-prototype");
-            Replacements.emplace_back(&F, NewF);
           } else if (NewType != DestType) {
             errs() << "warning: prototype-less function used with "
                       "conflicting signatures: "
@@ -116,8 +111,19 @@
       LLVM_DEBUG(
           dbgs() << "could not derive a function prototype from usage: " +
                         F.getName() + "\n");
-      continue;
+      // We could not derive a type for this function.  In this case strip
+      // the isVarArg and make it a simple zero-arg function.  This has more
+      // chance of being correct.  The current signature of (...) is illegal in
+      // C since it doesn't have any arguments before the "...", we this at
+      // least makes it possible for this symbol to be resolved by the linker.
+      NewType = FunctionType::get(F.getFunctionType()->getReturnType(), false);
     }
+
+    Function *NewF =
+        Function::Create(NewType, F.getLinkage(), F.getName() + ".fixed_sig");
+    NewF->setAttributes(F.getAttributes());
+    NewF->removeFnAttr("no-prototype");
+    Replacements.emplace_back(&F, NewF);
   }
 
   for (auto &Pair : Replacements) {
@@ -126,7 +132,7 @@
     std::string Name = OldF->getName();
     M.getFunctionList().push_back(NewF);
     OldF->replaceAllUsesWith(
-      ConstantExpr::getPointerBitCastOrAddrSpaceCast(NewF, OldF->getType()));
+        ConstantExpr::getPointerBitCastOrAddrSpaceCast(NewF, OldF->getType()));
     OldF->eraseFromParent();
     NewF->setName(Name);
   }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58488.187719.patch
Type: text/x-patch
Size: 3002 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190221/611fe3f4/attachment.bin>