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

Jacob Gravelle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 17:25:08 PST 2019


jgravelle-google accepted this revision.
jgravelle-google added a comment.
This revision is now accepted and ready to land.

Nice, lgtm



================
Comment at: lib/Target/WebAssembly/WebAssemblyISelLowering.cpp:677
+        const ISD::OutputArg &Out = Outs[I];
+        SDValue &Arg = OutVals[I];
       EVT VT = Arg.getValueType();
----------------
Is it this diff or is the spacing off here?


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