[InstCombine] Teach the udiv folding logic how to handle constant expressions.
authorAndrea Di Biagio <Andrea_DiBiagio@sn.scee.net>
Mon, 26 Sep 2016 12:07:23 +0000 (12:07 +0000)
committerAndrea Di Biagio <Andrea_DiBiagio@sn.scee.net>
Mon, 26 Sep 2016 12:07:23 +0000 (12:07 +0000)
commita82d52d11df261821c48f7fce5ec3db6d878cff5
tree0af1a9089d7de4340c016ea7116871ea30171512
parentc4ec11f45155f1f6fd45f2db751e19012b5320c0
[InstCombine] Teach the udiv folding logic how to handle constant expressions.

This patch fixes PR30366.

Function foldUDivShl() worked under the assumption that one of the values
in input to the function was always an instance of llvm::Instruction.
However, function visitUDivOperand() (the only user of foldUDivShl) was
clearly violating that precondition; internally, visitUDivOperand() uses pattern
matches to check the operands of a udiv. Pattern matchers for binary operators
know how to handle both Instruction and ConstantExpr values.

This patch fixes the problem in foldUDivShl(). Now we use pattern matchers
instead of explicit casts to Instruction. The reduced test case from PR30366
has been added to test file InstCombine/udiv-simplify.ll.

Differential Revision: https://reviews.llvm.org/D24565

llvm-svn: 282398
llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
llvm/test/Transforms/InstCombine/udiv-simplify.ll