[PATCH] D31032: [LoadCombine] Avoid analysing dead basic blocks

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added a comment.

KennethH gave me permission to upload the improved solution (that uses DominatorTree analysis). This feels like a more "kosher" solution.

The currently suggested solution (for avoiding to optimize on dead code in LoadCombine) is similar to the solution used in BBVectorize. So there is really nothing new invented here.

I'm just afraid that I might have missed something fundamental about "adding a new analysis" (my experience in playing around with analysis passes is very limited).
So any feedback is welcome!


https://reviews.llvm.org/D31032