[llvm] r213205 - Fixed formatting, removed bug reference, renamed testcase

[Sanjay Patel]

Author: spatel
Date: Wed Jul 16 17:40:28 2014
New Revision: 213205

URL: http://llvm.org/viewvc/llvm-project?rev=213205&view=rev
Log:
Fixed formatting, removed bug reference, renamed testcase

Thanks to Duncan Exon Smith for reviewing and cleanup suggestions. 

Added:
    llvm/trunk/test/CodeGen/X86/address-type-promotion-constantexpr.ll
Removed:
    llvm/trunk/test/CodeGen/X86/pr20314.ll
Modified:
    llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp

Modified: llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp?rev=213205&r1=213204&r2=213205&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp (original)
+++ llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp Wed Jul 16 17:40:28 2014
@@ -2036,7 +2036,8 @@ bool AddressingModeMatcher::MatchOperati
   case Instruction::Shl: {
     // Can only handle X*C and X << C.
     ConstantInt *RHS = dyn_cast<ConstantInt>(AddrInst->getOperand(1));
-    if (!RHS) return false;
+    if (!RHS)
+      return false;
     int64_t Scale = RHS->getSExtValue();
     if (Opcode == Instruction::Shl)
       Scale = 1LL << Scale;
@@ -2130,9 +2131,9 @@ bool AddressingModeMatcher::MatchOperati
     return true;
   }
   case Instruction::SExt: {
-    // Make sure this isn't a ConstantExpr (PR20314).
     Instruction *SExt = dyn_cast<Instruction>(AddrInst);
-    if (!SExt) return false;
+    if (!SExt)
+      return false;
 
     // Try to move this sext out of the way of the addressing mode.
     // Ask for a method for doing so.

Added: llvm/trunk/test/CodeGen/X86/address-type-promotion-constantexpr.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/address-type-promotion-constantexpr.ll?rev=213205&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/X86/address-type-promotion-constantexpr.ll (added)
+++ llvm/trunk/test/CodeGen/X86/address-type-promotion-constantexpr.ll Wed Jul 16 17:40:28 2014
@@ -0,0 +1,16 @@
+; RUN: llc < %s -mtriple=x86_64-pc-linux
+
+; PR20314 is a crashing bug. This program does nothing with the load, so just check that the return is 0.
+
+ at c = common global [2 x i32] zeroinitializer, align 4
+ at a = common global i32 0, align 4
+ at b = internal unnamed_addr constant [2 x i8] c"\01\00", align 1
+
+; CHECK-LABEL: main
+; CHECK: xor %eax, %eax
+define i32 @main() {
+entry:
+  %foo = load i8* getelementptr ([2 x i8]* @b, i64 0, i64 sext (i8 or (i8 zext (i1 icmp eq (i32* getelementptr inbounds ([2 x i32]* @c, i64 0, i64 1), i32* @a) to i8), i8 1) to i64)), align 1
+  ret i32 0
+}
+

Removed: llvm/trunk/test/CodeGen/X86/pr20314.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/pr20314.ll?rev=213204&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/X86/pr20314.ll (original)
+++ llvm/trunk/test/CodeGen/X86/pr20314.ll (removed)
@@ -1,14 +0,0 @@
-; RUN: llc < %s -mtriple=x86_64-pc-linux
-
-; No check as PR20314 is a crashing bug.
-
- at c = common global [2 x i32] zeroinitializer, align 4
- at a = common global i32 0, align 4
- at b = internal unnamed_addr constant [2 x i8] c"\01\00", align 1
-
-define i32 @main() {
-entry:
-  %foo = load i8* getelementptr ([2 x i8]* @b, i64 0, i64 sext (i8 or (i8 zext (i1 icmp eq (i32* getelementptr inbounds ([2 x i32]* @c, i64 0, i64 1), i32* @a) to i8), i8 1) to i64)), align 1
-  ret i32 0
-}
-