[SCCP] Use constant ranges for casts.

[fhahn]

Summary

For casts with constant range operands, we can use
ConstantRange::castOp.

[merge_guards_bot]

Unit tests: fail. 61132 tests passed, 1 failed and 728 were skipped.

failed: Clang.SemaOpenCL/numbered-address-space.cl

clang-tidy: fail. Please fix clang-tidy findings.

clang-format: pass.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

[efriedma]

llvm/lib/Transforms/Scalar/SCCP.cpp

813

DL.getTypeSizeInBits(DestTy)?

[fhahn]

Use DL.getTypeSizeInBits.

[merge_guards_bot]

Unit tests: unknown.

clang-tidy: unknown.

clang-format: unknown.

Build artifacts: diff.json, console-log.txt

[fhahn]

Only cast between integer types with constant ranges.

[jdoerfert]

I am somehow missing the extra folding in the test case. What is folded now?

[fhahn]

In D71938#1885319, @jdoerfert wrote:

I am somehow missing the extra folding in the test case. What is folded now?

Nothing, the test make sure we skip int -> fp casts, rather than crash :)

[jdoerfert]

In D71938#1885336, @fhahn wrote:

In D71938#1885319, @jdoerfert wrote:

I am somehow missing the extra folding in the test case. What is folded now?

Nothing, the test make sure we skip int -> fp casts, rather than crash :)

Gotcha.

[fhahn]

Ping. I think all required patches should have landed over the last week or so.

Rebased.

[fhahn]

ping :)

[efriedma]

llvm/lib/Transforms/Scalar/SCCP.cpp

807

This is calling both markConstant and mergeInValue?

809

If I'm following correctly, both the source and destination must be integers, so this is one of bitcast/trunc/sext/zext?

[fhahn]

Thanks Eli!

Removed the unnecessary mergeIn.

llvm/lib/Transforms/Scalar/SCCP.cpp

807

This could just be markConstant as it was originally I think. Not sure if there are any scenarios where we would get constant after already having a (non-singleton) range.

809

Yep, otherwise we would mark the result of %tmp4 = sitofp i32 %r to double as constant range, if %r is a constant range.

[efriedma]

LGTM

[uweigand]

I'm now getting an assertion failure in the LNT test suite:
http://lab.llvm.org:8011/builders/clang-s390x-linux-lnt/builds/17826/steps/test-suite/logs/stdio

clang: /home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/llvm/llvm/lib/IR/Constants.cpp:1661: static llvm::Constant* llvm::ConstantExpr::getCast(unsigned int, llvm::Constant*, llvm::Type*, bool): Assertion `CastInst::castIsValid(opc, C, Ty) && "Invalid constantexpr cast!"' failed.
PLEASE submit a bug report to https://bugs.llvm.org/ and include the crash backtrace, preprocessed source, and associated run script.
Stack dump:
0. Program arguments: /home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang -DNDEBUG -O3 -DNDEBUG -w -Werror=date-time -w -o CMakeFiles/GCC-C-execute-scal-to-vec1.dir/scal-to-vec1.c.o -c /home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/test/test-suite/SingleSource/Regression/C/gcc-c-torture/execute/scal-to-vec1.c

1. <eof> parser at end of file
2. Per-module optimization passes
3. Running pass 'CallGraph Pass Manager' on module '/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/test/test-suite/SingleSource/Regression/C/gcc-c-torture/execute/scal-to-vec1.c'.
4. Running pass 'Sparse Conditional Constant Propagation' on function '@main'

[...]
#9 0x000002aa1a5b6b70 llvm::ConstantExpr::getCast(unsigned int, llvm::Constant*, llvm::Type*, bool) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x2ab6b70)
#10 0x000002aa19ec233c llvm::ConstantFoldCastOperand(unsigned int, llvm::Constant*, llvm::Type*, llvm::DataLayout const&) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x23c233c)
#11 0x000002aa1ac69ab0 (anonymous namespace)::SCCPSolver::visitCastInst(llvm::CastInst&) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x3169ab0)
#12 0x000002aa1ac6ce72 llvm::InstVisitor<(anonymous namespace)::SCCPSolver, void>::visit(llvm::Instruction&) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x316ce72)
[...]

that appears to be directly related to this visitCastInst change.

[fhahn]

In D71938#1952589, @uweigand wrote:

I'm now getting an assertion failure in the LNT test suite:
http://lab.llvm.org:8011/builders/clang-s390x-linux-lnt/builds/17826/steps/test-suite/logs/stdio

clang: /home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/llvm/llvm/lib/IR/Constants.cpp:1661: static llvm::Constant* llvm::ConstantExpr::getCast(unsigned int, llvm::Constant*, llvm::Type*, bool): Assertion `CastInst::castIsValid(opc, C, Ty) && "Invalid constantexpr cast!"' failed.
PLEASE submit a bug report to https://bugs.llvm.org/ and include the crash backtrace, preprocessed source, and associated run script.
Stack dump:
0. Program arguments: /home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang -DNDEBUG -O3 -DNDEBUG -w -Werror=date-time -w -o CMakeFiles/GCC-C-execute-scal-to-vec1.dir/scal-to-vec1.c.o -c /home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/test/test-suite/SingleSource/Regression/C/gcc-c-torture/execute/scal-to-vec1.c

1. <eof> parser at end of file
2. Per-module optimization passes
3. Running pass 'CallGraph Pass Manager' on module '/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/test/test-suite/SingleSource/Regression/C/gcc-c-torture/execute/scal-to-vec1.c'.
4. Running pass 'Sparse Conditional Constant Propagation' on function '@main'

[...]
#9 0x000002aa1a5b6b70 llvm::ConstantExpr::getCast(unsigned int, llvm::Constant*, llvm::Type*, bool) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x2ab6b70)
#10 0x000002aa19ec233c llvm::ConstantFoldCastOperand(unsigned int, llvm::Constant*, llvm::Type*, llvm::DataLayout const&) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x23c233c)
#11 0x000002aa1ac69ab0 (anonymous namespace)::SCCPSolver::visitCastInst(llvm::CastInst&) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x3169ab0)
#12 0x000002aa1ac6ce72 llvm::InstVisitor<(anonymous namespace)::SCCPSolver, void>::visit(llvm::Instruction&) (/home/uweigand/sandbox/buildbot/clang-s390x-linux-lnt/stage1.install/bin/clang+0x316ce72)
[...]

that appears to be directly related to this visitCastInst change.

Thanks for letting me know! I'll push a fix shortly.

llvm/lib/Transforms/Scalar/SCCP.cpp

void SCCPSolver::visitTerminator(Instruction &TI) {
   for (unsigned i = 0, e = SuccFeasible.size(); i != e; ++i)
     if (SuccFeasible[i])
       markEdgeExecutable(BB, TI.getSuccessor(i));
 }
 
 void SCCPSolver::visitCastInst(CastInst &I) {
   // ResolvedUndefsIn might mark I as overdefined. Bail out, even if we would
   // discover a concrete value later.
-  if (isOverdefined(ValueState[&I]))
-    return (void)markOverdefined(&I);
+  if (ValueState[&I].isOverdefined())
+    return;
 
   ValueLatticeElement OpSt = getValueState(I.getOperand(0));
   if (Constant *OpC = getConstant(OpSt)) {
     // Fold the constant as we build.
     Constant *C = ConstantFoldCastOperand(I.getOpcode(), OpC, I.getType(), DL);
     if (isa<UndefValue>(C))
       return;
     // Propagate constant value
     markConstant(&I, C);
+  } else if (OpSt.isConstantRange() && I.getDestTy()->isIntegerTy()) {
+    auto &LV = getValueState(&I);
+    ConstantRange OpRange = OpSt.getConstantRange();
+    Type *DestTy = I.getDestTy();
+    ConstantRange Res =
+        OpRange.castOp(I.getOpcode(), DL.getTypeSizeInBits(DestTy));
+    mergeInValue(LV, &I, ValueLatticeElement::getRange(Res));
   } else if (!OpSt.isUnknownOrUndef())
     markOverdefined(&I);
 }
 
 void SCCPSolver::visitExtractValueInst(ExtractValueInst &EVI) {
   // ResolvedUndefsIn might mark I as overdefined. Bail out, even if we would
   // discover a concrete value later.
   if (isOverdefined(ValueState[&EVI]))

llvm/test/Transforms/SCCP/ip-ranges-casts.ll

+; RUN: opt < %s -ipsccp -S | FileCheck %s
+
+; x = [100, 301)
+define internal i1 @f.trunc(i32 %x) {
+; CHECK-LABEL: define internal i1 @f.trunc(i32 %x) {
+; CHECK-NEXT:    %t.1 = trunc i32 %x to i16
+; CHECK-NEXT:    %c.2 = icmp sgt i16 %t.1, 299
+; CHECK-NEXT:    %c.4 = icmp slt i16 %t.1, 101
+; CHECK-NEXT:    %res.1 = add i1 false, %c.2
+; CHECK-NEXT:    %res.2 = add i1 %res.1, false
+; CHECK-NEXT:    %res.3 = add i1 %res.2, %c.4
+; CHECK-NEXT:    %t.2 = trunc i32 %x to i8
+; CHECK-NEXT:    %c.5 = icmp sgt i8 %t.2, 44
+; CHECK-NEXT:    %c.6 = icmp sgt i8 %t.2, 43
+; CHECK-NEXT:    %c.7 = icmp slt i8 %t.2, 100
+; CHECK-NEXT:    %c.8 = icmp slt i8 %t.2, 101
+; CHECK-NEXT:    %res.4 = add i1 %res.3, %c.5
+; CHECK-NEXT:    %res.5 = add i1 %res.4, %c.6
+; CHECK-NEXT:    %res.6 = add i1 %res.5, %c.7
+; CHECK-NEXT:    %res.7 = add i1 %res.6, %c.8
+; CHECK-NEXT:    ret i1 %res.7
+
+  %t.1 = trunc i32 %x to i16
+  %c.1 = icmp sgt i16 %t.1, 300
+  %c.2 = icmp sgt i16 %t.1, 299
+  %c.3 = icmp slt i16 %t.1, 100
+  %c.4 = icmp slt i16 %t.1, 101
+  %res.1 = add i1 %c.1, %c.2
+  %res.2 = add i1 %res.1, %c.3
+  %res.3 = add i1 %res.2, %c.4
+  %t.2 = trunc i32 %x to i8
+  %c.5 = icmp sgt i8 %t.2, 300
+  %c.6 = icmp sgt i8 %t.2, 299
+  %c.7 = icmp slt i8 %t.2, 100
+  %c.8 = icmp slt i8 %t.2, 101
+  %res.4 = add i1 %res.3, %c.5
+  %res.5 = add i1 %res.4, %c.6
+  %res.6 = add i1 %res.5, %c.7
+  %res.7 = add i1 %res.6, %c.8
+  ret i1 %res.7
+}
+
+define i1 @caller1() {
+; CHECK-LABEL:  define i1 @caller1() {
+; CHECK-NEXT:    %call.1 = tail call i1 @f.trunc(i32 100)
+; CHECK-NEXT:    %call.2 = tail call i1 @f.trunc(i32 300)
+; CHECK-NEXT:    %res = and i1 %call.1, %call.2
+; CHECK-NEXT:    ret i1 %res
+;
+  %call.1 = tail call i1 @f.trunc(i32 100)
+  %call.2 = tail call i1 @f.trunc(i32 300)
+  %res = and i1 %call.1, %call.2
+  ret i1 %res
+}
+
+
+; x = [100, 301)
+define internal i1 @f.zext(i32 %x, i32 %y) {
+; CHECK-LABEL: define internal i1 @f.zext(i32 %x, i32 %y) {
+; CHECK-NEXT:    %t.1 = zext i32 %x to i64
+; CHECK-NEXT:    %c.2 = icmp sgt i64 %t.1, 299
+; CHECK-NEXT:    %c.4 = icmp slt i64 %t.1, 101
+; CHECK-NEXT:    %res.1 = add i1 false, %c.2
+; CHECK-NEXT:    %res.2 = add i1 %res.1, false
+; CHECK-NEXT:    %res.3 = add i1 %res.2, %c.4
+; CHECK-NEXT:    %t.2 = zext i32 %y to i64
+; CHECK-NEXT:    %c.5 = icmp sgt i64 %t.2, 300
+; CHECK-NEXT:    %c.6 = icmp sgt i64 %t.2, 299
+; CHECK-NEXT:    %c.8 = icmp slt i64 %t.2, 1
+; CHECK-NEXT:    %res.4 = add i1 %res.3, %c.5
+; CHECK-NEXT:    %res.5 = add i1 %res.4, %c.6
+; CHECK-NEXT:    %res.6 = add i1 %res.5, false
+; CHECK-NEXT:    %res.7 = add i1 %res.6, %c.8
+; CHECK-NEXT:    ret i1 %res.7
+
+  %t.1 = zext i32 %x to i64
+  %c.1 = icmp sgt i64 %t.1, 300
+  %c.2 = icmp sgt i64 %t.1, 299
+  %c.3 = icmp slt i64 %t.1, 100
+  %c.4 = icmp slt i64 %t.1, 101
+  %res.1 = add i1 %c.1, %c.2
+  %res.2 = add i1 %res.1, %c.3
+  %res.3 = add i1 %res.2, %c.4
+  %t.2 = zext i32 %y to i64
+  %c.5 = icmp sgt i64 %t.2, 300
+  %c.6 = icmp sgt i64 %t.2, 299
+  %c.7 = icmp slt i64 %t.2, 0
+  %c.8 = icmp slt i64 %t.2, 1
+  %res.4 = add i1 %res.3, %c.5
+  %res.5 = add i1 %res.4, %c.6
+  %res.6 = add i1 %res.5, %c.7
+  %res.7 = add i1 %res.6, %c.8
+  ret i1 %res.7
+}
+
+define i1 @caller.zext() {
+; CHECK-LABEL:  define i1 @caller.zext() {
+; CHECK-NEXT:    %call.1 = tail call i1 @f.zext(i32 100, i32 -120)
+; CHECK-NEXT:    %call.2 = tail call i1 @f.zext(i32 300, i32 900)
+; CHECK-NEXT:    %res = and i1 %call.1, %call.2
+; CHECK-NEXT:    ret i1 %res
+;
+  %call.1 = tail call i1 @f.zext(i32 100, i32 -120)
+  %call.2 = tail call i1 @f.zext(i32 300, i32 900)
+  %res = and i1 %call.1, %call.2
+  ret i1 %res
+}
+
+; x = [100, 301)
+define internal i1 @f.sext(i32 %x, i32 %y) {
+; CHECK-LABEL: define internal i1 @f.sext(i32 %x, i32 %y) {
+; CHECK-NEXT:    %t.1 = sext i32 %x to i64
+; CHECK-NEXT:    %c.2 = icmp sgt i64 %t.1, 299
+; CHECK-NEXT:    %c.4 = icmp slt i64 %t.1, 101
+; CHECK-NEXT:    %res.1 = add i1 false, %c.2
+; CHECK-NEXT:    %res.2 = add i1 %res.1, false
+; CHECK-NEXT:    %res.3 = add i1 %res.2, %c.4
+; CHECK-NEXT:    %t.2 = sext i32 %y to i64
+; CHECK-NEXT:    %c.6 = icmp sgt i64 %t.2, 899
+; CHECK-NEXT:    %c.8 = icmp slt i64 %t.2, -119
+; CHECK-NEXT:    %res.4 = add i1 %res.3, false
+; CHECK-NEXT:    %res.5 = add i1 %res.4, %c.6
+; CHECK-NEXT:    %res.6 = add i1 %res.5, false
+; CHECK-NEXT:    %res.7 = add i1 %res.6, %c.8
+; CHECK-NEXT:    ret i1 %res.7
+;
+  %t.1 = sext i32 %x to i64
+  %c.1 = icmp sgt i64 %t.1, 300
+  %c.2 = icmp sgt i64 %t.1, 299
+  %c.3 = icmp slt i64 %t.1, 100
+  %c.4 = icmp slt i64 %t.1, 101
+  %res.1 = add i1 %c.1, %c.2
+  %res.2 = add i1 %res.1, %c.3
+  %res.3 = add i1 %res.2, %c.4
+  %t.2 = sext i32 %y to i64
+  %c.5 = icmp sgt i64 %t.2, 900
+  %c.6 = icmp sgt i64 %t.2, 899
+  %c.7 = icmp slt i64 %t.2, -120
+  %c.8 = icmp slt i64 %t.2, -119
+  %res.4 = add i1 %res.3, %c.5
+  %res.5 = add i1 %res.4, %c.6
+  %res.6 = add i1 %res.5, %c.7
+  %res.7 = add i1 %res.6, %c.8
+  ret i1 %res.7
+}
+
+define i1 @caller.sext() {
+; CHECK-LABEL:  define i1 @caller.sext() {
+; CHECK-NEXT:    %call.1 = tail call i1 @f.sext(i32 100, i32 -120)
+; CHECK-NEXT:    %call.2 = tail call i1 @f.sext(i32 300, i32 900)
+; CHECK-NEXT:    %res = and i1 %call.1, %call.2
+; CHECK-NEXT:    ret i1 %res
+;
+  %call.1 = tail call i1 @f.sext(i32 100, i32 -120)
+  %call.2 = tail call i1 @f.sext(i32 300, i32 900)
+  %res = and i1 %call.1, %call.2
+  ret i1 %res
+}
+
+; There's nothing we can do besides going to the full range or overdefined.
+define internal i1 @f.fptosi(i32 %x) {
+; CHECK-LABEL: define internal i1 @f.fptosi(i32 %x) {
+; CHECK-NEXT:    %to.double = sitofp i32 %x to double
+; CHECK-NEXT:    %add = fadd double 0.000000e+00, %to.double
+; CHECK-NEXT:    %to.i32 = fptosi double %add to i32
+; CHECK-NEXT:    %c.1 = icmp sgt i32 %to.i32, 300
+; CHECK-NEXT:    %c.2 = icmp sgt i32 %to.i32, 299
+; CHECK-NEXT:    %c.3 = icmp slt i32 %to.i32, 100
+; CHECK-NEXT:    %c.4 = icmp slt i32 %to.i32, 101
+; CHECK-NEXT:    %res.1 = add i1 %c.1, %c.2
+; CHECK-NEXT:    %res.2 = add i1 %res.1, %c.3
+; CHECK-NEXT:    %res.3 = add i1 %res.2, %c.4
+; CHECK-NEXT:    ret i1 %res.3
+;
+  %to.double = sitofp i32 %x to double
+  %add = fadd double 0.000000e+00, %to.double
+  %to.i32 = fptosi double %add to i32
+  %c.1 = icmp sgt i32 %to.i32, 300
+  %c.2 = icmp sgt i32 %to.i32, 299
+  %c.3 = icmp slt i32 %to.i32, 100
+  %c.4 = icmp slt i32 %to.i32, 101
+  %res.1 = add i1 %c.1, %c.2
+  %res.2 = add i1 %res.1, %c.3
+  %res.3 = add i1 %res.2, %c.4
+  ret i1 %res.3
+}
+
+define i1 @caller.fptosi() {
+; CHECK-LABEL:  define i1 @caller.fptosi() {
+; CHECK-NEXT:    %call.1 = tail call i1 @f.fptosi(i32 100)
+; CHECK-NEXT:    %call.2 = tail call i1 @f.fptosi(i32 300)
+; CHECK-NEXT:    %res = and i1 %call.1, %call.2
+; CHECK-NEXT:    ret i1 %res
+;
+  %call.1 = tail call i1 @f.fptosi(i32 100)
+  %call.2 = tail call i1 @f.fptosi(i32 300)
+  %res = and i1 %call.1, %call.2
+  ret i1 %res
+}
+
+; There's nothing we can do besides going to the full range or overdefined.
+define internal i1 @f.fpext(i16 %x) {
+; CHECK-LABEL: define internal i1 @f.fpext(i16 %x) {
+; CHECK-NEXT:    %to.float = sitofp i16 %x to float
+; CHECK-NEXT:    %to.double = fpext float %to.float to double
+; CHECK-NEXT:    %to.i64 = fptoui float %to.float to i64
+; CHECK-NEXT:    %c.1 = icmp sgt i64 %to.i64, 300
+; CHECK-NEXT:    %c.2 = icmp sgt i64 %to.i64, 299
+; CHECK-NEXT:    %c.3 = icmp slt i64 %to.i64, 100
+; CHECK-NEXT:    %c.4 = icmp slt i64 %to.i64, 101
+; CHECK-NEXT:    %res.1 = add i1 %c.1, %c.2
+; CHECK-NEXT:    %res.2 = add i1 %res.1, %c.3
+; CHECK-NEXT:    %res.3 = add i1 %res.2, %c.4
+; CHECK-NEXT:    ret i1 %res.3
+;
+  %to.float = sitofp i16 %x to float
+  %to.double = fpext float %to.float  to double
+  %to.i64= fptoui float %to.float to i64
+  %c.1 = icmp sgt i64 %to.i64, 300
+  %c.2 = icmp sgt i64 %to.i64, 299
+  %c.3 = icmp slt i64 %to.i64, 100
+  %c.4 = icmp slt i64 %to.i64, 101
+  %res.1 = add i1 %c.1, %c.2
+  %res.2 = add i1 %res.1, %c.3
+  %res.3 = add i1 %res.2, %c.4
+  ret i1 %res.3
+}
+
+; There's nothing we can do besides going to the full range or overdefined.
+define i1 @caller.fpext() {
+; CHECK-LABEL:  define i1 @caller.fpext() {
+; CHECK-NEXT:    %call.1 = tail call i1 @f.fpext(i16 100)
+; CHECK-NEXT:    %call.2 = tail call i1 @f.fpext(i16 300)
+; CHECK-NEXT:    %res = and i1 %call.1, %call.2
+; CHECK-NEXT:    ret i1 %res
+;
+  %call.1 = tail call i1 @f.fpext(i16 100)
+  %call.2 = tail call i1 @f.fpext(i16 300)
+  %res = and i1 %call.1, %call.2
+  ret i1 %res
+}
+
+; There's nothing we can do besides going to the full range or overdefined.
+define internal i1 @f.inttoptr.ptrtoint(i64 %x) {
+; CHECK-LABEL: define internal i1 @f.inttoptr.ptrtoint(i64 %x) {
+; CHECK-NEXT:    %to.ptr = inttoptr i64 %x to i8*
+; CHECK-NEXT:    %to.i64 = ptrtoint i8* %to.ptr to i64
+; CHECK-NEXT:    %c.1 = icmp sgt i64 %to.i64, 300
+; CHECK-NEXT:    %c.2 = icmp sgt i64 %to.i64, 299
+; CHECK-NEXT:    %c.3 = icmp slt i64 %to.i64, 100
+; CHECK-NEXT:    %c.4 = icmp slt i64 %to.i64, 101
+; CHECK-NEXT:    %res.1 = add i1 %c.1, %c.2
+; CHECK-NEXT:    %res.2 = add i1 %res.1, %c.3
+; CHECK-NEXT:    %res.3 = add i1 %res.2, %c.4
+; CHECK-NEXT:    ret i1 %res.3
+;
+  %to.ptr = inttoptr i64 %x to i8*
+  %to.i64 = ptrtoint i8* %to.ptr to i64
+  %c.1 = icmp sgt i64 %to.i64, 300
+  %c.2 = icmp sgt i64 %to.i64, 299
+  %c.3 = icmp slt i64 %to.i64, 100
+  %c.4 = icmp slt i64 %to.i64, 101
+  %res.1 = add i1 %c.1, %c.2
+  %res.2 = add i1 %res.1, %c.3
+  %res.3 = add i1 %res.2, %c.4
+  ret i1 %res.3
+}
+
+define i1 @caller.inttoptr.ptrtoint() {
+; CHECK-LABEL:  define i1 @caller.inttoptr.ptrtoint() {
+; CHECK-NEXT:    %call.1 = tail call i1 @f.inttoptr.ptrtoint(i64 100)
+; CHECK-NEXT:    %call.2 = tail call i1 @f.inttoptr.ptrtoint(i64 300)
+; CHECK-NEXT:    %res = and i1 %call.1, %call.2
+; CHECK-NEXT:    ret i1 %res
+;
+  %call.1 = tail call i1 @f.inttoptr.ptrtoint(i64 100)
+  %call.2 = tail call i1 @f.inttoptr.ptrtoint(i64 300)
+  %res = and i1 %call.1, %call.2
+  ret i1 %res
+}
+
+; Make sure we do not create constant ranges for int to fp casts.
+define i1 @int_range_to_double_cast(i32 %a) {
+; CHECK-LABEL: define i1 @int_range_to_double_cast(i32 %a)
+; CHECK-NEXT:    %r = and i32 %a, 255
+; CHECK-NEXT:    %tmp4 = sitofp i32 %r to double
+; CHECK-NEXT:    %tmp10 = fadd double 0.000000e+00, %tmp4
+; CHECK-NEXT:    %tmp11 = fcmp olt double %tmp4, %tmp10
+; CHECK-NEXT:    ret i1 %tmp11
+;
+  %r = and i32 %a, 255
+  %tmp4 = sitofp i32 %r to double
+  %tmp10 = fadd double 0.000000e+00, %tmp4
+  %tmp11 = fcmp olt double %tmp4, %tmp10
+  ret i1 %tmp11
+}