From: Andrew Lenharth <andrewl@lenharth.org>
Date: Thu, 8 Nov 2007 18:45:15 +0000 (+0000)
Subject: Better check
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=b410df995c592491746ed76039edb62f5cdbf8cf;p=oota-llvm.git

Better check

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

diff --git a/lib/Transforms/Scalar/InstructionCombining.cpp b/lib/Transforms/Scalar/InstructionCombining.cpp
index c8d03be2c00..efcd8e32972 100644
--- a/lib/Transforms/Scalar/InstructionCombining.cpp
+++ b/lib/Transforms/Scalar/InstructionCombining.cpp
@@ -7582,7 +7582,7 @@ static unsigned GetOrEnforceKnownAlignment(Value *V, TargetData *TD,
                                            unsigned PrefAlign = 0) {
   if (GlobalVariable *GV = dyn_cast<GlobalVariable>(V)) {
     unsigned Align = GV->getAlignment();
-    if (Align == 0 && TD && !isa<OpaqueType>(GV->getType()->getElementType())) 
+    if (Align == 0 && TD && GV->getType()->getElementType()->isSized()) 
       Align = TD->getPrefTypeAlignment(GV->getType()->getElementType());
 
     // If there is a large requested alignment and we can, bump up the alignment
diff --git a/test/Transforms/InstCombine/2007-11-07-OpaqueAlignCrash.ll b/test/Transforms/InstCombine/2007-11-07-OpaqueAlignCrash.ll
index 61e67ad10c9..f3caf079763 100644
--- a/test/Transforms/InstCombine/2007-11-07-OpaqueAlignCrash.ll
+++ b/test/Transforms/InstCombine/2007-11-07-OpaqueAlignCrash.ll
@@ -6,12 +6,15 @@ target triple = "i686-pc-linux-gnu"
 
 %opaque_t = type opaque
 
+%op_ts = type {opaque, i32}
+
 @g = external global %opaque_t
+@h = external global %op_ts
 
 define i32 @foo() {
 entry:
         %x = load i8* bitcast (%opaque_t* @g to i8*)
-        %y = load i32* bitcast (%opaque_t* @g to i32*)
+        %y = load i32* bitcast (%op_ts* @h to i32*)
 	%z = zext i8 %x to i32
 	%r = add i32 %y, %z
         ret i32 %r