[llvm-commits] [llvm] r63620 - in /llvm/trunk: lib/Transforms/Scalar/ScalarReplAggregates.cpp test/Transforms/ScalarRepl/badarray.ll
Chris Lattner
sabre at nondot.org
Mon Feb 2 23:08:57 PST 2009
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
+}
More information about the llvm-commits
mailing list