4 Commits

Author	SHA1	Message	Date
Benjamin Kramer	36892682d9	[InstCombine] Don't crash when trying to take an element of a ConstantExpr. ... Fixes PR27786. llvm-svn: 269757	2016-05-17 12:08:55 +00:00
Sanjay Patel	59f49118ec	[InstCombine] check vector elements before trying to transform LE/GE vector icmp (PR27756) ... Fix a bug introduced with rL269426 : [InstCombine] canonicalize* LE/GE vector integer comparisons to LT/GT (PR26701, PR26819) We were assuming that a ConstantDataVector / ConstantVector / ConstantAggregateZero operand of an ICMP was composed of ConstantInt elements, but it might have ConstantExpr or UndefValue elements. Handle those appropriately. Also, refactor this function to join the scalar and vector paths and eliminate the switches. Differential Revision: http://reviews.llvm.org/D20289 llvm-svn: 269728	2016-05-17 00:57:57 +00:00
Sanjay Patel	e6e6eb6572	[InstCombine] handle zero constant vectors for LE/GE comparisons too ... Enhancement to: http://reviews.llvm.org/rL269426 With discussion in: http://reviews.llvm.org/D17859 This should complete the fixes for: PR26701, PR26819: https://llvm.org/bugs/show_bug.cgi?id=26701 https://llvm.org/bugs/show_bug.cgi?id=26819 llvm-svn: 269439	2016-05-13 17:28:12 +00:00
Sanjay Patel	2c688c8b52	[InstCombine] canonicalize* LE/GE vector integer comparisons to LT/GT (PR26701, PR26819) ... *We don't currently handle the edge case constants (min/max values), so it's not a complete canonicalization. To fully solve the motivating bugs, we need to enhance this to recognize a zero vector too because that's a ConstantAggregateZero which is a ConstantData, not a ConstantVector or a ConstantDataVector. Differential Revision: http://reviews.llvm.org/D17859 llvm-svn: 269426	2016-05-13 15:10:46 +00:00