maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

When comparing constants, consider a less wide constant to be "less complex" 

than a wider one, before trying to compare their contents which will crash
if their sizes are different.

llvm-svn: 74792
This commit is contained in:
Nick Lewycky 2009-07-04 17:24:52 +00:00
parent ca6d224d71
commit 3292908132
2 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
llvm/lib/Analysis/ScalarEvolution.cpp
View File

@ -495,6 +495,8 @@ namespace {
// Compare constant values.
if (const SCEVConstant *LC = dyn_cast<SCEVConstant>(LHS)) {
const SCEVConstant *RC = cast<SCEVConstant>(RHS);
if (LC->getValue()->getBitWidth() != RC->getValue()->getBitWidth())
return LC->getValue()->getBitWidth() < RC->getValue()->getBitWidth();
return LC->getValue()->getValue().ult(RC->getValue()->getValue());
}

16
llvm/test/Analysis/ScalarEvolution/2009-07-04-GroupConstantsWidthMismatch.ll Normal file
View File

@ -0,0 +1,16 @@
; RUN: llvm-as < %s | opt -analyze -scalar-evolution
; PR4501
define void @test() {
entry:
%0 = load i16* undef, align 1
%1 = lshr i16 %0, 8
%2 = and i16 %1, 3
%3 = zext i16 %2 to i32
%4 = load i8* undef, align 1
%5 = lshr i8 %4, 4
%6 = and i8 %5, 1
%7 = zext i8 %6 to i32
%t1 = add i32 %3, %7
ret void
}