Index: llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp =================================================================== --- llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp +++ llvm/trunk/lib/Transforms/Scalar/LoopStrengthReduce.cpp @@ -1646,7 +1646,12 @@ Instruction *IVIncInsertPos; /// Interesting factors between use strides. - SmallSetVector Factors; + /// + /// We explicitly use a SetVector which contains a SmallSet, instead of the + /// default, a SmallDenseSet, because we need to use the full range of + /// int64_ts, and there's currently no good way of doing that with + /// SmallDenseSet. + SetVector, SmallSet> Factors; /// Interesting use types, to facilitate truncation reuse. SmallSetVector Types; Index: llvm/trunk/test/CodeGen/X86/loop-strength-reduce-crash.ll =================================================================== --- llvm/trunk/test/CodeGen/X86/loop-strength-reduce-crash.ll +++ llvm/trunk/test/CodeGen/X86/loop-strength-reduce-crash.ll @@ -0,0 +1,24 @@ +; RUN: llc < %s -o /dev/null + +; Check that this doesn't crash (by virtue of using INT64_MAX as a constant in +; the loop). + +target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128" +target triple = "x86_64-apple-macosx10.12.0" + +define void @foo() { +entry: + br label %for + +for: + %0 = phi i64 [ %add, %for ], [ undef, %entry ] + %next = phi i32 [ %inc, %for ], [ undef, %entry ] + store i32 %next, i32* undef, align 4 + %add = add i64 %0, 9223372036854775807 + %inc = add nsw i32 %next, 1 + br i1 undef, label %exit, label %for + +exit: + store i64 %add, i64* undef + ret void +}