[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