[PATCH] D16610: [WebAssembly] Implement byval arguments

Derek Schuff via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 15:47:20 PST 2016


dschuff added inline comments.

================
Comment at: lib/Target/WebAssembly/WebAssemblyISelLowering.cpp:354
@@ -353,3 +353,1 @@
 
-  SmallVectorImpl<SDValue> &OutVals = CLI.OutVals;
-
----------------
This CL makes a new vector with copies of all the non-byval OutVals. Is that necessary, or can I overwrite CLI.OutVals?


http://reviews.llvm.org/D16610





More information about the llvm-commits mailing list