3 ms·
It's actually not. It's what GVNHoist does, but not MLSM. MLSM only handles diamonds. Fine, it's what MLSM aspires to: http://llvm-cs.pcc.me.uk/lib/Transforms/
by mindleyhilner 10y ago
It's actually not. It's what GVNHoist does, but not MLSM. MLSM only handles diamonds.
Fine, it's what MLSM aspires to: http://llvm-cs.pcc.me.uk/lib/Transforms/Scalar/MergedLoadStoreMotion.cpp#71 http://llvm-cs.pcc.me.uk/lib/Transforms/Scalar/MergedLoadSto... (:
- DannyBee 10y agoWriting "TODO: generalize to other regions" in a crazy N^3/N^4 approach to merging load/stores is like writing "TODO: solve prime number conjecture" in somebody's hand-rolled two's complement addition code. Maybe it happens, but it ain't gonna happen with anything like this code :)