[PATCH] D30572: Remove equal BBs from a function

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 11 13:12:15 PST 2017


davide added a comment.

You fixed all the style comments, so thanks. But you glossed over the design questions I asked. In particular, before getting performance numbers, there are two questions unanswered:

1. Why can't you do this in the middle-end? I assume there's a valid reason, but I'll be happy if you can actually elaborate.
2. Granted we want this in the backend, why are you putting the pass at that specific point in the pipeline?


https://reviews.llvm.org/D30572





More information about the llvm-commits mailing list