This pass hurts code size for wasm and sometimes generates irreducible
control flow.
Context: https://github.com/emscripten-core/emscripten/pull/8233
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
Code change lgtm, thanks!
lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp | ||
---|---|---|
293 ↗ | (On Diff #189279) | I would change "and sometimes generates" to "because it can generate", since I believe the issue is just that of irreducibility. (In theory if we could run that pass with a guarantee it would not create irredicuble control flow then it might be useful for us, although maybe not even then because the wasm VM might change the block order anyhow...) |