From 07b294a25bfeb658ea3e2420c3e23bbb39f1f390 Mon Sep 17 00:00:00 2001 From: Sanjay Patel Date: Wed, 16 Jul 2014 22:40:28 +0000 Subject: [PATCH] Fixed formatting, removed bug reference, renamed testcase Thanks to Duncan Exon Smith for reviewing and cleanup suggestions. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@213205 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/CodeGenPrepare.cpp | 7 ++++--- .../{pr20314.ll => address-type-promotion-constantexpr.ll} | 4 +++- 2 files changed, 7 insertions(+), 4 deletions(-) rename test/CodeGen/X86/{pr20314.ll => address-type-promotion-constantexpr.ll} (74%) diff --git a/lib/CodeGen/CodeGenPrepare.cpp b/lib/CodeGen/CodeGenPrepare.cpp index f2c7e642710..d5039b2e851 100644 --- a/lib/CodeGen/CodeGenPrepare.cpp +++ b/lib/CodeGen/CodeGenPrepare.cpp @@ -2036,7 +2036,8 @@ bool AddressingModeMatcher::MatchOperationAddr(User *AddrInst, unsigned Opcode, case Instruction::Shl: { // Can only handle X*C and X << C. ConstantInt *RHS = dyn_cast(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::MatchOperationAddr(User *AddrInst, unsigned Opcode, return true; } case Instruction::SExt: { - // Make sure this isn't a ConstantExpr (PR20314). Instruction *SExt = dyn_cast(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. diff --git a/test/CodeGen/X86/pr20314.ll b/test/CodeGen/X86/address-type-promotion-constantexpr.ll similarity index 74% rename from test/CodeGen/X86/pr20314.ll rename to test/CodeGen/X86/address-type-promotion-constantexpr.ll index 4448afba4af..32f29bd3cad 100644 --- a/test/CodeGen/X86/pr20314.ll +++ b/test/CodeGen/X86/address-type-promotion-constantexpr.ll @@ -1,11 +1,13 @@ ; RUN: llc < %s -mtriple=x86_64-pc-linux -; No check as PR20314 is a crashing bug. +; PR20314 is a crashing bug. This program does nothing with the load, so just check that the return is 0. @c = common global [2 x i32] zeroinitializer, align 4 @a = common global i32 0, align 4 @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 -- 2.34.1