diff options
| author | Devang Patel <dpatel@apple.com> | 2006-10-19 20:59:13 +0000 |
|---|---|---|
| committer | Devang Patel <dpatel@apple.com> | 2006-10-19 20:59:13 +0000 |
| commit | 5d6df959e3cd897398cba5c25f9cb1d4f319f001 (patch) | |
| tree | db534dd5ec5f8d9e243425b103f6c0501bd3ba6e /llvm/lib/Transforms | |
| parent | 88afd00d1d2aa1533d990f54431cff1ed9296f47 (diff) | |
| download | bcm5719-llvm-5d6df959e3cd897398cba5c25f9cb1d4f319f001.tar.gz bcm5719-llvm-5d6df959e3cd897398cba5c25f9cb1d4f319f001.zip | |
It is OK to remove extra cast if operation is EQ/NE even though source
and destination sign may not match but other conditions are met.
llvm-svn: 31056
Diffstat (limited to 'llvm/lib/Transforms')
| -rw-r--r-- | llvm/lib/Transforms/Scalar/InstructionCombining.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Scalar/InstructionCombining.cpp b/llvm/lib/Transforms/Scalar/InstructionCombining.cpp index 7e75dcab1df..97b7cebbb68 100644 --- a/llvm/lib/Transforms/Scalar/InstructionCombining.cpp +++ b/llvm/lib/Transforms/Scalar/InstructionCombining.cpp @@ -4795,8 +4795,9 @@ Instruction *InstCombiner::visitSetCondInstWithCastAndCast(SetCondInst &SCI) { // %B = setgt short %X, 1330 // // because %A may have negative value. - // However, it is OK if SrcTy is bool. See cast-set.ll testcase. - if (isSignSrc == isSignDest || SrcTy == Type::BoolTy) + // However, it is OK if SrcTy is bool (See cast-set.ll testcase) + // OR operation is EQ/NE. + if (isSignSrc == isSignDest || SrcTy == Type::BoolTy || SCI.isEquality()) RHSCIOp = Res; else return 0; |
