[PATCH] D58656: [WebAssembly] Properly align fp128 arguments in outgoing varargs arguments

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 20:47:01 PST 2019


sunfish marked 2 inline comments as done.
sunfish added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyISelLowering.cpp:677
+        const ISD::OutputArg &Out = Outs[I];
+        SDValue &Arg = OutVals[I];
       EVT VT = Arg.getValueType();
----------------
jgravelle-google wrote:
> Is it this diff or is the spacing off here?
I was somehow in 4-space indent mode there. Fixed.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58656





More information about the llvm-commits mailing list