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

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 23 15:09:30 PDT 2015


chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

I've been wanting to kill the SSAUpdater path in SROA since i started rewriting it. I could swear that when I was rewriting it, adding it here forced a second run of DomTree.... But your analysis is clearly correct, and we no longer need this. I'll proceed with ripping out all the complexity SSAUpdater imposes later. There are still more compile time wins when we *know* that we have domtree here.


http://reviews.llvm.org/D12267





More information about the llvm-commits mailing list