From 3d668d392898f5ed8b893d356ab79e301c53f42c Mon Sep 17 00:00:00 2001 From: Jacob Gravelle Date: Thu, 16 Aug 2018 19:24:31 +0000 Subject: [PATCH] [WebAssembly] Remove temporary workaround for function bitcasts Summary: EM_ASM no longer is lowered as varargs in C, so this workaround is obsolete. Reviewers: dschuff, sunfish Subscribers: sbc100, aheejin, llvm-commits Differential Revision: https://reviews.llvm.org/D50859 llvm-svn: 339925 --- llvm/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp | 5 ----- 1 file changed, 5 deletions(-) diff --git a/llvm/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp index 4f6dcb1..742fe40 100644 --- a/llvm/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp +++ b/llvm/lib/Target/WebAssembly/WebAssemblyFixFunctionBitcasts.cpp @@ -257,11 +257,6 @@ bool FixFunctionBitcasts::runOnModule(Module &M) { if (!Ty) continue; - // Bitcasted vararg functions occur in Emscripten's implementation of - // EM_ASM, so suppress wrappers for them for now. - if (TemporaryWorkarounds && (Ty->isVarArg() || F->isVarArg())) - continue; - auto Pair = Wrappers.insert(std::make_pair(std::make_pair(F, Ty), nullptr)); if (Pair.second) Pair.first->second = CreateWrapper(F, Ty); -- 2.7.4