[PATCH] D18234: [WebAssembly] Stackify code emitted by eliminateFrameIndex and SP writeback

Dan Gohman via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 17 08:56:04 PDT 2016


sunfish accepted this revision.
sunfish added a comment.
This revision is now accepted and ready to land.

I'll review this more closely when I get more time, but at a first readthrough it looks good.


================
Comment at: lib/Target/WebAssembly/WebAssemblyRegisterInfo.cpp:64
@@ +63,3 @@
+
+static bool canStackify(const MachineInstr& Inst) {
+  unsigned Op = Inst.getOpcode();
----------------
Would it make sense to name this canStackifyOperand?


http://reviews.llvm.org/D18234





More information about the llvm-commits mailing list