[PATCH] D12267: Require Dominator Tree For SROA, improve compile-time

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 23 11:10:19 PDT 2015


joker.eph added a comment.

As discussed on IRC: apparently it is a known limitation of SSAUpdater. It tries to compute dominance on the fly and does not cope well with large CFGs. 
That said I'm looking independently if something trivial can be done there.

After this patch SROA will never be used with SSAUpdater in the current pipeline, but it is used by other places so we might encounter the same kind of issue with it in the future.

Chandler might comment here if it makes sense to keep the option to run SROA without the DominatorTree?


http://reviews.llvm.org/D12267





More information about the llvm-commits mailing list