Hacker News Reader

Comment by mindleyhilner | Hacker News Reader

  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/MergedLoadSto... (:

View on HN

Writing "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 :)

Reply on news.ycombinator.com