[PATCH] D68684: [WebAssembly] Make returns variadic

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 9 14:02:53 PDT 2019


aheejin accepted this revision.
aheejin added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyFastISel.cpp:1309
 
+  // TODO: should probably be <= 2? But be conservative to start...
+  assert(Ret->getNumOperands() < 2 && "Multivalue return not supported yet");
----------------
Why <=2? If multivalues are supported in fastisel later, do we still have that limit?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68684





More information about the llvm-commits mailing list