[PATCH] D30537: [DAGCombiner] allow transforming (select Cond, C +/- 1, C) to (add(ext Cond), C)

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 11:21:30 PST 2017


efriedma added a comment.

All of the testcase changes you're showing here involve a select where the condition is a TRUNCATE (so the extension is free, so it's profitable to transform on all platforms).  Does this have any impact on the more common patterns, like SELECT(SETCC)?


https://reviews.llvm.org/D30537





More information about the llvm-commits mailing list