[llvm-commits] [llvm] r63620 - in /llvm/trunk: lib/Transforms/Scalar/ScalarReplAggregates.cpp test/Transforms/ScalarRepl/badarray.ll

[Chris Lattner]

Author: lattner
Date: Tue Feb  3 01:08:57 2009
New Revision: 63620

URL: http://llvm.org/viewvc/llvm-project?rev=63620&view=rev
Log:
add another case of undefined behavior without crashing, PR3466.

Modified:
    llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp
    llvm/trunk/test/Transforms/ScalarRepl/badarray.ll

Modified: llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp?rev=63620&r1=63619&r2=63620&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/ScalarReplAggregates.cpp Tue Feb  3 01:08:57 2009
@@ -1431,8 +1431,9 @@
   // 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())) {

Modified: llvm/trunk/test/Transforms/ScalarRepl/badarray.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/ScalarRepl/badarray.ll?rev=63620&r1=63619&r2=63620&view=diff

==============================================================================
--- llvm/trunk/test/Transforms/ScalarRepl/badarray.ll (original)
+++ llvm/trunk/test/Transforms/ScalarRepl/badarray.ll Tue Feb  3 01:08:57 2009
@@ -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 @@
 	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
+}