[PATCH] Teaching loop simplify to preserve LCSSA
Dinesh Dwivedi
dinesh.d at samsung.com
Fri May 9 16:12:15 PDT 2014
Yes,
While working on PR18861 [http://reviews.llvm.org/D2976], I saw Chandler's comment in LICM.cpp about formLCSSARecursively() being heavy.
I observed that formLCSSARecursively() is added in LoopUnroll to fix LCSSA breakage during loop simplify which is happening due to deletion of PHI nodes with single incoming value. As per my understanding, keeping few PHI nodes and removing them later is much cheaper than formLCSSARecursively() and that's why this patch.
http://reviews.llvm.org/D2986
More information about the llvm-commits
mailing list