[PATCH] D14122: [SimplifyCFG] Trim duplicate basic blocks in switch cases

Dylan McKay via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 28 22:24:58 PDT 2015


dylanmckay abandoned this revision.
dylanmckay added a comment.

Upon further reflection, I believe regular expressions are the best choice (as the x86 tests shouldn't care about the names of basic blocks - they should only care that i.e. the switch statements are converted to selects).

I will fix the tests properly, remove the extraneous command line param, generalise to a basic block prefix merge optimisation and open a new diff once this is done.

Thanks for the reviews Sanjoy and Phillip, I will make sure to address all points in the next patch.


http://reviews.llvm.org/D14122





More information about the llvm-commits mailing list