add another case of undefined behavior without crashing, PR3466.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@63620 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Transforms/Scalar/ScalarReplAggregates.cpp b/lib/Transforms/Scalar/ScalarReplAggregates.cpp
index 83572d65dae3507f7d6989add2fc21b69fda0cfa..c67c67a4b6200903077811605ea848c15333063e 100644 (file)
--- a/lib/Transforms/Scalar/ScalarReplAggregates.cpp
+++ b/lib/Transforms/Scalar/ScalarReplAggregates.cpp
@@ -1431,8 +1431,9 @@ Value *SROA::ConvertUsesOfLoadToScalar(LoadInst *LI, AllocaInst *NewAI,
   // Finally, unconditionally truncate the integer to the right width.
   unsigned LIBitWidth = TD->getTypeSizeInBits(LI->getType());
   if (LIBitWidth < NTy->getBitWidth())
-    NV = new TruncInst(NV, IntegerType::get(LIBitWidth),
-                       LI->getName(), LI);
+    NV = new TruncInst(NV, IntegerType::get(LIBitWidth), LI->getName(), LI);
+  else if (LIBitWidth > NTy->getBitWidth())
+    NV = new ZExtInst(NV, IntegerType::get(LIBitWidth), LI->getName(), LI);
 
   // If the result is an integer, this is a trunc or bitcast.
   if (isa<IntegerType>(LI->getType())) {

diff --git a/test/Transforms/ScalarRepl/badarray.ll b/test/Transforms/ScalarRepl/badarray.ll
index acac5a61822fc66d1a608d02beec4f43a0238792..c48848630564bb2e02c7152ee079bceba099b40d 100644 (file)
--- a/test/Transforms/ScalarRepl/badarray.ll
+++ b/test/Transforms/ScalarRepl/badarray.ll
@@ -1,4 +1,5 @@
 ; RUN: llvm-as < %s | opt -scalarrepl -instcombine | llvm-dis | not grep alloca
+; PR3466
 
 define i32 @test() {
        %X = alloca [4 x i32]           ; <[4 x i32]*> [#uses=1]
@@ -9,3 +10,11 @@ define i32 @test() {
        ret i32 %Z
 }
 
+
+define i32 @test2() nounwind {
+entry:
+        %yx2.i = alloca float, align 4          ; <float*> [#uses=1]            
+        %yx26.i = bitcast float* %yx2.i to i64*         ; <i64*> [#uses=1]      
+        %0 = load i64* %yx26.i, align 8         ; <i64> [#uses=0]               
+        unreachable
+}