[PATCH] D19821: [EarlyCSE] Use MemorySSA if available.

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 20 16:06:52 PDT 2016


reames added a comment.

p.s. The newer structure of using the original fast path check with memory ssa as a backup which is optional if available is much cleaner than the original code.  I'm okay with something like this landing (once other reviewers have signed off) even if the compile time question isn't fully resolved provided that the force memory SSA pass is under an off by default option.  As structured, this doesn't complicate the existing code much at all.


https://reviews.llvm.org/D19821





More information about the llvm-commits mailing list