[PATCH] D11691: [WebAssembly] Add Relooper

Dan Gohman dan433584 at gmail.com
Mon Aug 3 13:11:49 PDT 2015


sunfish added inline comments.

================
Comment at: lib/Target/WebAssembly/CMakeLists.txt:12
@@ -11,2 +11,3 @@
 add_llvm_target(WebAssemblyCodeGen
+  Relooper.cpp
   WebAssemblyAsmPrinter.cpp
----------------
jfb wrote:
> arsenm wrote:
> > Will this pass purely be doing the structurizing, or will it also be responsible for insertion of the higher level control flow instructions? I think one of the many failings of AMDILCFGStructurizer is that it attempts to do both at the same time / in the same pass.
> It identifies the structure, but acts like an `AnalysisPass` in that it just passes that info onto later code.
The current Relooper algorithm works on arbitrary CFGs, so it doesn't need to do any structurizing.


Repository:
  rL LLVM

http://reviews.llvm.org/D11691







More information about the llvm-commits mailing list