[PATCH] D58591: [WebAssembly] Remove unnecessary instructions after TRY marker placement

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 20:11:48 PST 2019


aheejin added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyCFGStackify.cpp:847
+  // Remove unnecessary instructions to reduce code size.
+  removeUnnecessaryInstrs(MF);
+
----------------
dschuff wrote:
> should this unconditional run still be here?
No. I forgot to delete it. Thanks for the catch.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58591





More information about the llvm-commits mailing list