[PATCH] D17859: [InstCombine] convert 'isPositive' and 'isNegative' vector comparisons to shifts (PR26701, PR26819)

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Fri May 13 07:05:04 PDT 2016


spatel added inline comments.

================
Comment at: lib/Transforms/InstCombine/InstCombineCompares.cpp:3145-3146
@@ +3144,4 @@
+    // Unlike the scalar case above, we can't rely on any previous
+    // simplification to handle this for us (although it would make sense for
+    // InstSimplify to handle cases with min/max value *splatted* constants).
+    
----------------
chandlerc wrote:
> I'd not have the comment talk about a missing instsimplify pass as we'll likely fail to remove it when adding that...
Good point. I'll replace this with a blurb about the trade-offs that we discussed here. Thanks!


http://reviews.llvm.org/D17859





More information about the llvm-commits mailing list