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
- Build Status
Buildable 28790 Build 28789: arc lint + arc unit
Event Timeline
Comment Actions
Code change lgtm, thanks!
lib/Target/WebAssembly/WebAssemblyTargetMachine.cpp | ||
---|---|---|
293 | 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...) |
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...)