2003-05-27 15:45:27 +00:00
|
|
|
//===- ScalarReplAggregates.cpp - Scalar Replacement of Aggregates --------===//
|
2005-04-21 23:48:37 +00:00
|
|
|
//
|
2003-10-20 19:43:21 +00:00
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
2007-12-29 20:36:04 +00:00
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
2005-04-21 23:48:37 +00:00
|
|
|
//
|
2003-10-20 19:43:21 +00:00
|
|
|
//===----------------------------------------------------------------------===//
|
2003-05-27 15:45:27 +00:00
|
|
|
//
|
|
|
|
// This transformation implements the well known scalar replacement of
|
|
|
|
// aggregates transformation. This xform breaks up alloca instructions of
|
|
|
|
// aggregate type (structure or array) into individual alloca instructions for
|
2003-09-11 16:45:55 +00:00
|
|
|
// each member (if possible). Then, if possible, it transforms the individual
|
|
|
|
// alloca instructions into nice clean scalar SSA form.
|
|
|
|
//
|
|
|
|
// This combines a simple SRoA algorithm with the Mem2Reg algorithm because
|
|
|
|
// often interact, especially for C++ programs. As such, iterating between
|
|
|
|
// SRoA, then Mem2Reg until we run out of things to promote works well.
|
2003-05-27 15:45:27 +00:00
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2006-12-19 21:40:18 +00:00
|
|
|
#define DEBUG_TYPE "scalarrepl"
|
2003-05-27 15:45:27 +00:00
|
|
|
#include "llvm/Transforms/Scalar.h"
|
2003-09-11 16:45:55 +00:00
|
|
|
#include "llvm/Constants.h"
|
|
|
|
#include "llvm/DerivedTypes.h"
|
2003-05-27 15:45:27 +00:00
|
|
|
#include "llvm/Function.h"
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
#include "llvm/GlobalVariable.h"
|
2004-07-29 17:05:13 +00:00
|
|
|
#include "llvm/Instructions.h"
|
2007-03-05 07:52:57 +00:00
|
|
|
#include "llvm/IntrinsicInst.h"
|
2009-07-03 19:42:02 +00:00
|
|
|
#include "llvm/LLVMContext.h"
|
2007-03-05 07:52:57 +00:00
|
|
|
#include "llvm/Pass.h"
|
2003-09-11 16:45:55 +00:00
|
|
|
#include "llvm/Analysis/Dominators.h"
|
|
|
|
#include "llvm/Target/TargetData.h"
|
|
|
|
#include "llvm/Transforms/Utils/PromoteMemToReg.h"
|
2009-02-10 07:00:59 +00:00
|
|
|
#include "llvm/Transforms/Utils/Local.h"
|
2006-06-28 23:17:24 +00:00
|
|
|
#include "llvm/Support/Debug.h"
|
2009-07-11 13:10:19 +00:00
|
|
|
#include "llvm/Support/ErrorHandling.h"
|
2005-12-12 07:19:13 +00:00
|
|
|
#include "llvm/Support/GetElementPtrTypeIterator.h"
|
2009-02-03 19:41:50 +00:00
|
|
|
#include "llvm/Support/IRBuilder.h"
|
2005-12-12 07:19:13 +00:00
|
|
|
#include "llvm/Support/MathExtras.h"
|
2006-08-27 12:54:02 +00:00
|
|
|
#include "llvm/Support/Compiler.h"
|
2007-02-12 22:56:41 +00:00
|
|
|
#include "llvm/ADT/SmallVector.h"
|
2004-09-01 22:55:40 +00:00
|
|
|
#include "llvm/ADT/Statistic.h"
|
|
|
|
#include "llvm/ADT/StringExtras.h"
|
2003-12-02 17:43:55 +00:00
|
|
|
using namespace llvm;
|
2003-11-11 22:41:34 +00:00
|
|
|
|
2006-12-19 21:40:18 +00:00
|
|
|
STATISTIC(NumReplaced, "Number of allocas broken up");
|
|
|
|
STATISTIC(NumPromoted, "Number of allocas promoted");
|
|
|
|
STATISTIC(NumConverted, "Number of aggregates converted to scalar");
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
STATISTIC(NumGlobals, "Number of allocas copied from constant global");
|
2003-05-27 15:45:27 +00:00
|
|
|
|
2006-12-19 21:40:18 +00:00
|
|
|
namespace {
|
2006-06-28 23:17:24 +00:00
|
|
|
struct VISIBILITY_HIDDEN SROA : public FunctionPass {
|
2007-05-06 13:37:16 +00:00
|
|
|
static char ID; // Pass identification, replacement for typeid
|
2008-09-04 17:05:41 +00:00
|
|
|
explicit SROA(signed T = -1) : FunctionPass(&ID) {
|
2007-07-09 21:19:23 +00:00
|
|
|
if (T == -1)
|
2007-08-02 21:33:36 +00:00
|
|
|
SRThreshold = 128;
|
2007-07-09 21:19:23 +00:00
|
|
|
else
|
|
|
|
SRThreshold = T;
|
|
|
|
}
|
2007-05-01 21:15:47 +00:00
|
|
|
|
2003-05-27 15:45:27 +00:00
|
|
|
bool runOnFunction(Function &F);
|
|
|
|
|
2003-09-11 16:45:55 +00:00
|
|
|
bool performScalarRepl(Function &F);
|
|
|
|
bool performPromotion(Function &F);
|
|
|
|
|
2003-08-31 00:45:13 +00:00
|
|
|
// getAnalysisUsage - This pass does not require any passes, but we know it
|
|
|
|
// will not alter the CFG, so say so.
|
|
|
|
virtual void getAnalysisUsage(AnalysisUsage &AU) const {
|
2007-06-07 21:57:03 +00:00
|
|
|
AU.addRequired<DominatorTree>();
|
2003-09-11 16:45:55 +00:00
|
|
|
AU.addRequired<DominanceFrontier>();
|
|
|
|
AU.addRequired<TargetData>();
|
2003-08-31 00:45:13 +00:00
|
|
|
AU.setPreservesCFG();
|
|
|
|
}
|
|
|
|
|
2003-05-27 15:45:27 +00:00
|
|
|
private:
|
2009-01-07 06:34:28 +00:00
|
|
|
TargetData *TD;
|
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
/// AllocaInfo - When analyzing uses of an alloca instruction, this captures
|
|
|
|
/// information about the uses. All these fields are initialized to false
|
|
|
|
/// and set to true when something is learned.
|
|
|
|
struct AllocaInfo {
|
|
|
|
/// isUnsafe - This is set to true if the alloca cannot be SROA'd.
|
|
|
|
bool isUnsafe : 1;
|
|
|
|
|
2009-02-10 07:00:59 +00:00
|
|
|
/// needsCleanup - This is set to true if there is some use of the alloca
|
|
|
|
/// that requires cleanup.
|
|
|
|
bool needsCleanup : 1;
|
2007-05-30 06:11:23 +00:00
|
|
|
|
|
|
|
/// isMemCpySrc - This is true if this aggregate is memcpy'd from.
|
|
|
|
bool isMemCpySrc : 1;
|
|
|
|
|
2007-07-06 06:01:16 +00:00
|
|
|
/// isMemCpyDst - This is true if this aggregate is memcpy'd into.
|
2007-05-30 06:11:23 +00:00
|
|
|
bool isMemCpyDst : 1;
|
|
|
|
|
|
|
|
AllocaInfo()
|
2009-02-10 07:00:59 +00:00
|
|
|
: isUnsafe(false), needsCleanup(false),
|
2007-05-30 06:11:23 +00:00
|
|
|
isMemCpySrc(false), isMemCpyDst(false) {}
|
|
|
|
};
|
|
|
|
|
2007-07-09 21:19:23 +00:00
|
|
|
unsigned SRThreshold;
|
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
void MarkUnsafe(AllocaInfo &I) { I.isUnsafe = true; }
|
|
|
|
|
2004-11-14 04:24:28 +00:00
|
|
|
int isSafeAllocaToScalarRepl(AllocationInst *AI);
|
2007-05-30 06:11:23 +00:00
|
|
|
|
|
|
|
void isSafeUseOfAllocation(Instruction *User, AllocationInst *AI,
|
|
|
|
AllocaInfo &Info);
|
|
|
|
void isSafeElementUse(Value *Ptr, bool isFirstElt, AllocationInst *AI,
|
|
|
|
AllocaInfo &Info);
|
|
|
|
void isSafeMemIntrinsicOnAllocation(MemIntrinsic *MI, AllocationInst *AI,
|
|
|
|
unsigned OpNo, AllocaInfo &Info);
|
|
|
|
void isSafeUseOfBitCastedAllocation(BitCastInst *User, AllocationInst *AI,
|
|
|
|
AllocaInfo &Info);
|
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
void DoScalarReplacement(AllocationInst *AI,
|
|
|
|
std::vector<AllocationInst*> &WorkList);
|
2009-02-10 07:00:59 +00:00
|
|
|
void CleanupGEP(GetElementPtrInst *GEP);
|
|
|
|
void CleanupAllocaUsers(AllocationInst *AI);
|
2003-05-27 15:45:27 +00:00
|
|
|
AllocaInst *AddNewAlloca(Function &F, const Type *Ty, AllocationInst *Base);
|
2005-12-12 07:19:13 +00:00
|
|
|
|
2007-03-19 00:16:43 +00:00
|
|
|
void RewriteBitCastUserOfAlloca(Instruction *BCInst, AllocationInst *AI,
|
2007-03-05 07:52:57 +00:00
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
void RewriteMemIntrinUserOfAlloca(MemIntrinsic *MI, Instruction *BCInst,
|
|
|
|
AllocationInst *AI,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
void RewriteStoreUserOfWholeAlloca(StoreInst *SI, AllocationInst *AI,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
2009-01-08 05:42:05 +00:00
|
|
|
void RewriteLoadUserOfWholeAlloca(LoadInst *LI, AllocationInst *AI,
|
2009-01-28 20:16:43 +00:00
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
2009-01-07 07:18:45 +00:00
|
|
|
|
2009-02-03 01:30:09 +00:00
|
|
|
bool CanConvertToScalar(Value *V, bool &IsNotTrivial, const Type *&VecTy,
|
2009-02-03 18:15:05 +00:00
|
|
|
bool &SawVec, uint64_t Offset, unsigned AllocaSize);
|
2009-01-31 02:28:54 +00:00
|
|
|
void ConvertUsesToScalar(Value *Ptr, AllocaInst *NewAI, uint64_t Offset);
|
2009-02-03 21:01:03 +00:00
|
|
|
Value *ConvertScalar_ExtractValue(Value *NV, const Type *ToType,
|
2009-02-03 19:45:44 +00:00
|
|
|
uint64_t Offset, IRBuilder<> &Builder);
|
2009-02-03 19:30:11 +00:00
|
|
|
Value *ConvertScalar_InsertValue(Value *StoredVal, Value *ExistingVal,
|
2009-02-03 19:41:50 +00:00
|
|
|
uint64_t Offset, IRBuilder<> &Builder);
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
static Instruction *isOnlyCopiedFromConstantGlobal(AllocationInst *AI);
|
2003-05-27 15:45:27 +00:00
|
|
|
};
|
|
|
|
}
|
|
|
|
|
2008-05-13 00:00:25 +00:00
|
|
|
char SROA::ID = 0;
|
|
|
|
static RegisterPass<SROA> X("scalarrepl", "Scalar Replacement of Aggregates");
|
|
|
|
|
2003-11-11 22:41:34 +00:00
|
|
|
// Public interface to the ScalarReplAggregates pass
|
2007-07-09 21:19:23 +00:00
|
|
|
FunctionPass *llvm::createScalarReplAggregatesPass(signed int Threshold) {
|
|
|
|
return new SROA(Threshold);
|
|
|
|
}
|
2003-05-27 15:45:27 +00:00
|
|
|
|
|
|
|
|
|
|
|
bool SROA::runOnFunction(Function &F) {
|
2009-01-07 06:34:28 +00:00
|
|
|
TD = &getAnalysis<TargetData>();
|
|
|
|
|
2003-09-12 15:36:03 +00:00
|
|
|
bool Changed = performPromotion(F);
|
|
|
|
while (1) {
|
|
|
|
bool LocalChange = performScalarRepl(F);
|
|
|
|
if (!LocalChange) break; // No need to repromote if no scalarrepl
|
|
|
|
Changed = true;
|
|
|
|
LocalChange = performPromotion(F);
|
|
|
|
if (!LocalChange) break; // No need to re-scalarrepl if no promotion
|
|
|
|
}
|
2003-09-11 16:45:55 +00:00
|
|
|
|
|
|
|
return Changed;
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
bool SROA::performPromotion(Function &F) {
|
|
|
|
std::vector<AllocaInst*> Allocas;
|
2007-06-07 21:57:03 +00:00
|
|
|
DominatorTree &DT = getAnalysis<DominatorTree>();
|
2003-10-05 21:20:13 +00:00
|
|
|
DominanceFrontier &DF = getAnalysis<DominanceFrontier>();
|
2003-09-11 16:45:55 +00:00
|
|
|
|
2003-09-20 14:39:18 +00:00
|
|
|
BasicBlock &BB = F.getEntryBlock(); // Get the entry node for the function
|
2003-09-11 16:45:55 +00:00
|
|
|
|
2003-09-12 15:36:03 +00:00
|
|
|
bool Changed = false;
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2003-09-11 16:45:55 +00:00
|
|
|
while (1) {
|
|
|
|
Allocas.clear();
|
|
|
|
|
|
|
|
// Find allocas that are safe to promote, by looking at all instructions in
|
|
|
|
// the entry node
|
|
|
|
for (BasicBlock::iterator I = BB.begin(), E = --BB.end(); I != E; ++I)
|
|
|
|
if (AllocaInst *AI = dyn_cast<AllocaInst>(I)) // Is it an alloca?
|
2007-04-25 17:15:20 +00:00
|
|
|
if (isAllocaPromotable(AI))
|
2003-09-11 16:45:55 +00:00
|
|
|
Allocas.push_back(AI);
|
|
|
|
|
|
|
|
if (Allocas.empty()) break;
|
|
|
|
|
2009-07-22 00:24:57 +00:00
|
|
|
PromoteMemToReg(Allocas, DT, DF, F.getContext());
|
2003-09-11 16:45:55 +00:00
|
|
|
NumPromoted += Allocas.size();
|
|
|
|
Changed = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
return Changed;
|
|
|
|
}
|
|
|
|
|
2008-06-22 17:46:21 +00:00
|
|
|
/// getNumSAElements - Return the number of elements in the specific struct or
|
|
|
|
/// array.
|
|
|
|
static uint64_t getNumSAElements(const Type *T) {
|
|
|
|
if (const StructType *ST = dyn_cast<StructType>(T))
|
|
|
|
return ST->getNumElements();
|
|
|
|
return cast<ArrayType>(T)->getNumElements();
|
|
|
|
}
|
|
|
|
|
2003-09-11 16:45:55 +00:00
|
|
|
// performScalarRepl - This algorithm is a simple worklist driven algorithm,
|
|
|
|
// which runs on all of the malloc/alloca instructions in the function, removing
|
|
|
|
// them if they are only used by getelementptr instructions.
|
|
|
|
//
|
|
|
|
bool SROA::performScalarRepl(Function &F) {
|
2003-05-27 15:45:27 +00:00
|
|
|
std::vector<AllocationInst*> WorkList;
|
|
|
|
|
|
|
|
// Scan the entry basic block, adding any alloca's and mallocs to the worklist
|
2003-09-20 14:39:18 +00:00
|
|
|
BasicBlock &BB = F.getEntryBlock();
|
2003-05-27 15:45:27 +00:00
|
|
|
for (BasicBlock::iterator I = BB.begin(), E = BB.end(); I != E; ++I)
|
|
|
|
if (AllocationInst *A = dyn_cast<AllocationInst>(I))
|
|
|
|
WorkList.push_back(A);
|
|
|
|
|
|
|
|
// Process the worklist
|
|
|
|
bool Changed = false;
|
|
|
|
while (!WorkList.empty()) {
|
|
|
|
AllocationInst *AI = WorkList.back();
|
|
|
|
WorkList.pop_back();
|
2005-12-12 07:19:13 +00:00
|
|
|
|
2006-12-22 23:14:42 +00:00
|
|
|
// Handle dead allocas trivially. These can be formed by SROA'ing arrays
|
|
|
|
// with unused elements.
|
|
|
|
if (AI->use_empty()) {
|
|
|
|
AI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
2009-02-03 01:30:09 +00:00
|
|
|
|
|
|
|
// If this alloca is impossible for us to promote, reject it early.
|
|
|
|
if (AI->isArrayAllocation() || !AI->getAllocatedType()->isSized())
|
|
|
|
continue;
|
|
|
|
|
|
|
|
// Check to see if this allocation is only modified by a memcpy/memmove from
|
|
|
|
// a constant global. If this is the case, we can change all users to use
|
|
|
|
// the constant global instead. This is commonly produced by the CFE by
|
|
|
|
// constructs like "void foo() { int A[] = {1,2,3,4,5,6,7,8,9...}; }" if 'A'
|
|
|
|
// is only subsequently read.
|
|
|
|
if (Instruction *TheCopy = isOnlyCopiedFromConstantGlobal(AI)) {
|
|
|
|
DOUT << "Found alloca equal to global: " << *AI;
|
|
|
|
DOUT << " memcpy = " << *TheCopy;
|
|
|
|
Constant *TheSrc = cast<Constant>(TheCopy->getOperand(2));
|
2009-07-29 18:55:55 +00:00
|
|
|
AI->replaceAllUsesWith(ConstantExpr::getBitCast(TheSrc, AI->getType()));
|
2009-02-03 01:30:09 +00:00
|
|
|
TheCopy->eraseFromParent(); // Don't mutate the global.
|
|
|
|
AI->eraseFromParent();
|
|
|
|
++NumGlobals;
|
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
2006-12-22 23:14:42 +00:00
|
|
|
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
// Check to see if we can perform the core SROA transformation. We cannot
|
|
|
|
// transform the allocation instruction if it is an array allocation
|
|
|
|
// (allocations OF arrays are ok though), and an allocation of a scalar
|
|
|
|
// value cannot be decomposed at all.
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t AllocaSize = TD->getTypeAllocSize(AI->getAllocatedType());
|
2009-03-03 12:12:58 +00:00
|
|
|
|
|
|
|
// Do not promote any struct whose size is too big.
|
2009-03-03 19:18:49 +00:00
|
|
|
if (AllocaSize > SRThreshold) continue;
|
2009-03-01 03:55:12 +00:00
|
|
|
|
2009-02-03 01:30:09 +00:00
|
|
|
if ((isa<StructType>(AI->getAllocatedType()) ||
|
2007-05-24 18:43:04 +00:00
|
|
|
isa<ArrayType>(AI->getAllocatedType())) &&
|
2008-06-22 17:46:21 +00:00
|
|
|
// Do not promote any struct into more than "32" separate vars.
|
2009-03-06 00:56:43 +00:00
|
|
|
getNumSAElements(AI->getAllocatedType()) <= SRThreshold/4) {
|
2007-04-25 05:02:56 +00:00
|
|
|
// Check that all of the users of the allocation are capable of being
|
|
|
|
// transformed.
|
|
|
|
switch (isSafeAllocaToScalarRepl(AI)) {
|
2009-07-14 16:55:14 +00:00
|
|
|
default: llvm_unreachable("Unexpected value!");
|
2007-04-25 05:02:56 +00:00
|
|
|
case 0: // Not safe to scalar replace.
|
|
|
|
break;
|
|
|
|
case 1: // Safe, but requires cleanup/canonicalizations first
|
2009-02-10 07:00:59 +00:00
|
|
|
CleanupAllocaUsers(AI);
|
2007-04-25 05:02:56 +00:00
|
|
|
// FALL THROUGH.
|
|
|
|
case 3: // Safe to scalar replace.
|
|
|
|
DoScalarReplacement(AI, WorkList);
|
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
2004-11-14 04:24:28 +00:00
|
|
|
}
|
2009-01-28 20:16:43 +00:00
|
|
|
|
|
|
|
// If we can turn this aggregate value (potentially with casts) into a
|
|
|
|
// simple scalar value that can be mem2reg'd into a register value.
|
2009-01-31 02:28:54 +00:00
|
|
|
// IsNotTrivial tracks whether this is something that mem2reg could have
|
|
|
|
// promoted itself. If so, we don't want to transform it needlessly. Note
|
|
|
|
// that we can't just check based on the type: the alloca may be of an i32
|
|
|
|
// but that has pointer arithmetic to set byte 3 of it or something.
|
2009-01-28 20:16:43 +00:00
|
|
|
bool IsNotTrivial = false;
|
2009-02-03 01:30:09 +00:00
|
|
|
const Type *VectorTy = 0;
|
2009-02-03 18:15:05 +00:00
|
|
|
bool HadAVector = false;
|
|
|
|
if (CanConvertToScalar(AI, IsNotTrivial, VectorTy, HadAVector,
|
2009-03-04 19:22:30 +00:00
|
|
|
0, unsigned(AllocaSize)) && IsNotTrivial) {
|
2009-02-03 01:30:09 +00:00
|
|
|
AllocaInst *NewAI;
|
2009-02-03 18:15:05 +00:00
|
|
|
// If we were able to find a vector type that can handle this with
|
|
|
|
// insert/extract elements, and if there was at least one use that had
|
|
|
|
// a vector type, promote this to a vector. We don't want to promote
|
|
|
|
// random stuff that doesn't use vectors (e.g. <9 x double>) because then
|
|
|
|
// we just get a lot of insert/extracts. If at least one vector is
|
|
|
|
// involved, then we probably really do have a union of vector/array.
|
|
|
|
if (VectorTy && isa<VectorType>(VectorTy) && HadAVector) {
|
2009-02-03 01:30:09 +00:00
|
|
|
DOUT << "CONVERT TO VECTOR: " << *AI << " TYPE = " << *VectorTy <<"\n";
|
2009-02-02 20:44:45 +00:00
|
|
|
|
2009-02-03 01:30:09 +00:00
|
|
|
// Create and insert the vector alloca.
|
2009-07-15 23:53:25 +00:00
|
|
|
NewAI = new AllocaInst(VectorTy, 0, "", AI->getParent()->begin());
|
2009-02-03 01:30:09 +00:00
|
|
|
ConvertUsesToScalar(AI, NewAI, 0);
|
|
|
|
} else {
|
|
|
|
DOUT << "CONVERT TO SCALAR INTEGER: " << *AI << "\n";
|
|
|
|
|
|
|
|
// Create and insert the integer alloca.
|
2009-07-22 00:24:57 +00:00
|
|
|
const Type *NewTy = F.getContext().getIntegerType(AllocaSize*8);
|
2009-07-15 23:53:25 +00:00
|
|
|
NewAI = new AllocaInst(NewTy, 0, "", AI->getParent()->begin());
|
2009-02-02 20:44:45 +00:00
|
|
|
ConvertUsesToScalar(AI, NewAI, 0);
|
2009-01-28 20:16:43 +00:00
|
|
|
}
|
2009-02-03 01:30:09 +00:00
|
|
|
NewAI->takeName(AI);
|
|
|
|
AI->eraseFromParent();
|
|
|
|
++NumConverted;
|
|
|
|
Changed = true;
|
|
|
|
continue;
|
|
|
|
}
|
2009-01-28 20:16:43 +00:00
|
|
|
|
2009-02-03 01:30:09 +00:00
|
|
|
// Otherwise, couldn't process this alloca.
|
2007-04-25 05:02:56 +00:00
|
|
|
}
|
2003-05-27 15:45:27 +00:00
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
return Changed;
|
|
|
|
}
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
/// DoScalarReplacement - This alloca satisfied the isSafeAllocaToScalarRepl
|
|
|
|
/// predicate, do SROA now.
|
|
|
|
void SROA::DoScalarReplacement(AllocationInst *AI,
|
|
|
|
std::vector<AllocationInst*> &WorkList) {
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
DOUT << "Found inst to SROA: " << *AI;
|
2007-04-25 05:02:56 +00:00
|
|
|
SmallVector<AllocaInst*, 32> ElementAllocas;
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = AI->getContext();
|
2007-04-25 05:02:56 +00:00
|
|
|
if (const StructType *ST = dyn_cast<StructType>(AI->getAllocatedType())) {
|
|
|
|
ElementAllocas.reserve(ST->getNumContainedTypes());
|
|
|
|
for (unsigned i = 0, e = ST->getNumContainedTypes(); i != e; ++i) {
|
2009-07-15 23:53:25 +00:00
|
|
|
AllocaInst *NA = new AllocaInst(ST->getContainedType(i), 0,
|
2007-04-25 05:02:56 +00:00
|
|
|
AI->getAlignment(),
|
|
|
|
AI->getName() + "." + utostr(i), AI);
|
|
|
|
ElementAllocas.push_back(NA);
|
|
|
|
WorkList.push_back(NA); // Add to worklist for recursive processing
|
|
|
|
}
|
|
|
|
} else {
|
|
|
|
const ArrayType *AT = cast<ArrayType>(AI->getAllocatedType());
|
|
|
|
ElementAllocas.reserve(AT->getNumElements());
|
|
|
|
const Type *ElTy = AT->getElementType();
|
|
|
|
for (unsigned i = 0, e = AT->getNumElements(); i != e; ++i) {
|
2009-07-15 23:53:25 +00:00
|
|
|
AllocaInst *NA = new AllocaInst(ElTy, 0, AI->getAlignment(),
|
2007-04-25 05:02:56 +00:00
|
|
|
AI->getName() + "." + utostr(i), AI);
|
|
|
|
ElementAllocas.push_back(NA);
|
|
|
|
WorkList.push_back(NA); // Add to worklist for recursive processing
|
2003-05-27 15:45:27 +00:00
|
|
|
}
|
2007-04-25 05:02:56 +00:00
|
|
|
}
|
2005-04-21 23:48:37 +00:00
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
// Now that we have created the alloca instructions that we want to use,
|
|
|
|
// expand the getelementptr instructions to use them.
|
|
|
|
//
|
|
|
|
while (!AI->use_empty()) {
|
|
|
|
Instruction *User = cast<Instruction>(AI->use_back());
|
|
|
|
if (BitCastInst *BCInst = dyn_cast<BitCastInst>(User)) {
|
|
|
|
RewriteBitCastUserOfAlloca(BCInst, AI, ElementAllocas);
|
|
|
|
BCInst->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2008-06-23 17:11:23 +00:00
|
|
|
// Replace:
|
|
|
|
// %res = load { i32, i32 }* %alloc
|
|
|
|
// with:
|
|
|
|
// %load.0 = load i32* %alloc.0
|
|
|
|
// %insert.0 insertvalue { i32, i32 } zeroinitializer, i32 %load.0, 0
|
|
|
|
// %load.1 = load i32* %alloc.1
|
|
|
|
// %insert = insertvalue { i32, i32 } %insert.0, i32 %load.1, 1
|
2008-06-05 12:51:53 +00:00
|
|
|
// (Also works for arrays instead of structs)
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(User)) {
|
2009-07-22 00:24:57 +00:00
|
|
|
Value *Insert = Context.getUndef(LI->getType());
|
2008-06-05 12:51:53 +00:00
|
|
|
for (unsigned i = 0, e = ElementAllocas.size(); i != e; ++i) {
|
|
|
|
Value *Load = new LoadInst(ElementAllocas[i], "load", LI);
|
|
|
|
Insert = InsertValueInst::Create(Insert, Load, i, "insert", LI);
|
|
|
|
}
|
|
|
|
LI->replaceAllUsesWith(Insert);
|
|
|
|
LI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2008-06-23 17:11:23 +00:00
|
|
|
// Replace:
|
|
|
|
// store { i32, i32 } %val, { i32, i32 }* %alloc
|
|
|
|
// with:
|
|
|
|
// %val.0 = extractvalue { i32, i32 } %val, 0
|
|
|
|
// store i32 %val.0, i32* %alloc.0
|
|
|
|
// %val.1 = extractvalue { i32, i32 } %val, 1
|
|
|
|
// store i32 %val.1, i32* %alloc.1
|
2008-06-05 12:51:53 +00:00
|
|
|
// (Also works for arrays instead of structs)
|
|
|
|
if (StoreInst *SI = dyn_cast<StoreInst>(User)) {
|
|
|
|
Value *Val = SI->getOperand(0);
|
|
|
|
for (unsigned i = 0, e = ElementAllocas.size(); i != e; ++i) {
|
|
|
|
Value *Extract = ExtractValueInst::Create(Val, i, Val->getName(), SI);
|
|
|
|
new StoreInst(Extract, ElementAllocas[i], SI);
|
|
|
|
}
|
|
|
|
SI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
GetElementPtrInst *GEPI = cast<GetElementPtrInst>(User);
|
|
|
|
// We now know that the GEP is of the form: GEP <ptr>, 0, <cst>
|
|
|
|
unsigned Idx =
|
|
|
|
(unsigned)cast<ConstantInt>(GEPI->getOperand(2))->getZExtValue();
|
|
|
|
|
|
|
|
assert(Idx < ElementAllocas.size() && "Index out of range?");
|
|
|
|
AllocaInst *AllocaToUse = ElementAllocas[Idx];
|
|
|
|
|
|
|
|
Value *RepValue;
|
|
|
|
if (GEPI->getNumOperands() == 3) {
|
|
|
|
// Do not insert a new getelementptr instruction with zero indices, only
|
|
|
|
// to have it optimized out later.
|
|
|
|
RepValue = AllocaToUse;
|
|
|
|
} else {
|
|
|
|
// We are indexing deeply into the structure, so we still need a
|
|
|
|
// getelement ptr instruction to finish the indexing. This may be
|
|
|
|
// expanded itself once the worklist is rerun.
|
|
|
|
//
|
|
|
|
SmallVector<Value*, 8> NewArgs;
|
2009-07-22 00:24:57 +00:00
|
|
|
NewArgs.push_back(Context.getNullValue(Type::Int32Ty));
|
2007-04-25 05:02:56 +00:00
|
|
|
NewArgs.append(GEPI->op_begin()+3, GEPI->op_end());
|
2008-04-06 20:25:17 +00:00
|
|
|
RepValue = GetElementPtrInst::Create(AllocaToUse, NewArgs.begin(),
|
|
|
|
NewArgs.end(), "", GEPI);
|
2007-04-25 05:02:56 +00:00
|
|
|
RepValue->takeName(GEPI);
|
|
|
|
}
|
|
|
|
|
|
|
|
// If this GEP is to the start of the aggregate, check for memcpys.
|
2009-01-07 06:25:07 +00:00
|
|
|
if (Idx == 0 && GEPI->hasAllZeroIndices())
|
|
|
|
RewriteBitCastUserOfAlloca(GEPI, AI, ElementAllocas);
|
2003-05-27 15:45:27 +00:00
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
// Move all of the users over to the new GEP.
|
|
|
|
GEPI->replaceAllUsesWith(RepValue);
|
|
|
|
// Delete the old GEP
|
|
|
|
GEPI->eraseFromParent();
|
2003-05-27 15:45:27 +00:00
|
|
|
}
|
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
// Finally, delete the Alloca instruction
|
|
|
|
AI->eraseFromParent();
|
|
|
|
NumReplaced++;
|
2003-05-27 15:45:27 +00:00
|
|
|
}
|
2003-05-30 04:15:41 +00:00
|
|
|
|
|
|
|
|
2004-11-14 04:24:28 +00:00
|
|
|
/// isSafeElementUse - Check to see if this use is an allowed use for a
|
2007-03-19 00:16:43 +00:00
|
|
|
/// getelementptr instruction of an array aggregate allocation. isFirstElt
|
|
|
|
/// indicates whether Ptr is known to the start of the aggregate.
|
2004-11-14 04:24:28 +00:00
|
|
|
///
|
2007-05-30 06:11:23 +00:00
|
|
|
void SROA::isSafeElementUse(Value *Ptr, bool isFirstElt, AllocationInst *AI,
|
|
|
|
AllocaInfo &Info) {
|
2004-11-14 04:24:28 +00:00
|
|
|
for (Value::use_iterator I = Ptr->use_begin(), E = Ptr->use_end();
|
|
|
|
I != E; ++I) {
|
|
|
|
Instruction *User = cast<Instruction>(*I);
|
|
|
|
switch (User->getOpcode()) {
|
|
|
|
case Instruction::Load: break;
|
|
|
|
case Instruction::Store:
|
|
|
|
// Store is ok if storing INTO the pointer, not storing the pointer
|
2007-05-30 06:11:23 +00:00
|
|
|
if (User->getOperand(0) == Ptr) return MarkUnsafe(Info);
|
2004-11-14 04:24:28 +00:00
|
|
|
break;
|
|
|
|
case Instruction::GetElementPtr: {
|
|
|
|
GetElementPtrInst *GEP = cast<GetElementPtrInst>(User);
|
2007-03-19 00:16:43 +00:00
|
|
|
bool AreAllZeroIndices = isFirstElt;
|
2004-11-14 04:24:28 +00:00
|
|
|
if (GEP->getNumOperands() > 1) {
|
2007-03-19 00:16:43 +00:00
|
|
|
if (!isa<ConstantInt>(GEP->getOperand(1)) ||
|
|
|
|
!cast<ConstantInt>(GEP->getOperand(1))->isZero())
|
2007-05-30 06:11:23 +00:00
|
|
|
// Using pointer arithmetic to navigate the array.
|
|
|
|
return MarkUnsafe(Info);
|
2007-03-19 00:16:43 +00:00
|
|
|
|
2009-01-07 06:25:07 +00:00
|
|
|
if (AreAllZeroIndices)
|
|
|
|
AreAllZeroIndices = GEP->hasAllZeroIndices();
|
2004-11-14 04:24:28 +00:00
|
|
|
}
|
2007-05-30 06:11:23 +00:00
|
|
|
isSafeElementUse(GEP, AreAllZeroIndices, AI, Info);
|
|
|
|
if (Info.isUnsafe) return;
|
2004-11-14 04:24:28 +00:00
|
|
|
break;
|
|
|
|
}
|
2007-03-19 00:16:43 +00:00
|
|
|
case Instruction::BitCast:
|
2007-05-30 06:11:23 +00:00
|
|
|
if (isFirstElt) {
|
|
|
|
isSafeUseOfBitCastedAllocation(cast<BitCastInst>(User), AI, Info);
|
|
|
|
if (Info.isUnsafe) return;
|
2007-03-19 00:16:43 +00:00
|
|
|
break;
|
2007-05-30 06:11:23 +00:00
|
|
|
}
|
2007-03-19 00:16:43 +00:00
|
|
|
DOUT << " Transformation preventing inst: " << *User;
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
2007-03-19 00:16:43 +00:00
|
|
|
case Instruction::Call:
|
|
|
|
if (MemIntrinsic *MI = dyn_cast<MemIntrinsic>(User)) {
|
2007-05-30 06:11:23 +00:00
|
|
|
if (isFirstElt) {
|
|
|
|
isSafeMemIntrinsicOnAllocation(MI, AI, I.getOperandNo(), Info);
|
|
|
|
if (Info.isUnsafe) return;
|
2007-03-19 00:16:43 +00:00
|
|
|
break;
|
2007-05-30 06:11:23 +00:00
|
|
|
}
|
2007-03-19 00:16:43 +00:00
|
|
|
}
|
|
|
|
DOUT << " Transformation preventing inst: " << *User;
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
2004-11-14 04:24:28 +00:00
|
|
|
default:
|
2006-11-26 09:46:52 +00:00
|
|
|
DOUT << " Transformation preventing inst: " << *User;
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
2004-11-14 04:24:28 +00:00
|
|
|
}
|
|
|
|
}
|
2007-05-30 06:11:23 +00:00
|
|
|
return; // All users look ok :)
|
2004-11-14 04:24:28 +00:00
|
|
|
}
|
|
|
|
|
2004-11-14 05:00:19 +00:00
|
|
|
/// AllUsersAreLoads - Return true if all users of this value are loads.
|
|
|
|
static bool AllUsersAreLoads(Value *Ptr) {
|
|
|
|
for (Value::use_iterator I = Ptr->use_begin(), E = Ptr->use_end();
|
|
|
|
I != E; ++I)
|
|
|
|
if (cast<Instruction>(*I)->getOpcode() != Instruction::Load)
|
|
|
|
return false;
|
2005-04-21 23:48:37 +00:00
|
|
|
return true;
|
2004-11-14 05:00:19 +00:00
|
|
|
}
|
|
|
|
|
2003-05-30 04:15:41 +00:00
|
|
|
/// isSafeUseOfAllocation - Check to see if this user is an allowed use for an
|
|
|
|
/// aggregate allocation.
|
|
|
|
///
|
2007-05-30 06:11:23 +00:00
|
|
|
void SROA::isSafeUseOfAllocation(Instruction *User, AllocationInst *AI,
|
|
|
|
AllocaInfo &Info) {
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = User->getContext();
|
2007-03-05 07:52:57 +00:00
|
|
|
if (BitCastInst *C = dyn_cast<BitCastInst>(User))
|
2007-05-30 06:11:23 +00:00
|
|
|
return isSafeUseOfBitCastedAllocation(C, AI, Info);
|
|
|
|
|
2009-01-28 20:16:43 +00:00
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(User))
|
|
|
|
if (!LI->isVolatile())
|
|
|
|
return;// Loads (returning a first class aggregrate) are always rewritable
|
2008-06-05 12:51:53 +00:00
|
|
|
|
2009-01-28 20:16:43 +00:00
|
|
|
if (StoreInst *SI = dyn_cast<StoreInst>(User))
|
|
|
|
if (!SI->isVolatile() && SI->getOperand(0) != AI)
|
|
|
|
return;// Store is ok if storing INTO the pointer, not storing the pointer
|
2008-06-05 12:51:53 +00:00
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(User);
|
|
|
|
if (GEPI == 0)
|
|
|
|
return MarkUnsafe(Info);
|
2003-10-29 17:55:44 +00:00
|
|
|
|
2003-11-25 21:09:18 +00:00
|
|
|
gep_type_iterator I = gep_type_begin(GEPI), E = gep_type_end(GEPI);
|
|
|
|
|
2006-03-08 01:05:29 +00:00
|
|
|
// The GEP is not safe to transform if not of the form "GEP <ptr>, 0, <cst>".
|
2003-11-25 21:09:18 +00:00
|
|
|
if (I == E ||
|
2009-07-22 00:24:57 +00:00
|
|
|
I.getOperand() != Context.getNullValue(I.getOperand()->getType())) {
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
|
|
|
}
|
2003-11-25 21:09:18 +00:00
|
|
|
|
|
|
|
++I;
|
2007-05-30 06:11:23 +00:00
|
|
|
if (I == E) return MarkUnsafe(Info); // ran out of GEP indices??
|
2003-10-29 17:55:44 +00:00
|
|
|
|
2007-03-19 00:16:43 +00:00
|
|
|
bool IsAllZeroIndices = true;
|
|
|
|
|
2008-08-23 05:21:06 +00:00
|
|
|
// If the first index is a non-constant index into an array, see if we can
|
|
|
|
// handle it as a special case.
|
2003-11-25 21:09:18 +00:00
|
|
|
if (const ArrayType *AT = dyn_cast<ArrayType>(*I)) {
|
2008-08-23 05:21:06 +00:00
|
|
|
if (!isa<ConstantInt>(I.getOperand())) {
|
2007-03-19 00:16:43 +00:00
|
|
|
IsAllZeroIndices = 0;
|
2008-08-23 05:21:06 +00:00
|
|
|
uint64_t NumElements = AT->getNumElements();
|
2007-03-19 00:16:43 +00:00
|
|
|
|
2004-11-14 05:00:19 +00:00
|
|
|
// If this is an array index and the index is not constant, we cannot
|
|
|
|
// promote... that is unless the array has exactly one or two elements in
|
|
|
|
// it, in which case we CAN promote it, but we have to canonicalize this
|
|
|
|
// out if this is the only problem.
|
2006-03-08 01:05:29 +00:00
|
|
|
if ((NumElements == 1 || NumElements == 2) &&
|
2007-05-30 06:11:23 +00:00
|
|
|
AllUsersAreLoads(GEPI)) {
|
2009-02-10 07:00:59 +00:00
|
|
|
Info.needsCleanup = true;
|
2007-05-30 06:11:23 +00:00
|
|
|
return; // Canonicalization required!
|
|
|
|
}
|
|
|
|
return MarkUnsafe(Info);
|
2004-11-14 05:00:19 +00:00
|
|
|
}
|
2003-05-30 04:15:41 +00:00
|
|
|
}
|
2008-10-06 16:23:31 +00:00
|
|
|
|
2008-08-23 05:21:06 +00:00
|
|
|
// Walk through the GEP type indices, checking the types that this indexes
|
|
|
|
// into.
|
|
|
|
for (; I != E; ++I) {
|
|
|
|
// Ignore struct elements, no extra checking needed for these.
|
|
|
|
if (isa<StructType>(*I))
|
|
|
|
continue;
|
|
|
|
|
|
|
|
ConstantInt *IdxVal = dyn_cast<ConstantInt>(I.getOperand());
|
|
|
|
if (!IdxVal) return MarkUnsafe(Info);
|
2008-10-06 16:23:31 +00:00
|
|
|
|
|
|
|
// Are all indices still zero?
|
2008-08-23 05:21:06 +00:00
|
|
|
IsAllZeroIndices &= IdxVal->isZero();
|
2008-10-06 16:23:31 +00:00
|
|
|
|
|
|
|
if (const ArrayType *AT = dyn_cast<ArrayType>(*I)) {
|
|
|
|
// This GEP indexes an array. Verify that this is an in-range constant
|
|
|
|
// integer. Specifically, consider A[0][i]. We cannot know that the user
|
|
|
|
// isn't doing invalid things like allowing i to index an out-of-range
|
|
|
|
// subscript that accesses A[1]. Because of this, we have to reject SROA
|
2008-11-04 20:54:03 +00:00
|
|
|
// of any accesses into structs where any of the components are variables.
|
2008-10-06 16:23:31 +00:00
|
|
|
if (IdxVal->getZExtValue() >= AT->getNumElements())
|
|
|
|
return MarkUnsafe(Info);
|
2008-11-04 20:54:03 +00:00
|
|
|
} else if (const VectorType *VT = dyn_cast<VectorType>(*I)) {
|
|
|
|
if (IdxVal->getZExtValue() >= VT->getNumElements())
|
|
|
|
return MarkUnsafe(Info);
|
2008-10-06 16:23:31 +00:00
|
|
|
}
|
2008-08-23 05:21:06 +00:00
|
|
|
}
|
|
|
|
|
2003-11-25 21:09:18 +00:00
|
|
|
// If there are any non-simple uses of this getelementptr, make sure to reject
|
|
|
|
// them.
|
2007-05-30 06:11:23 +00:00
|
|
|
return isSafeElementUse(GEPI, IsAllZeroIndices, AI, Info);
|
2007-03-19 00:16:43 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/// isSafeMemIntrinsicOnAllocation - Return true if the specified memory
|
|
|
|
/// intrinsic can be promoted by SROA. At this point, we know that the operand
|
|
|
|
/// of the memintrinsic is a pointer to the beginning of the allocation.
|
2007-05-30 06:11:23 +00:00
|
|
|
void SROA::isSafeMemIntrinsicOnAllocation(MemIntrinsic *MI, AllocationInst *AI,
|
|
|
|
unsigned OpNo, AllocaInfo &Info) {
|
2007-03-19 00:16:43 +00:00
|
|
|
// If not constant length, give up.
|
|
|
|
ConstantInt *Length = dyn_cast<ConstantInt>(MI->getLength());
|
2007-05-30 06:11:23 +00:00
|
|
|
if (!Length) return MarkUnsafe(Info);
|
2007-03-19 00:16:43 +00:00
|
|
|
|
|
|
|
// If not the whole aggregate, give up.
|
2007-11-04 14:43:57 +00:00
|
|
|
if (Length->getZExtValue() !=
|
2009-05-09 07:06:46 +00:00
|
|
|
TD->getTypeAllocSize(AI->getType()->getElementType()))
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
2007-03-19 00:16:43 +00:00
|
|
|
|
|
|
|
// We only know about memcpy/memset/memmove.
|
2009-03-08 03:37:16 +00:00
|
|
|
if (!isa<MemIntrinsic>(MI))
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
|
|
|
|
|
|
|
// Otherwise, we can transform it. Determine whether this is a memcpy/set
|
|
|
|
// into or out of the aggregate.
|
|
|
|
if (OpNo == 1)
|
|
|
|
Info.isMemCpyDst = true;
|
|
|
|
else {
|
|
|
|
assert(OpNo == 2);
|
|
|
|
Info.isMemCpySrc = true;
|
|
|
|
}
|
2003-05-30 04:15:41 +00:00
|
|
|
}
|
|
|
|
|
2007-03-05 07:52:57 +00:00
|
|
|
/// isSafeUseOfBitCastedAllocation - Return true if all users of this bitcast
|
|
|
|
/// are
|
2007-05-30 06:11:23 +00:00
|
|
|
void SROA::isSafeUseOfBitCastedAllocation(BitCastInst *BC, AllocationInst *AI,
|
|
|
|
AllocaInfo &Info) {
|
2007-03-05 07:52:57 +00:00
|
|
|
for (Value::use_iterator UI = BC->use_begin(), E = BC->use_end();
|
|
|
|
UI != E; ++UI) {
|
|
|
|
if (BitCastInst *BCU = dyn_cast<BitCastInst>(UI)) {
|
2007-05-30 06:11:23 +00:00
|
|
|
isSafeUseOfBitCastedAllocation(BCU, AI, Info);
|
2007-03-05 07:52:57 +00:00
|
|
|
} else if (MemIntrinsic *MI = dyn_cast<MemIntrinsic>(UI)) {
|
2007-05-30 06:11:23 +00:00
|
|
|
isSafeMemIntrinsicOnAllocation(MI, AI, UI.getOperandNo(), Info);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
} else if (StoreInst *SI = dyn_cast<StoreInst>(UI)) {
|
2009-01-28 20:16:43 +00:00
|
|
|
if (SI->isVolatile())
|
|
|
|
return MarkUnsafe(Info);
|
|
|
|
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
// If storing the entire alloca in one chunk through a bitcasted pointer
|
|
|
|
// to integer, we can transform it. This happens (for example) when you
|
|
|
|
// cast a {i32,i32}* to i64* and store through it. This is similar to the
|
|
|
|
// memcpy case and occurs in various "byval" cases and emulated memcpys.
|
|
|
|
if (isa<IntegerType>(SI->getOperand(0)->getType()) &&
|
2009-05-09 07:06:46 +00:00
|
|
|
TD->getTypeAllocSize(SI->getOperand(0)->getType()) ==
|
|
|
|
TD->getTypeAllocSize(AI->getType()->getElementType())) {
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
Info.isMemCpyDst = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
return MarkUnsafe(Info);
|
2009-01-08 05:42:05 +00:00
|
|
|
} else if (LoadInst *LI = dyn_cast<LoadInst>(UI)) {
|
2009-01-28 20:16:43 +00:00
|
|
|
if (LI->isVolatile())
|
|
|
|
return MarkUnsafe(Info);
|
|
|
|
|
2009-01-08 05:42:05 +00:00
|
|
|
// If loading the entire alloca in one chunk through a bitcasted pointer
|
|
|
|
// to integer, we can transform it. This happens (for example) when you
|
|
|
|
// cast a {i32,i32}* to i64* and load through it. This is similar to the
|
|
|
|
// memcpy case and occurs in various "byval" cases and emulated memcpys.
|
|
|
|
if (isa<IntegerType>(LI->getType()) &&
|
2009-05-09 07:06:46 +00:00
|
|
|
TD->getTypeAllocSize(LI->getType()) ==
|
|
|
|
TD->getTypeAllocSize(AI->getType()->getElementType())) {
|
2009-01-08 05:42:05 +00:00
|
|
|
Info.isMemCpySrc = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
return MarkUnsafe(Info);
|
2009-02-10 07:00:59 +00:00
|
|
|
} else if (isa<DbgInfoIntrinsic>(UI)) {
|
|
|
|
// If one user is DbgInfoIntrinsic then check if all users are
|
|
|
|
// DbgInfoIntrinsics.
|
|
|
|
if (OnlyUsedByDbgInfoIntrinsics(BC)) {
|
|
|
|
Info.needsCleanup = true;
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
else
|
|
|
|
MarkUnsafe(Info);
|
|
|
|
}
|
|
|
|
else {
|
2007-05-30 06:11:23 +00:00
|
|
|
return MarkUnsafe(Info);
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
2007-05-30 06:11:23 +00:00
|
|
|
if (Info.isUnsafe) return;
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2007-03-19 00:16:43 +00:00
|
|
|
/// RewriteBitCastUserOfAlloca - BCInst (transitively) bitcasts AI, or indexes
|
|
|
|
/// to its first element. Transform users of the cast to use the new values
|
|
|
|
/// instead.
|
|
|
|
void SROA::RewriteBitCastUserOfAlloca(Instruction *BCInst, AllocationInst *AI,
|
2007-03-05 07:52:57 +00:00
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
2007-03-19 00:16:43 +00:00
|
|
|
Value::use_iterator UI = BCInst->use_begin(), UE = BCInst->use_end();
|
|
|
|
while (UI != UE) {
|
2009-01-07 07:18:45 +00:00
|
|
|
Instruction *User = cast<Instruction>(*UI++);
|
|
|
|
if (BitCastInst *BCU = dyn_cast<BitCastInst>(User)) {
|
2007-03-05 07:52:57 +00:00
|
|
|
RewriteBitCastUserOfAlloca(BCU, AI, NewElts);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
if (BCU->use_empty()) BCU->eraseFromParent();
|
2007-03-05 07:52:57 +00:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
if (MemIntrinsic *MI = dyn_cast<MemIntrinsic>(User)) {
|
|
|
|
// This must be memcpy/memmove/memset of the entire aggregate.
|
|
|
|
// Split into one per element.
|
|
|
|
RewriteMemIntrinUserOfAlloca(MI, BCInst, AI, NewElts);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
if (StoreInst *SI = dyn_cast<StoreInst>(User)) {
|
2009-01-08 05:42:05 +00:00
|
|
|
// If this is a store of the entire alloca from an integer, rewrite it.
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
RewriteStoreUserOfWholeAlloca(SI, AI, NewElts);
|
|
|
|
continue;
|
|
|
|
}
|
2009-01-08 05:42:05 +00:00
|
|
|
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(User)) {
|
|
|
|
// If this is a load of the entire alloca to an integer, rewrite it.
|
|
|
|
RewriteLoadUserOfWholeAlloca(LI, AI, NewElts);
|
|
|
|
continue;
|
|
|
|
}
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
|
|
|
|
// Otherwise it must be some other user of a gep of the first pointer. Just
|
|
|
|
// leave these alone.
|
2009-01-07 07:18:45 +00:00
|
|
|
continue;
|
2009-01-08 05:42:05 +00:00
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
/// RewriteMemIntrinUserOfAlloca - MI is a memcpy/memset/memmove from or to AI.
|
|
|
|
/// Rewrite it to copy or set the elements of the scalarized memory.
|
|
|
|
void SROA::RewriteMemIntrinUserOfAlloca(MemIntrinsic *MI, Instruction *BCInst,
|
|
|
|
AllocationInst *AI,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
|
|
|
|
|
|
|
// If this is a memcpy/memmove, construct the other pointer as the
|
2009-03-04 19:23:25 +00:00
|
|
|
// appropriate type. The "Other" pointer is the pointer that goes to memory
|
|
|
|
// that doesn't have anything to do with the alloca that we are promoting. For
|
|
|
|
// memset, this Value* stays null.
|
2009-01-07 07:18:45 +00:00
|
|
|
Value *OtherPtr = 0;
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = MI->getContext();
|
2009-03-08 03:59:00 +00:00
|
|
|
unsigned MemAlignment = MI->getAlignment();
|
2009-03-08 03:37:16 +00:00
|
|
|
if (MemTransferInst *MTI = dyn_cast<MemTransferInst>(MI)) { // memmove/memcopy
|
|
|
|
if (BCInst == MTI->getRawDest())
|
|
|
|
OtherPtr = MTI->getRawSource();
|
2009-01-07 07:18:45 +00:00
|
|
|
else {
|
2009-03-08 03:37:16 +00:00
|
|
|
assert(BCInst == MTI->getRawSource());
|
|
|
|
OtherPtr = MTI->getRawDest();
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// If there is an other pointer, we want to convert it to the same pointer
|
|
|
|
// type as AI has, so we can GEP through it safely.
|
|
|
|
if (OtherPtr) {
|
|
|
|
// It is likely that OtherPtr is a bitcast, if so, remove it.
|
|
|
|
if (BitCastInst *BC = dyn_cast<BitCastInst>(OtherPtr))
|
|
|
|
OtherPtr = BC->getOperand(0);
|
|
|
|
// All zero GEPs are effectively bitcasts.
|
|
|
|
if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(OtherPtr))
|
|
|
|
if (GEP->hasAllZeroIndices())
|
|
|
|
OtherPtr = GEP->getOperand(0);
|
2007-03-05 07:52:57 +00:00
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
if (ConstantExpr *BCE = dyn_cast<ConstantExpr>(OtherPtr))
|
|
|
|
if (BCE->getOpcode() == Instruction::BitCast)
|
|
|
|
OtherPtr = BCE->getOperand(0);
|
|
|
|
|
|
|
|
// If the pointer is not the right type, insert a bitcast to the right
|
|
|
|
// type.
|
|
|
|
if (OtherPtr->getType() != AI->getType())
|
|
|
|
OtherPtr = new BitCastInst(OtherPtr, AI->getType(), OtherPtr->getName(),
|
|
|
|
MI);
|
|
|
|
}
|
|
|
|
|
|
|
|
// Process each element of the aggregate.
|
|
|
|
Value *TheFn = MI->getOperand(0);
|
|
|
|
const Type *BytePtrTy = MI->getRawDest()->getType();
|
|
|
|
bool SROADest = MI->getRawDest() == BCInst;
|
|
|
|
|
2009-07-22 00:24:57 +00:00
|
|
|
Constant *Zero = Context.getNullValue(Type::Int32Ty);
|
2007-03-05 07:52:57 +00:00
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
|
|
|
|
// If this is a memcpy/memmove, emit a GEP of the other element address.
|
|
|
|
Value *OtherElt = 0;
|
2009-03-04 19:20:50 +00:00
|
|
|
unsigned OtherEltAlign = MemAlignment;
|
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
if (OtherPtr) {
|
2009-07-24 23:12:02 +00:00
|
|
|
Value *Idx[2] = { Zero, ConstantInt::get(Type::Int32Ty, i) };
|
2009-01-07 07:18:45 +00:00
|
|
|
OtherElt = GetElementPtrInst::Create(OtherPtr, Idx, Idx + 2,
|
2008-06-22 17:46:21 +00:00
|
|
|
OtherPtr->getNameStr()+"."+utostr(i),
|
2009-01-07 07:18:45 +00:00
|
|
|
MI);
|
2009-03-04 19:20:50 +00:00
|
|
|
uint64_t EltOffset;
|
|
|
|
const PointerType *OtherPtrTy = cast<PointerType>(OtherPtr->getType());
|
|
|
|
if (const StructType *ST =
|
|
|
|
dyn_cast<StructType>(OtherPtrTy->getElementType())) {
|
|
|
|
EltOffset = TD->getStructLayout(ST)->getElementOffset(i);
|
|
|
|
} else {
|
|
|
|
const Type *EltTy =
|
|
|
|
cast<SequentialType>(OtherPtr->getType())->getElementType();
|
2009-05-09 07:06:46 +00:00
|
|
|
EltOffset = TD->getTypeAllocSize(EltTy)*i;
|
2009-03-04 19:20:50 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// The alignment of the other pointer is the guaranteed alignment of the
|
|
|
|
// element, which is affected by both the known alignment of the whole
|
|
|
|
// mem intrinsic and the alignment of the element. If the alignment of
|
|
|
|
// the memcpy (f.e.) is 32 but the element is at a 4-byte offset, then the
|
|
|
|
// known alignment is just 4 bytes.
|
|
|
|
OtherEltAlign = (unsigned)MinAlign(OtherEltAlign, EltOffset);
|
2009-01-07 07:18:45 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
Value *EltPtr = NewElts[i];
|
2009-03-04 19:20:50 +00:00
|
|
|
const Type *EltTy = cast<PointerType>(EltPtr->getType())->getElementType();
|
2009-01-07 07:18:45 +00:00
|
|
|
|
|
|
|
// If we got down to a scalar, insert a load or store as appropriate.
|
|
|
|
if (EltTy->isSingleValueType()) {
|
2009-03-08 03:37:16 +00:00
|
|
|
if (isa<MemTransferInst>(MI)) {
|
2009-03-04 19:20:50 +00:00
|
|
|
if (SROADest) {
|
|
|
|
// From Other to Alloca.
|
|
|
|
Value *Elt = new LoadInst(OtherElt, "tmp", false, OtherEltAlign, MI);
|
|
|
|
new StoreInst(Elt, EltPtr, MI);
|
|
|
|
} else {
|
|
|
|
// From Alloca to Other.
|
|
|
|
Value *Elt = new LoadInst(EltPtr, "tmp", MI);
|
|
|
|
new StoreInst(Elt, OtherElt, false, OtherEltAlign, MI);
|
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
continue;
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
assert(isa<MemSetInst>(MI));
|
2007-03-08 06:36:54 +00:00
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
// If the stored element is zero (common case), just store a null
|
|
|
|
// constant.
|
|
|
|
Constant *StoreVal;
|
|
|
|
if (ConstantInt *CI = dyn_cast<ConstantInt>(MI->getOperand(2))) {
|
|
|
|
if (CI->isZero()) {
|
2009-07-22 00:24:57 +00:00
|
|
|
StoreVal = Context.getNullValue(EltTy); // 0.0, null, 0, <0,0>
|
2007-03-08 06:36:54 +00:00
|
|
|
} else {
|
2009-01-07 07:18:45 +00:00
|
|
|
// If EltTy is a vector type, get the element type.
|
2009-06-16 00:20:26 +00:00
|
|
|
const Type *ValTy = EltTy->getScalarType();
|
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
// Construct an integer with the right value.
|
|
|
|
unsigned EltSize = TD->getTypeSizeInBits(ValTy);
|
|
|
|
APInt OneVal(EltSize, CI->getZExtValue());
|
|
|
|
APInt TotalVal(OneVal);
|
|
|
|
// Set each byte.
|
|
|
|
for (unsigned i = 0; 8*i < EltSize; ++i) {
|
|
|
|
TotalVal = TotalVal.shl(8);
|
|
|
|
TotalVal |= OneVal;
|
|
|
|
}
|
|
|
|
|
|
|
|
// Convert the integer value to the appropriate type.
|
2009-07-24 23:12:02 +00:00
|
|
|
StoreVal = ConstantInt::get(Context, TotalVal);
|
2009-01-07 07:18:45 +00:00
|
|
|
if (isa<PointerType>(ValTy))
|
2009-07-29 18:55:55 +00:00
|
|
|
StoreVal = ConstantExpr::getIntToPtr(StoreVal, ValTy);
|
2009-01-07 07:18:45 +00:00
|
|
|
else if (ValTy->isFloatingPoint())
|
2009-07-29 18:55:55 +00:00
|
|
|
StoreVal = ConstantExpr::getBitCast(StoreVal, ValTy);
|
2009-01-07 07:18:45 +00:00
|
|
|
assert(StoreVal->getType() == ValTy && "Type mismatch!");
|
|
|
|
|
|
|
|
// If the requested value was a vector constant, create it.
|
|
|
|
if (EltTy != ValTy) {
|
|
|
|
unsigned NumElts = cast<VectorType>(ValTy)->getNumElements();
|
|
|
|
SmallVector<Constant*, 16> Elts(NumElts, StoreVal);
|
2009-07-28 21:19:26 +00:00
|
|
|
StoreVal = ConstantVector::get(&Elts[0], NumElts);
|
2007-03-08 06:36:54 +00:00
|
|
|
}
|
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
new StoreInst(StoreVal, EltPtr, MI);
|
|
|
|
continue;
|
2007-03-08 06:36:54 +00:00
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
// Otherwise, if we're storing a byte variable, use a memset call for
|
|
|
|
// this element.
|
|
|
|
}
|
2007-03-08 06:36:54 +00:00
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
// Cast the element pointer to BytePtrTy.
|
|
|
|
if (EltPtr->getType() != BytePtrTy)
|
|
|
|
EltPtr = new BitCastInst(EltPtr, BytePtrTy, EltPtr->getNameStr(), MI);
|
2007-03-08 06:36:54 +00:00
|
|
|
|
2009-01-07 07:18:45 +00:00
|
|
|
// Cast the other pointer (if we have one) to BytePtrTy.
|
|
|
|
if (OtherElt && OtherElt->getType() != BytePtrTy)
|
|
|
|
OtherElt = new BitCastInst(OtherElt, BytePtrTy,OtherElt->getNameStr(),
|
|
|
|
MI);
|
|
|
|
|
2009-05-09 07:06:46 +00:00
|
|
|
unsigned EltSize = TD->getTypeAllocSize(EltTy);
|
2009-01-07 07:18:45 +00:00
|
|
|
|
|
|
|
// Finally, insert the meminst for this element.
|
2009-03-08 03:37:16 +00:00
|
|
|
if (isa<MemTransferInst>(MI)) {
|
2009-01-07 07:18:45 +00:00
|
|
|
Value *Ops[] = {
|
|
|
|
SROADest ? EltPtr : OtherElt, // Dest ptr
|
|
|
|
SROADest ? OtherElt : EltPtr, // Src ptr
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(MI->getOperand(3)->getType(), EltSize), // Size
|
|
|
|
ConstantInt::get(Type::Int32Ty, OtherEltAlign) // Align
|
2009-01-07 07:18:45 +00:00
|
|
|
};
|
|
|
|
CallInst::Create(TheFn, Ops, Ops + 4, "", MI);
|
|
|
|
} else {
|
|
|
|
assert(isa<MemSetInst>(MI));
|
|
|
|
Value *Ops[] = {
|
|
|
|
EltPtr, MI->getOperand(2), // Dest, Value,
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(MI->getOperand(3)->getType(), EltSize), // Size
|
2009-01-07 07:18:45 +00:00
|
|
|
Zero // Align
|
|
|
|
};
|
|
|
|
CallInst::Create(TheFn, Ops, Ops + 4, "", MI);
|
2007-03-08 06:36:54 +00:00
|
|
|
}
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
MI->eraseFromParent();
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
|
|
|
|
/// RewriteStoreUserOfWholeAlloca - We found an store of an integer that
|
|
|
|
/// overwrites the entire allocation. Extract out the pieces of the stored
|
|
|
|
/// integer and store them individually.
|
|
|
|
void SROA::RewriteStoreUserOfWholeAlloca(StoreInst *SI,
|
|
|
|
AllocationInst *AI,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts){
|
|
|
|
// Extract each element out of the integer according to its structure offset
|
|
|
|
// and store the element value to the individual alloca.
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = SI->getContext();
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
Value *SrcVal = SI->getOperand(0);
|
|
|
|
const Type *AllocaEltTy = AI->getType()->getElementType();
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t AllocaSizeBits = TD->getTypeAllocSizeInBits(AllocaEltTy);
|
2009-01-07 07:18:45 +00:00
|
|
|
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
// If this isn't a store of an integer to the whole alloca, it may be a store
|
|
|
|
// to the first element. Just ignore the store in this case and normal SROA
|
2009-06-01 09:14:32 +00:00
|
|
|
// will handle it.
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
if (!isa<IntegerType>(SrcVal->getType()) ||
|
2009-06-01 09:14:32 +00:00
|
|
|
TD->getTypeAllocSizeInBits(SrcVal->getType()) != AllocaSizeBits)
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
return;
|
2009-06-01 09:14:32 +00:00
|
|
|
// Handle tail padding by extending the operand
|
|
|
|
if (TD->getTypeSizeInBits(SrcVal->getType()) != AllocaSizeBits)
|
2009-07-03 19:42:02 +00:00
|
|
|
SrcVal = new ZExtInst(SrcVal,
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getIntegerType(AllocaSizeBits), "", SI);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
|
|
|
|
DOUT << "PROMOTING STORE TO WHOLE ALLOCA: " << *AI << *SI;
|
|
|
|
|
|
|
|
// There are two forms here: AI could be an array or struct. Both cases
|
|
|
|
// have different ways to compute the element offset.
|
|
|
|
if (const StructType *EltSTy = dyn_cast<StructType>(AllocaEltTy)) {
|
|
|
|
const StructLayout *Layout = TD->getStructLayout(EltSTy);
|
|
|
|
|
|
|
|
for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
|
|
|
|
// Get the number of bits to shift SrcVal to get the value.
|
|
|
|
const Type *FieldTy = EltSTy->getElementType(i);
|
|
|
|
uint64_t Shift = Layout->getElementOffsetInBits(i);
|
|
|
|
|
|
|
|
if (TD->isBigEndian())
|
2009-05-09 07:06:46 +00:00
|
|
|
Shift = AllocaSizeBits-Shift-TD->getTypeAllocSizeInBits(FieldTy);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
|
|
|
|
Value *EltVal = SrcVal;
|
|
|
|
if (Shift) {
|
2009-07-24 23:12:02 +00:00
|
|
|
Value *ShiftVal = ConstantInt::get(EltVal->getType(), Shift);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
EltVal = BinaryOperator::CreateLShr(EltVal, ShiftVal,
|
|
|
|
"sroa.store.elt", SI);
|
|
|
|
}
|
|
|
|
|
|
|
|
// Truncate down to an integer of the right size.
|
|
|
|
uint64_t FieldSizeBits = TD->getTypeSizeInBits(FieldTy);
|
2009-01-09 18:18:43 +00:00
|
|
|
|
|
|
|
// Ignore zero sized fields like {}, they obviously contain no data.
|
|
|
|
if (FieldSizeBits == 0) continue;
|
|
|
|
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
if (FieldSizeBits != AllocaSizeBits)
|
2009-07-03 19:42:02 +00:00
|
|
|
EltVal = new TruncInst(EltVal,
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getIntegerType(FieldSizeBits), "", SI);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
Value *DestField = NewElts[i];
|
|
|
|
if (EltVal->getType() == FieldTy) {
|
|
|
|
// Storing to an integer field of this size, just do it.
|
|
|
|
} else if (FieldTy->isFloatingPoint() || isa<VectorType>(FieldTy)) {
|
|
|
|
// Bitcast to the right element type (for fp/vector values).
|
|
|
|
EltVal = new BitCastInst(EltVal, FieldTy, "", SI);
|
|
|
|
} else {
|
|
|
|
// Otherwise, bitcast the dest pointer (for aggregates).
|
|
|
|
DestField = new BitCastInst(DestField,
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getPointerTypeUnqual(EltVal->getType()),
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
"", SI);
|
|
|
|
}
|
|
|
|
new StoreInst(EltVal, DestField, SI);
|
|
|
|
}
|
|
|
|
|
|
|
|
} else {
|
|
|
|
const ArrayType *ATy = cast<ArrayType>(AllocaEltTy);
|
|
|
|
const Type *ArrayEltTy = ATy->getElementType();
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t ElementOffset = TD->getTypeAllocSizeInBits(ArrayEltTy);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
uint64_t ElementSizeBits = TD->getTypeSizeInBits(ArrayEltTy);
|
|
|
|
|
|
|
|
uint64_t Shift;
|
|
|
|
|
|
|
|
if (TD->isBigEndian())
|
|
|
|
Shift = AllocaSizeBits-ElementOffset;
|
|
|
|
else
|
|
|
|
Shift = 0;
|
|
|
|
|
|
|
|
for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
|
2009-01-09 18:18:43 +00:00
|
|
|
// Ignore zero sized fields like {}, they obviously contain no data.
|
|
|
|
if (ElementSizeBits == 0) continue;
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
|
|
|
|
Value *EltVal = SrcVal;
|
|
|
|
if (Shift) {
|
2009-07-24 23:12:02 +00:00
|
|
|
Value *ShiftVal = ConstantInt::get(EltVal->getType(), Shift);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
EltVal = BinaryOperator::CreateLShr(EltVal, ShiftVal,
|
|
|
|
"sroa.store.elt", SI);
|
|
|
|
}
|
|
|
|
|
|
|
|
// Truncate down to an integer of the right size.
|
|
|
|
if (ElementSizeBits != AllocaSizeBits)
|
2009-07-03 19:42:02 +00:00
|
|
|
EltVal = new TruncInst(EltVal,
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getIntegerType(ElementSizeBits),"",SI);
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
Value *DestField = NewElts[i];
|
|
|
|
if (EltVal->getType() == ArrayEltTy) {
|
|
|
|
// Storing to an integer field of this size, just do it.
|
|
|
|
} else if (ArrayEltTy->isFloatingPoint() || isa<VectorType>(ArrayEltTy)) {
|
|
|
|
// Bitcast to the right element type (for fp/vector values).
|
|
|
|
EltVal = new BitCastInst(EltVal, ArrayEltTy, "", SI);
|
|
|
|
} else {
|
|
|
|
// Otherwise, bitcast the dest pointer (for aggregates).
|
|
|
|
DestField = new BitCastInst(DestField,
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getPointerTypeUnqual(EltVal->getType()),
|
Implement the first half of PR3290: if there is a store of an
integer to a (transitive) bitcast the alloca and if that integer
has the full size of the alloca, then it clobbers the whole thing.
Handle this by extracting pieces out of the stored integer and
filing them away in the SROA'd elements.
This triggers fairly frequently because the CFE uses integers to
pass small structs by value and the inliner exposes these. For
example, in kimwitu++, I see a bunch of these with i64 stores to
"%struct.std::pair<std::_Rb_tree_const_iterator<kc::impl_abstract_phylum*>,bool>"
In 176.gcc I see a few i32 stores to "%struct..0anon".
In the testcase, this is a difference between compiling test1 to:
_test1:
subl $12, %esp
movl 20(%esp), %eax
movl %eax, 4(%esp)
movl 16(%esp), %eax
movl %eax, (%esp)
movl (%esp), %eax
addl 4(%esp), %eax
addl $12, %esp
ret
vs:
_test1:
movl 8(%esp), %eax
addl 4(%esp), %eax
ret
The second half of this will be to handle loads of the same form.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@61853 91177308-0d34-0410-b5e6-96231b3b80d8
2009-01-07 08:11:13 +00:00
|
|
|
"", SI);
|
|
|
|
}
|
|
|
|
new StoreInst(EltVal, DestField, SI);
|
|
|
|
|
|
|
|
if (TD->isBigEndian())
|
|
|
|
Shift -= ElementOffset;
|
|
|
|
else
|
|
|
|
Shift += ElementOffset;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
SI->eraseFromParent();
|
|
|
|
}
|
|
|
|
|
2009-01-08 05:42:05 +00:00
|
|
|
/// RewriteLoadUserOfWholeAlloca - We found an load of the entire allocation to
|
|
|
|
/// an integer. Load the individual pieces to form the aggregate value.
|
|
|
|
void SROA::RewriteLoadUserOfWholeAlloca(LoadInst *LI, AllocationInst *AI,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
|
|
|
// Extract each element out of the NewElts according to its structure offset
|
|
|
|
// and form the result value.
|
|
|
|
const Type *AllocaEltTy = AI->getType()->getElementType();
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t AllocaSizeBits = TD->getTypeAllocSizeInBits(AllocaEltTy);
|
2009-01-08 05:42:05 +00:00
|
|
|
|
|
|
|
// If this isn't a load of the whole alloca to an integer, it may be a load
|
|
|
|
// of the first element. Just ignore the load in this case and normal SROA
|
2009-06-01 09:14:32 +00:00
|
|
|
// will handle it.
|
2009-01-08 05:42:05 +00:00
|
|
|
if (!isa<IntegerType>(LI->getType()) ||
|
2009-06-01 09:14:32 +00:00
|
|
|
TD->getTypeAllocSizeInBits(LI->getType()) != AllocaSizeBits)
|
2009-01-08 05:42:05 +00:00
|
|
|
return;
|
|
|
|
|
|
|
|
DOUT << "PROMOTING LOAD OF WHOLE ALLOCA: " << *AI << *LI;
|
|
|
|
|
|
|
|
// There are two forms here: AI could be an array or struct. Both cases
|
|
|
|
// have different ways to compute the element offset.
|
|
|
|
const StructLayout *Layout = 0;
|
|
|
|
uint64_t ArrayEltBitOffset = 0;
|
|
|
|
if (const StructType *EltSTy = dyn_cast<StructType>(AllocaEltTy)) {
|
|
|
|
Layout = TD->getStructLayout(EltSTy);
|
|
|
|
} else {
|
|
|
|
const Type *ArrayEltTy = cast<ArrayType>(AllocaEltTy)->getElementType();
|
2009-05-09 07:06:46 +00:00
|
|
|
ArrayEltBitOffset = TD->getTypeAllocSizeInBits(ArrayEltTy);
|
2009-01-08 05:42:05 +00:00
|
|
|
}
|
2009-07-22 00:24:57 +00:00
|
|
|
|
|
|
|
LLVMContext &Context = LI->getContext();
|
|
|
|
|
|
|
|
Value *ResultVal =
|
|
|
|
Context.getNullValue(Context.getIntegerType(AllocaSizeBits));
|
2009-01-08 05:42:05 +00:00
|
|
|
|
|
|
|
for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
|
|
|
|
// Load the value from the alloca. If the NewElt is an aggregate, cast
|
|
|
|
// the pointer to an integer of the same size before doing the load.
|
|
|
|
Value *SrcField = NewElts[i];
|
|
|
|
const Type *FieldTy =
|
|
|
|
cast<PointerType>(SrcField->getType())->getElementType();
|
2009-01-09 18:18:43 +00:00
|
|
|
uint64_t FieldSizeBits = TD->getTypeSizeInBits(FieldTy);
|
|
|
|
|
|
|
|
// Ignore zero sized fields like {}, they obviously contain no data.
|
|
|
|
if (FieldSizeBits == 0) continue;
|
|
|
|
|
2009-07-22 00:24:57 +00:00
|
|
|
const IntegerType *FieldIntTy = Context.getIntegerType(FieldSizeBits);
|
2009-01-08 05:42:05 +00:00
|
|
|
if (!isa<IntegerType>(FieldTy) && !FieldTy->isFloatingPoint() &&
|
|
|
|
!isa<VectorType>(FieldTy))
|
2009-07-03 19:42:02 +00:00
|
|
|
SrcField = new BitCastInst(SrcField,
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getPointerTypeUnqual(FieldIntTy),
|
2009-01-08 05:42:05 +00:00
|
|
|
"", LI);
|
|
|
|
SrcField = new LoadInst(SrcField, "sroa.load.elt", LI);
|
|
|
|
|
|
|
|
// If SrcField is a fp or vector of the right size but that isn't an
|
|
|
|
// integer type, bitcast to an integer so we can shift it.
|
|
|
|
if (SrcField->getType() != FieldIntTy)
|
|
|
|
SrcField = new BitCastInst(SrcField, FieldIntTy, "", LI);
|
|
|
|
|
|
|
|
// Zero extend the field to be the same size as the final alloca so that
|
|
|
|
// we can shift and insert it.
|
|
|
|
if (SrcField->getType() != ResultVal->getType())
|
|
|
|
SrcField = new ZExtInst(SrcField, ResultVal->getType(), "", LI);
|
|
|
|
|
|
|
|
// Determine the number of bits to shift SrcField.
|
|
|
|
uint64_t Shift;
|
|
|
|
if (Layout) // Struct case.
|
|
|
|
Shift = Layout->getElementOffsetInBits(i);
|
|
|
|
else // Array case.
|
|
|
|
Shift = i*ArrayEltBitOffset;
|
|
|
|
|
|
|
|
if (TD->isBigEndian())
|
|
|
|
Shift = AllocaSizeBits-Shift-FieldIntTy->getBitWidth();
|
|
|
|
|
|
|
|
if (Shift) {
|
2009-07-24 23:12:02 +00:00
|
|
|
Value *ShiftVal = ConstantInt::get(SrcField->getType(), Shift);
|
2009-01-08 05:42:05 +00:00
|
|
|
SrcField = BinaryOperator::CreateShl(SrcField, ShiftVal, "", LI);
|
|
|
|
}
|
|
|
|
|
|
|
|
ResultVal = BinaryOperator::CreateOr(SrcField, ResultVal, "", LI);
|
|
|
|
}
|
2009-06-01 09:14:32 +00:00
|
|
|
|
|
|
|
// Handle tail padding by truncating the result
|
|
|
|
if (TD->getTypeSizeInBits(LI->getType()) != AllocaSizeBits)
|
|
|
|
ResultVal = new TruncInst(ResultVal, LI->getType(), "", LI);
|
|
|
|
|
2009-01-08 05:42:05 +00:00
|
|
|
LI->replaceAllUsesWith(ResultVal);
|
|
|
|
LI->eraseFromParent();
|
|
|
|
}
|
|
|
|
|
2007-03-05 07:52:57 +00:00
|
|
|
|
2007-11-04 14:43:57 +00:00
|
|
|
/// HasPadding - Return true if the specified type has any structure or
|
|
|
|
/// alignment padding, false otherwise.
|
2008-06-04 08:21:45 +00:00
|
|
|
static bool HasPadding(const Type *Ty, const TargetData &TD) {
|
2007-05-30 06:11:23 +00:00
|
|
|
if (const StructType *STy = dyn_cast<StructType>(Ty)) {
|
|
|
|
const StructLayout *SL = TD.getStructLayout(STy);
|
|
|
|
unsigned PrevFieldBitOffset = 0;
|
|
|
|
for (unsigned i = 0, e = STy->getNumElements(); i != e; ++i) {
|
2007-11-04 14:43:57 +00:00
|
|
|
unsigned FieldBitOffset = SL->getElementOffsetInBits(i);
|
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
// Padding in sub-elements?
|
2008-06-04 08:21:45 +00:00
|
|
|
if (HasPadding(STy->getElementType(i), TD))
|
2007-05-30 06:11:23 +00:00
|
|
|
return true;
|
2007-11-04 14:43:57 +00:00
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
// Check to see if there is any padding between this element and the
|
|
|
|
// previous one.
|
|
|
|
if (i) {
|
2007-11-04 14:43:57 +00:00
|
|
|
unsigned PrevFieldEnd =
|
2007-05-30 06:11:23 +00:00
|
|
|
PrevFieldBitOffset+TD.getTypeSizeInBits(STy->getElementType(i-1));
|
|
|
|
if (PrevFieldEnd < FieldBitOffset)
|
|
|
|
return true;
|
|
|
|
}
|
2007-11-04 14:43:57 +00:00
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
PrevFieldBitOffset = FieldBitOffset;
|
|
|
|
}
|
2007-11-04 14:43:57 +00:00
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
// Check for tail padding.
|
|
|
|
if (unsigned EltCount = STy->getNumElements()) {
|
|
|
|
unsigned PrevFieldEnd = PrevFieldBitOffset +
|
|
|
|
TD.getTypeSizeInBits(STy->getElementType(EltCount-1));
|
2007-11-04 14:43:57 +00:00
|
|
|
if (PrevFieldEnd < SL->getSizeInBits())
|
2007-05-30 06:11:23 +00:00
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
} else if (const ArrayType *ATy = dyn_cast<ArrayType>(Ty)) {
|
2008-06-04 08:21:45 +00:00
|
|
|
return HasPadding(ATy->getElementType(), TD);
|
2007-11-04 14:43:57 +00:00
|
|
|
} else if (const VectorType *VTy = dyn_cast<VectorType>(Ty)) {
|
2008-06-04 08:21:45 +00:00
|
|
|
return HasPadding(VTy->getElementType(), TD);
|
2007-05-30 06:11:23 +00:00
|
|
|
}
|
2009-05-09 07:06:46 +00:00
|
|
|
return TD.getTypeSizeInBits(Ty) != TD.getTypeAllocSizeInBits(Ty);
|
2007-05-30 06:11:23 +00:00
|
|
|
}
|
2007-03-05 07:52:57 +00:00
|
|
|
|
2004-11-14 04:24:28 +00:00
|
|
|
/// isSafeStructAllocaToScalarRepl - Check to see if the specified allocation of
|
|
|
|
/// an aggregate can be broken down into elements. Return 0 if not, 3 if safe,
|
|
|
|
/// or 1 if safe after canonicalization has been performed.
|
2003-05-30 04:15:41 +00:00
|
|
|
///
|
2004-11-14 04:24:28 +00:00
|
|
|
int SROA::isSafeAllocaToScalarRepl(AllocationInst *AI) {
|
2003-05-30 04:15:41 +00:00
|
|
|
// Loop over the use list of the alloca. We can only transform it if all of
|
|
|
|
// the users are safe to transform.
|
2007-05-30 06:11:23 +00:00
|
|
|
AllocaInfo Info;
|
|
|
|
|
2003-05-30 04:15:41 +00:00
|
|
|
for (Value::use_iterator I = AI->use_begin(), E = AI->use_end();
|
2004-11-14 04:24:28 +00:00
|
|
|
I != E; ++I) {
|
2007-05-30 06:11:23 +00:00
|
|
|
isSafeUseOfAllocation(cast<Instruction>(*I), AI, Info);
|
|
|
|
if (Info.isUnsafe) {
|
2006-11-26 09:46:52 +00:00
|
|
|
DOUT << "Cannot transform: " << *AI << " due to user: " << **I;
|
2004-11-14 04:24:28 +00:00
|
|
|
return 0;
|
2003-05-30 04:15:41 +00:00
|
|
|
}
|
2004-11-14 04:24:28 +00:00
|
|
|
}
|
2007-05-30 06:11:23 +00:00
|
|
|
|
|
|
|
// Okay, we know all the users are promotable. If the aggregate is a memcpy
|
|
|
|
// source and destination, we have to be careful. In particular, the memcpy
|
|
|
|
// could be moving around elements that live in structure padding of the LLVM
|
|
|
|
// types, but may actually be used. In these cases, we refuse to promote the
|
|
|
|
// struct.
|
|
|
|
if (Info.isMemCpySrc && Info.isMemCpyDst &&
|
2009-01-07 06:34:28 +00:00
|
|
|
HasPadding(AI->getType()->getElementType(), *TD))
|
2007-05-30 06:11:23 +00:00
|
|
|
return 0;
|
2007-11-04 14:43:57 +00:00
|
|
|
|
2007-05-30 06:11:23 +00:00
|
|
|
// If we require cleanup, return 1, otherwise return 3.
|
2009-02-10 07:00:59 +00:00
|
|
|
return Info.needsCleanup ? 1 : 3;
|
2004-11-14 04:24:28 +00:00
|
|
|
}
|
|
|
|
|
2009-02-10 07:00:59 +00:00
|
|
|
/// CleanupGEP - GEP is used by an Alloca, which can be prompted after the GEP
|
|
|
|
/// is canonicalized here.
|
|
|
|
void SROA::CleanupGEP(GetElementPtrInst *GEPI) {
|
|
|
|
gep_type_iterator I = gep_type_begin(GEPI);
|
|
|
|
++I;
|
|
|
|
|
2009-02-10 19:28:07 +00:00
|
|
|
const ArrayType *AT = dyn_cast<ArrayType>(*I);
|
|
|
|
if (!AT)
|
|
|
|
return;
|
|
|
|
|
|
|
|
uint64_t NumElements = AT->getNumElements();
|
|
|
|
|
|
|
|
if (isa<ConstantInt>(I.getOperand()))
|
|
|
|
return;
|
|
|
|
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = GEPI->getContext();
|
|
|
|
|
2009-02-10 19:28:07 +00:00
|
|
|
if (NumElements == 1) {
|
2009-07-22 00:24:57 +00:00
|
|
|
GEPI->setOperand(2, Context.getNullValue(Type::Int32Ty));
|
2009-02-10 19:28:07 +00:00
|
|
|
return;
|
|
|
|
}
|
2009-02-10 07:00:59 +00:00
|
|
|
|
2009-02-10 19:28:07 +00:00
|
|
|
assert(NumElements == 2 && "Unhandled case!");
|
|
|
|
// All users of the GEP must be loads. At each use of the GEP, insert
|
|
|
|
// two loads of the appropriate indexed GEP and select between them.
|
2009-07-09 23:48:35 +00:00
|
|
|
Value *IsOne = new ICmpInst(GEPI, ICmpInst::ICMP_NE, I.getOperand(),
|
2009-07-22 00:24:57 +00:00
|
|
|
Context.getNullValue(I.getOperand()->getType()),
|
2009-07-09 23:48:35 +00:00
|
|
|
"isone");
|
2009-02-10 19:28:07 +00:00
|
|
|
// Insert the new GEP instructions, which are properly indexed.
|
|
|
|
SmallVector<Value*, 8> Indices(GEPI->op_begin()+1, GEPI->op_end());
|
2009-07-22 00:24:57 +00:00
|
|
|
Indices[1] = Context.getNullValue(Type::Int32Ty);
|
2009-02-10 19:28:07 +00:00
|
|
|
Value *ZeroIdx = GetElementPtrInst::Create(GEPI->getOperand(0),
|
|
|
|
Indices.begin(),
|
|
|
|
Indices.end(),
|
|
|
|
GEPI->getName()+".0", GEPI);
|
2009-07-24 23:12:02 +00:00
|
|
|
Indices[1] = ConstantInt::get(Type::Int32Ty, 1);
|
2009-02-10 19:28:07 +00:00
|
|
|
Value *OneIdx = GetElementPtrInst::Create(GEPI->getOperand(0),
|
|
|
|
Indices.begin(),
|
|
|
|
Indices.end(),
|
|
|
|
GEPI->getName()+".1", GEPI);
|
|
|
|
// Replace all loads of the variable index GEP with loads from both
|
|
|
|
// indexes and a select.
|
|
|
|
while (!GEPI->use_empty()) {
|
|
|
|
LoadInst *LI = cast<LoadInst>(GEPI->use_back());
|
|
|
|
Value *Zero = new LoadInst(ZeroIdx, LI->getName()+".0", LI);
|
|
|
|
Value *One = new LoadInst(OneIdx , LI->getName()+".1", LI);
|
|
|
|
Value *R = SelectInst::Create(IsOne, One, Zero, LI->getName(), LI);
|
|
|
|
LI->replaceAllUsesWith(R);
|
|
|
|
LI->eraseFromParent();
|
2009-02-10 07:00:59 +00:00
|
|
|
}
|
2009-02-10 19:28:07 +00:00
|
|
|
GEPI->eraseFromParent();
|
2009-02-10 07:00:59 +00:00
|
|
|
}
|
|
|
|
|
2009-02-10 19:28:07 +00:00
|
|
|
|
2009-02-10 07:00:59 +00:00
|
|
|
/// CleanupAllocaUsers - If SROA reported that it can promote the specified
|
2004-11-14 04:24:28 +00:00
|
|
|
/// allocation, but only if cleaned up, perform the cleanups required.
|
2009-02-10 07:00:59 +00:00
|
|
|
void SROA::CleanupAllocaUsers(AllocationInst *AI) {
|
2004-11-14 05:00:19 +00:00
|
|
|
// At this point, we know that the end result will be SROA'd and promoted, so
|
|
|
|
// we can insert ugly code if required so long as sroa+mem2reg will clean it
|
|
|
|
// up.
|
|
|
|
for (Value::use_iterator UI = AI->use_begin(), E = AI->use_end();
|
|
|
|
UI != E; ) {
|
2009-02-10 07:00:59 +00:00
|
|
|
User *U = *UI++;
|
|
|
|
if (GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(U))
|
|
|
|
CleanupGEP(GEPI);
|
2009-06-06 17:49:35 +00:00
|
|
|
else {
|
|
|
|
Instruction *I = cast<Instruction>(U);
|
2009-02-10 07:00:59 +00:00
|
|
|
SmallVector<DbgInfoIntrinsic *, 2> DbgInUses;
|
2009-03-18 12:48:48 +00:00
|
|
|
if (!isa<StoreInst>(I) && OnlyUsedByDbgInfoIntrinsics(I, &DbgInUses)) {
|
2009-02-10 07:00:59 +00:00
|
|
|
// Safe to remove debug info uses.
|
|
|
|
while (!DbgInUses.empty()) {
|
|
|
|
DbgInfoIntrinsic *DI = DbgInUses.back(); DbgInUses.pop_back();
|
|
|
|
DI->eraseFromParent();
|
2004-11-14 05:00:19 +00:00
|
|
|
}
|
2009-02-10 07:00:59 +00:00
|
|
|
I->eraseFromParent();
|
2004-11-14 05:00:19 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
2003-05-30 04:15:41 +00:00
|
|
|
}
|
2005-12-12 07:19:13 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
/// MergeInType - Add the 'In' type to the accumulated type (Accum) so far at
|
|
|
|
/// the offset specified by Offset (which is specified in bytes).
|
2006-04-14 21:42:41 +00:00
|
|
|
///
|
2009-01-31 02:28:54 +00:00
|
|
|
/// There are two cases we handle here:
|
|
|
|
/// 1) A union of vector types of the same size and potentially its elements.
|
2006-12-15 07:32:38 +00:00
|
|
|
/// Here we turn element accesses into insert/extract element operations.
|
2009-01-31 02:28:54 +00:00
|
|
|
/// This promotes a <4 x float> with a store of float to the third element
|
|
|
|
/// into a <4 x float> that uses insert element.
|
|
|
|
/// 2) A fully general blob of memory, which we turn into some (potentially
|
|
|
|
/// large) integer type with extract and insert operations where the loads
|
|
|
|
/// and stores would mutate the memory.
|
2009-02-03 01:30:09 +00:00
|
|
|
static void MergeInType(const Type *In, uint64_t Offset, const Type *&VecTy,
|
2009-07-03 19:42:02 +00:00
|
|
|
unsigned AllocaSize, const TargetData &TD,
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context) {
|
2009-02-03 01:30:09 +00:00
|
|
|
// If this could be contributing to a vector, analyze it.
|
|
|
|
if (VecTy != Type::VoidTy) { // either null or a vector type.
|
|
|
|
|
|
|
|
// If the In type is a vector that is the same size as the alloca, see if it
|
|
|
|
// matches the existing VecTy.
|
|
|
|
if (const VectorType *VInTy = dyn_cast<VectorType>(In)) {
|
|
|
|
if (VInTy->getBitWidth()/8 == AllocaSize && Offset == 0) {
|
|
|
|
// If we're storing/loading a vector of the right size, allow it as a
|
|
|
|
// vector. If this the first vector we see, remember the type so that
|
|
|
|
// we know the element size.
|
|
|
|
if (VecTy == 0)
|
|
|
|
VecTy = VInTy;
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
} else if (In == Type::FloatTy || In == Type::DoubleTy ||
|
|
|
|
(isa<IntegerType>(In) && In->getPrimitiveSizeInBits() >= 8 &&
|
|
|
|
isPowerOf2_32(In->getPrimitiveSizeInBits()))) {
|
|
|
|
// If we're accessing something that could be an element of a vector, see
|
|
|
|
// if the implied vector agrees with what we already have and if Offset is
|
|
|
|
// compatible with it.
|
|
|
|
unsigned EltSize = In->getPrimitiveSizeInBits()/8;
|
|
|
|
if (Offset % EltSize == 0 &&
|
|
|
|
AllocaSize % EltSize == 0 &&
|
|
|
|
(VecTy == 0 ||
|
|
|
|
cast<VectorType>(VecTy)->getElementType()
|
|
|
|
->getPrimitiveSizeInBits()/8 == EltSize)) {
|
|
|
|
if (VecTy == 0)
|
2009-07-22 00:24:57 +00:00
|
|
|
VecTy = In->getContext().getVectorType(In, AllocaSize/EltSize);
|
2009-02-03 01:30:09 +00:00
|
|
|
return;
|
|
|
|
}
|
2009-01-31 02:28:54 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-02-03 01:30:09 +00:00
|
|
|
// Otherwise, we have a case that we can't handle with an optimized vector
|
|
|
|
// form. We can still turn this into a large integer.
|
|
|
|
VecTy = Type::VoidTy;
|
2005-12-12 07:19:13 +00:00
|
|
|
}
|
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
/// CanConvertToScalar - V is a pointer. If we can convert the pointee and all
|
2009-02-03 01:30:09 +00:00
|
|
|
/// its accesses to use a to single vector type, return true, and set VecTy to
|
|
|
|
/// the new type. If we could convert the alloca into a single promotable
|
|
|
|
/// integer, return true but set VecTy to VoidTy. Further, if the use is not a
|
|
|
|
/// completely trivial use that mem2reg could promote, set IsNotTrivial. Offset
|
|
|
|
/// is the current offset from the base of the alloca being analyzed.
|
2005-12-12 07:19:13 +00:00
|
|
|
///
|
2009-02-03 18:15:05 +00:00
|
|
|
/// If we see at least one access to the value that is as a vector type, set the
|
|
|
|
/// SawVec flag.
|
|
|
|
///
|
|
|
|
bool SROA::CanConvertToScalar(Value *V, bool &IsNotTrivial, const Type *&VecTy,
|
|
|
|
bool &SawVec, uint64_t Offset,
|
2009-02-03 01:30:09 +00:00
|
|
|
unsigned AllocaSize) {
|
2005-12-12 07:19:13 +00:00
|
|
|
for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI!=E; ++UI) {
|
|
|
|
Instruction *User = cast<Instruction>(*UI);
|
|
|
|
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(User)) {
|
2009-01-31 02:28:54 +00:00
|
|
|
// Don't break volatile loads.
|
2009-01-28 20:16:43 +00:00
|
|
|
if (LI->isVolatile())
|
2009-01-31 02:28:54 +00:00
|
|
|
return false;
|
2009-07-22 00:24:57 +00:00
|
|
|
MergeInType(LI->getType(), Offset, VecTy,
|
|
|
|
AllocaSize, *TD, V->getContext());
|
2009-02-03 18:15:05 +00:00
|
|
|
SawVec |= isa<VectorType>(LI->getType());
|
2009-01-07 06:39:58 +00:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (StoreInst *SI = dyn_cast<StoreInst>(User)) {
|
2007-01-21 00:29:26 +00:00
|
|
|
// Storing the pointer, not into the value?
|
2009-01-28 20:16:43 +00:00
|
|
|
if (SI->getOperand(0) == V || SI->isVolatile()) return 0;
|
2009-07-03 19:42:02 +00:00
|
|
|
MergeInType(SI->getOperand(0)->getType(), Offset,
|
2009-07-22 00:24:57 +00:00
|
|
|
VecTy, AllocaSize, *TD, V->getContext());
|
2009-02-03 18:15:05 +00:00
|
|
|
SawVec |= isa<VectorType>(SI->getOperand(0)->getType());
|
2009-01-07 06:39:58 +00:00
|
|
|
continue;
|
|
|
|
}
|
2009-01-31 02:28:54 +00:00
|
|
|
|
|
|
|
if (BitCastInst *BCI = dyn_cast<BitCastInst>(User)) {
|
2009-02-03 18:15:05 +00:00
|
|
|
if (!CanConvertToScalar(BCI, IsNotTrivial, VecTy, SawVec, Offset,
|
|
|
|
AllocaSize))
|
2009-01-31 02:28:54 +00:00
|
|
|
return false;
|
2005-12-12 07:19:13 +00:00
|
|
|
IsNotTrivial = true;
|
2009-01-07 06:39:58 +00:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(User)) {
|
2009-01-31 02:28:54 +00:00
|
|
|
// If this is a GEP with a variable indices, we can't handle it.
|
|
|
|
if (!GEP->hasAllConstantIndices())
|
|
|
|
return false;
|
2009-01-07 06:39:58 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// Compute the offset that this GEP adds to the pointer.
|
|
|
|
SmallVector<Value*, 8> Indices(GEP->op_begin()+1, GEP->op_end());
|
|
|
|
uint64_t GEPOffset = TD->getIndexedOffset(GEP->getOperand(0)->getType(),
|
|
|
|
&Indices[0], Indices.size());
|
|
|
|
// See if all uses can be converted.
|
2009-02-03 18:15:05 +00:00
|
|
|
if (!CanConvertToScalar(GEP, IsNotTrivial, VecTy, SawVec,Offset+GEPOffset,
|
2009-02-03 01:30:09 +00:00
|
|
|
AllocaSize))
|
2009-01-31 02:28:54 +00:00
|
|
|
return false;
|
|
|
|
IsNotTrivial = true;
|
|
|
|
continue;
|
2005-12-12 07:19:13 +00:00
|
|
|
}
|
2009-03-08 03:37:16 +00:00
|
|
|
|
2009-02-03 02:01:43 +00:00
|
|
|
// If this is a constant sized memset of a constant value (e.g. 0) we can
|
|
|
|
// handle it.
|
2009-03-08 03:37:16 +00:00
|
|
|
if (MemSetInst *MSI = dyn_cast<MemSetInst>(User)) {
|
|
|
|
// Store of constant value and constant size.
|
|
|
|
if (isa<ConstantInt>(MSI->getValue()) &&
|
|
|
|
isa<ConstantInt>(MSI->getLength())) {
|
|
|
|
IsNotTrivial = true;
|
|
|
|
continue;
|
|
|
|
}
|
2009-02-03 02:01:43 +00:00
|
|
|
}
|
2009-03-08 04:04:21 +00:00
|
|
|
|
|
|
|
// If this is a memcpy or memmove into or out of the whole allocation, we
|
|
|
|
// can handle it like a load or store of the scalar type.
|
|
|
|
if (MemTransferInst *MTI = dyn_cast<MemTransferInst>(User)) {
|
|
|
|
if (ConstantInt *Len = dyn_cast<ConstantInt>(MTI->getLength()))
|
|
|
|
if (Len->getZExtValue() == AllocaSize && Offset == 0) {
|
|
|
|
IsNotTrivial = true;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
2009-03-08 03:59:00 +00:00
|
|
|
|
2009-03-06 07:03:54 +00:00
|
|
|
// Ignore dbg intrinsic.
|
|
|
|
if (isa<DbgInfoIntrinsic>(User))
|
|
|
|
continue;
|
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// Otherwise, we cannot handle this!
|
|
|
|
return false;
|
2005-12-12 07:19:13 +00:00
|
|
|
}
|
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
return true;
|
2005-12-12 07:19:13 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
/// ConvertUsesToScalar - Convert all of the users of Ptr to use the new alloca
|
2006-04-14 21:42:41 +00:00
|
|
|
/// directly. This happens when we are converting an "integer union" to a
|
|
|
|
/// single integer scalar, or when we are converting a "vector union" to a
|
|
|
|
/// vector with insert/extractelement instructions.
|
|
|
|
///
|
|
|
|
/// Offset is an offset from the original alloca, in bits that need to be
|
|
|
|
/// shifted to the right. By the end of this, there should be no uses of Ptr.
|
2009-01-31 02:28:54 +00:00
|
|
|
void SROA::ConvertUsesToScalar(Value *Ptr, AllocaInst *NewAI, uint64_t Offset) {
|
2005-12-12 07:19:13 +00:00
|
|
|
while (!Ptr->use_empty()) {
|
|
|
|
Instruction *User = cast<Instruction>(Ptr->use_back());
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-07 06:39:58 +00:00
|
|
|
if (BitCastInst *CI = dyn_cast<BitCastInst>(User)) {
|
2008-01-30 00:39:15 +00:00
|
|
|
ConvertUsesToScalar(CI, NewAI, Offset);
|
2005-12-12 07:19:13 +00:00
|
|
|
CI->eraseFromParent();
|
2009-01-07 06:39:58 +00:00
|
|
|
continue;
|
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-07 06:39:58 +00:00
|
|
|
if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(User)) {
|
2009-01-31 02:28:54 +00:00
|
|
|
// Compute the offset that this GEP adds to the pointer.
|
|
|
|
SmallVector<Value*, 8> Indices(GEP->op_begin()+1, GEP->op_end());
|
|
|
|
uint64_t GEPOffset = TD->getIndexedOffset(GEP->getOperand(0)->getType(),
|
|
|
|
&Indices[0], Indices.size());
|
|
|
|
ConvertUsesToScalar(GEP, NewAI, Offset+GEPOffset*8);
|
2005-12-12 07:19:13 +00:00
|
|
|
GEP->eraseFromParent();
|
2009-01-07 06:39:58 +00:00
|
|
|
continue;
|
2005-12-12 07:19:13 +00:00
|
|
|
}
|
2009-02-03 02:01:43 +00:00
|
|
|
|
2009-02-03 19:45:44 +00:00
|
|
|
IRBuilder<> Builder(User->getParent(), User);
|
|
|
|
|
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(User)) {
|
2009-02-03 21:01:03 +00:00
|
|
|
// The load is a bit extract from NewAI shifted right by Offset bits.
|
|
|
|
Value *LoadedVal = Builder.CreateLoad(NewAI, "tmp");
|
|
|
|
Value *NewLoadVal
|
|
|
|
= ConvertScalar_ExtractValue(LoadedVal, LI->getType(), Offset, Builder);
|
|
|
|
LI->replaceAllUsesWith(NewLoadVal);
|
2009-02-03 19:45:44 +00:00
|
|
|
LI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (StoreInst *SI = dyn_cast<StoreInst>(User)) {
|
|
|
|
assert(SI->getOperand(0) != Ptr && "Consistency error!");
|
2009-07-25 04:41:11 +00:00
|
|
|
// FIXME: Remove once builder has Twine API.
|
|
|
|
Value *Old = Builder.CreateLoad(NewAI, (NewAI->getName()+".in").str().c_str());
|
2009-02-03 19:45:44 +00:00
|
|
|
Value *New = ConvertScalar_InsertValue(SI->getOperand(0), Old, Offset,
|
|
|
|
Builder);
|
|
|
|
Builder.CreateStore(New, NewAI);
|
|
|
|
SI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2009-02-03 02:01:43 +00:00
|
|
|
// If this is a constant sized memset of a constant value (e.g. 0) we can
|
|
|
|
// transform it into a store of the expanded constant value.
|
|
|
|
if (MemSetInst *MSI = dyn_cast<MemSetInst>(User)) {
|
|
|
|
assert(MSI->getRawDest() == Ptr && "Consistency error!");
|
|
|
|
unsigned NumBytes = cast<ConstantInt>(MSI->getLength())->getZExtValue();
|
2009-04-21 16:52:12 +00:00
|
|
|
if (NumBytes != 0) {
|
|
|
|
unsigned Val = cast<ConstantInt>(MSI->getValue())->getZExtValue();
|
|
|
|
|
|
|
|
// Compute the value replicated the right number of times.
|
|
|
|
APInt APVal(NumBytes*8, Val);
|
2009-02-03 02:01:43 +00:00
|
|
|
|
2009-04-21 16:52:12 +00:00
|
|
|
// Splat the value if non-zero.
|
|
|
|
if (Val)
|
|
|
|
for (unsigned i = 1; i != NumBytes; ++i)
|
|
|
|
APVal |= APVal << 8;
|
|
|
|
|
2009-07-25 04:41:11 +00:00
|
|
|
// FIXME: Remove once builder has Twine API.
|
|
|
|
Value *Old = Builder.CreateLoad(NewAI, (NewAI->getName()+".in").str().c_str());
|
2009-07-22 00:24:57 +00:00
|
|
|
Value *New = ConvertScalar_InsertValue(
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(User->getContext(), APVal),
|
2009-07-03 19:42:02 +00:00
|
|
|
Old, Offset, Builder);
|
2009-04-21 16:52:12 +00:00
|
|
|
Builder.CreateStore(New, NewAI);
|
|
|
|
}
|
2009-02-03 02:01:43 +00:00
|
|
|
MSI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
2009-03-08 04:04:21 +00:00
|
|
|
|
|
|
|
// If this is a memcpy or memmove into or out of the whole allocation, we
|
|
|
|
// can handle it like a load or store of the scalar type.
|
|
|
|
if (MemTransferInst *MTI = dyn_cast<MemTransferInst>(User)) {
|
|
|
|
assert(Offset == 0 && "must be store to start of alloca");
|
|
|
|
|
|
|
|
// If the source and destination are both to the same alloca, then this is
|
|
|
|
// a noop copy-to-self, just delete it. Otherwise, emit a load and store
|
|
|
|
// as appropriate.
|
|
|
|
AllocaInst *OrigAI = cast<AllocaInst>(Ptr->getUnderlyingObject());
|
|
|
|
|
|
|
|
if (MTI->getSource()->getUnderlyingObject() != OrigAI) {
|
|
|
|
// Dest must be OrigAI, change this to be a load from the original
|
|
|
|
// pointer (bitcasted), then a store to our new alloca.
|
|
|
|
assert(MTI->getRawDest() == Ptr && "Neither use is of pointer?");
|
|
|
|
Value *SrcPtr = MTI->getSource();
|
|
|
|
SrcPtr = Builder.CreateBitCast(SrcPtr, NewAI->getType());
|
|
|
|
|
|
|
|
LoadInst *SrcVal = Builder.CreateLoad(SrcPtr, "srcval");
|
|
|
|
SrcVal->setAlignment(MTI->getAlignment());
|
|
|
|
Builder.CreateStore(SrcVal, NewAI);
|
|
|
|
} else if (MTI->getDest()->getUnderlyingObject() != OrigAI) {
|
|
|
|
// Src must be OrigAI, change this to be a load from NewAI then a store
|
|
|
|
// through the original dest pointer (bitcasted).
|
|
|
|
assert(MTI->getRawSource() == Ptr && "Neither use is of pointer?");
|
|
|
|
LoadInst *SrcVal = Builder.CreateLoad(NewAI, "srcval");
|
|
|
|
|
|
|
|
Value *DstPtr = Builder.CreateBitCast(MTI->getDest(), NewAI->getType());
|
|
|
|
StoreInst *NewStore = Builder.CreateStore(SrcVal, DstPtr);
|
|
|
|
NewStore->setAlignment(MTI->getAlignment());
|
|
|
|
} else {
|
|
|
|
// Noop transfer. Src == Dst
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
MTI->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
2009-03-08 03:59:00 +00:00
|
|
|
|
2009-03-06 07:03:54 +00:00
|
|
|
// If user is a dbg info intrinsic then it is safe to remove it.
|
|
|
|
if (isa<DbgInfoIntrinsic>(User)) {
|
|
|
|
User->eraseFromParent();
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2009-07-14 16:55:14 +00:00
|
|
|
llvm_unreachable("Unsupported operation!");
|
2005-12-12 07:19:13 +00:00
|
|
|
}
|
|
|
|
}
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
|
2009-02-03 21:01:03 +00:00
|
|
|
/// ConvertScalar_ExtractValue - Extract a value of type ToType from an integer
|
|
|
|
/// or vector value FromVal, extracting the bits from the offset specified by
|
|
|
|
/// Offset. This returns the value, which is of type ToType.
|
|
|
|
///
|
|
|
|
/// This happens when we are converting an "integer union" to a single
|
2009-02-02 10:06:20 +00:00
|
|
|
/// integer scalar, or when we are converting a "vector union" to a vector with
|
|
|
|
/// insert/extractelement instructions.
|
2008-02-29 07:03:13 +00:00
|
|
|
///
|
2009-02-02 10:06:20 +00:00
|
|
|
/// Offset is an offset from the original alloca, in bits that need to be
|
2009-02-03 21:01:03 +00:00
|
|
|
/// shifted to the right.
|
|
|
|
Value *SROA::ConvertScalar_ExtractValue(Value *FromVal, const Type *ToType,
|
|
|
|
uint64_t Offset, IRBuilder<> &Builder) {
|
2009-01-31 02:28:54 +00:00
|
|
|
// If the load is of the whole new alloca, no conversion is needed.
|
2009-02-03 21:01:03 +00:00
|
|
|
if (FromVal->getType() == ToType && Offset == 0)
|
|
|
|
return FromVal;
|
2008-02-29 07:12:06 +00:00
|
|
|
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = FromVal->getContext();
|
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// If the result alloca is a vector type, this is either an element
|
|
|
|
// access or a bitcast to another vector type of the same size.
|
2009-02-03 21:01:03 +00:00
|
|
|
if (const VectorType *VTy = dyn_cast<VectorType>(FromVal->getType())) {
|
|
|
|
if (isa<VectorType>(ToType))
|
|
|
|
return Builder.CreateBitCast(FromVal, ToType, "tmp");
|
2008-02-29 07:12:06 +00:00
|
|
|
|
|
|
|
// Otherwise it must be an element access.
|
|
|
|
unsigned Elt = 0;
|
|
|
|
if (Offset) {
|
2009-05-09 07:06:46 +00:00
|
|
|
unsigned EltSize = TD->getTypeAllocSizeInBits(VTy->getElementType());
|
2008-02-29 07:12:06 +00:00
|
|
|
Elt = Offset/EltSize;
|
2009-01-31 02:28:54 +00:00
|
|
|
assert(EltSize*Elt == Offset && "Invalid modulus in validity checking");
|
2008-02-29 07:03:13 +00:00
|
|
|
}
|
2009-01-31 02:28:54 +00:00
|
|
|
// Return the element extracted out of it.
|
2009-02-03 21:01:03 +00:00
|
|
|
Value *V = Builder.CreateExtractElement(FromVal,
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(Type::Int32Ty,Elt),
|
2009-02-03 19:45:44 +00:00
|
|
|
"tmp");
|
2009-02-03 21:01:03 +00:00
|
|
|
if (V->getType() != ToType)
|
|
|
|
V = Builder.CreateBitCast(V, ToType, "tmp");
|
2009-02-03 01:30:09 +00:00
|
|
|
return V;
|
2008-02-29 07:12:06 +00:00
|
|
|
}
|
2009-02-03 21:08:45 +00:00
|
|
|
|
|
|
|
// If ToType is a first class aggregate, extract out each of the pieces and
|
|
|
|
// use insertvalue's to form the FCA.
|
|
|
|
if (const StructType *ST = dyn_cast<StructType>(ToType)) {
|
|
|
|
const StructLayout &Layout = *TD->getStructLayout(ST);
|
2009-07-22 00:24:57 +00:00
|
|
|
Value *Res = Context.getUndef(ST);
|
2009-02-03 21:08:45 +00:00
|
|
|
for (unsigned i = 0, e = ST->getNumElements(); i != e; ++i) {
|
|
|
|
Value *Elt = ConvertScalar_ExtractValue(FromVal, ST->getElementType(i),
|
2009-02-06 04:34:07 +00:00
|
|
|
Offset+Layout.getElementOffsetInBits(i),
|
2009-02-03 21:08:45 +00:00
|
|
|
Builder);
|
|
|
|
Res = Builder.CreateInsertValue(Res, Elt, i, "tmp");
|
|
|
|
}
|
|
|
|
return Res;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (const ArrayType *AT = dyn_cast<ArrayType>(ToType)) {
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t EltSize = TD->getTypeAllocSizeInBits(AT->getElementType());
|
2009-07-22 00:24:57 +00:00
|
|
|
Value *Res = Context.getUndef(AT);
|
2009-02-03 21:08:45 +00:00
|
|
|
for (unsigned i = 0, e = AT->getNumElements(); i != e; ++i) {
|
|
|
|
Value *Elt = ConvertScalar_ExtractValue(FromVal, AT->getElementType(),
|
|
|
|
Offset+i*EltSize, Builder);
|
|
|
|
Res = Builder.CreateInsertValue(Res, Elt, i, "tmp");
|
|
|
|
}
|
|
|
|
return Res;
|
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// Otherwise, this must be a union that was converted to an integer value.
|
2009-02-03 21:01:03 +00:00
|
|
|
const IntegerType *NTy = cast<IntegerType>(FromVal->getType());
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2008-02-29 07:12:06 +00:00
|
|
|
// If this is a big-endian system and the load is narrower than the
|
|
|
|
// full alloca type, we need to do a shift to get the right bits.
|
|
|
|
int ShAmt = 0;
|
2009-01-07 06:34:28 +00:00
|
|
|
if (TD->isBigEndian()) {
|
2008-02-29 07:12:06 +00:00
|
|
|
// On big-endian machines, the lowest bit is stored at the bit offset
|
|
|
|
// from the pointer given by getTypeStoreSizeInBits. This matters for
|
|
|
|
// integers with a bitwidth that is not a multiple of 8.
|
2009-01-07 06:34:28 +00:00
|
|
|
ShAmt = TD->getTypeStoreSizeInBits(NTy) -
|
2009-02-03 21:01:03 +00:00
|
|
|
TD->getTypeStoreSizeInBits(ToType) - Offset;
|
2008-02-29 07:12:06 +00:00
|
|
|
} else {
|
|
|
|
ShAmt = Offset;
|
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2008-02-29 07:12:06 +00:00
|
|
|
// Note: we support negative bitwidths (with shl) which are not defined.
|
|
|
|
// We do this to support (f.e.) loads off the end of a structure where
|
|
|
|
// only some bits are used.
|
|
|
|
if (ShAmt > 0 && (unsigned)ShAmt < NTy->getBitWidth())
|
2009-07-03 19:42:02 +00:00
|
|
|
FromVal = Builder.CreateLShr(FromVal,
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(FromVal->getType(),
|
2009-02-03 21:08:45 +00:00
|
|
|
ShAmt), "tmp");
|
2008-02-29 07:12:06 +00:00
|
|
|
else if (ShAmt < 0 && (unsigned)-ShAmt < NTy->getBitWidth())
|
2009-07-03 19:42:02 +00:00
|
|
|
FromVal = Builder.CreateShl(FromVal,
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(FromVal->getType(),
|
2009-02-03 21:08:45 +00:00
|
|
|
-ShAmt), "tmp");
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2008-02-29 07:12:06 +00:00
|
|
|
// Finally, unconditionally truncate the integer to the right width.
|
2009-02-03 21:01:03 +00:00
|
|
|
unsigned LIBitWidth = TD->getTypeSizeInBits(ToType);
|
2008-02-29 07:12:06 +00:00
|
|
|
if (LIBitWidth < NTy->getBitWidth())
|
2009-07-03 19:42:02 +00:00
|
|
|
FromVal =
|
2009-07-22 00:24:57 +00:00
|
|
|
Builder.CreateTrunc(FromVal, Context.getIntegerType(LIBitWidth), "tmp");
|
2009-02-03 07:08:57 +00:00
|
|
|
else if (LIBitWidth > NTy->getBitWidth())
|
2009-07-03 19:42:02 +00:00
|
|
|
FromVal =
|
2009-07-22 00:24:57 +00:00
|
|
|
Builder.CreateZExt(FromVal, Context.getIntegerType(LIBitWidth), "tmp");
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2008-02-29 07:12:06 +00:00
|
|
|
// If the result is an integer, this is a trunc or bitcast.
|
2009-02-03 21:01:03 +00:00
|
|
|
if (isa<IntegerType>(ToType)) {
|
2008-02-29 07:12:06 +00:00
|
|
|
// Should be done.
|
2009-02-03 21:01:03 +00:00
|
|
|
} else if (ToType->isFloatingPoint() || isa<VectorType>(ToType)) {
|
2008-02-29 07:12:06 +00:00
|
|
|
// Just do a bitcast, we know the sizes match up.
|
2009-02-03 21:01:03 +00:00
|
|
|
FromVal = Builder.CreateBitCast(FromVal, ToType, "tmp");
|
2008-02-29 07:12:06 +00:00
|
|
|
} else {
|
|
|
|
// Otherwise must be a pointer.
|
2009-02-03 21:01:03 +00:00
|
|
|
FromVal = Builder.CreateIntToPtr(FromVal, ToType, "tmp");
|
2008-02-29 07:03:13 +00:00
|
|
|
}
|
2009-02-03 21:01:03 +00:00
|
|
|
assert(FromVal->getType() == ToType && "Didn't convert right?");
|
|
|
|
return FromVal;
|
2008-02-29 07:03:13 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2009-02-03 19:30:11 +00:00
|
|
|
/// ConvertScalar_InsertValue - Insert the value "SV" into the existing integer
|
|
|
|
/// or vector value "Old" at the offset specified by Offset.
|
|
|
|
///
|
|
|
|
/// This happens when we are converting an "integer union" to a
|
2008-02-29 07:03:13 +00:00
|
|
|
/// single integer scalar, or when we are converting a "vector union" to a
|
|
|
|
/// vector with insert/extractelement instructions.
|
|
|
|
///
|
|
|
|
/// Offset is an offset from the original alloca, in bits that need to be
|
2009-02-03 19:30:11 +00:00
|
|
|
/// shifted to the right.
|
|
|
|
Value *SROA::ConvertScalar_InsertValue(Value *SV, Value *Old,
|
2009-02-03 19:41:50 +00:00
|
|
|
uint64_t Offset, IRBuilder<> &Builder) {
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2008-02-29 07:03:13 +00:00
|
|
|
// Convert the stored type to the actual type, shift it left to insert
|
|
|
|
// then 'or' into place.
|
2009-02-03 19:30:11 +00:00
|
|
|
const Type *AllocaType = Old->getType();
|
2009-07-22 00:24:57 +00:00
|
|
|
LLVMContext &Context = Old->getContext();
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
if (const VectorType *VTy = dyn_cast<VectorType>(AllocaType)) {
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t VecSize = TD->getTypeAllocSizeInBits(VTy);
|
|
|
|
uint64_t ValSize = TD->getTypeAllocSizeInBits(SV->getType());
|
2009-03-08 04:17:04 +00:00
|
|
|
|
|
|
|
// Changing the whole vector with memset or with an access of a different
|
|
|
|
// vector type?
|
|
|
|
if (ValSize == VecSize)
|
|
|
|
return Builder.CreateBitCast(SV, AllocaType, "tmp");
|
|
|
|
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t EltSize = TD->getTypeAllocSizeInBits(VTy->getElementType());
|
2009-03-08 04:17:04 +00:00
|
|
|
|
|
|
|
// Must be an element insertion.
|
|
|
|
unsigned Elt = Offset/EltSize;
|
|
|
|
|
|
|
|
if (SV->getType() != VTy->getElementType())
|
|
|
|
SV = Builder.CreateBitCast(SV, VTy->getElementType(), "tmp");
|
|
|
|
|
|
|
|
SV = Builder.CreateInsertElement(Old, SV,
|
2009-07-24 23:12:02 +00:00
|
|
|
ConstantInt::get(Type::Int32Ty, Elt),
|
2009-03-08 04:17:04 +00:00
|
|
|
"tmp");
|
2009-01-31 02:28:54 +00:00
|
|
|
return SV;
|
|
|
|
}
|
2009-02-03 19:30:11 +00:00
|
|
|
|
|
|
|
// If SV is a first-class aggregate value, insert each value recursively.
|
|
|
|
if (const StructType *ST = dyn_cast<StructType>(SV->getType())) {
|
|
|
|
const StructLayout &Layout = *TD->getStructLayout(ST);
|
|
|
|
for (unsigned i = 0, e = ST->getNumElements(); i != e; ++i) {
|
2009-02-03 19:41:50 +00:00
|
|
|
Value *Elt = Builder.CreateExtractValue(SV, i, "tmp");
|
2009-02-03 19:30:11 +00:00
|
|
|
Old = ConvertScalar_InsertValue(Elt, Old,
|
2009-02-06 04:34:07 +00:00
|
|
|
Offset+Layout.getElementOffsetInBits(i),
|
2009-02-03 19:41:50 +00:00
|
|
|
Builder);
|
2009-02-03 19:30:11 +00:00
|
|
|
}
|
|
|
|
return Old;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (const ArrayType *AT = dyn_cast<ArrayType>(SV->getType())) {
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t EltSize = TD->getTypeAllocSizeInBits(AT->getElementType());
|
2009-02-03 19:30:11 +00:00
|
|
|
for (unsigned i = 0, e = AT->getNumElements(); i != e; ++i) {
|
2009-02-03 19:41:50 +00:00
|
|
|
Value *Elt = Builder.CreateExtractValue(SV, i, "tmp");
|
|
|
|
Old = ConvertScalar_InsertValue(Elt, Old, Offset+i*EltSize, Builder);
|
2009-02-03 19:30:11 +00:00
|
|
|
}
|
|
|
|
return Old;
|
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// If SV is a float, convert it to the appropriate integer type.
|
2009-02-03 19:30:11 +00:00
|
|
|
// If it is a pointer, do the same.
|
2009-01-31 02:28:54 +00:00
|
|
|
unsigned SrcWidth = TD->getTypeSizeInBits(SV->getType());
|
|
|
|
unsigned DestWidth = TD->getTypeSizeInBits(AllocaType);
|
|
|
|
unsigned SrcStoreWidth = TD->getTypeStoreSizeInBits(SV->getType());
|
|
|
|
unsigned DestStoreWidth = TD->getTypeStoreSizeInBits(AllocaType);
|
|
|
|
if (SV->getType()->isFloatingPoint() || isa<VectorType>(SV->getType()))
|
2009-07-22 00:24:57 +00:00
|
|
|
SV = Builder.CreateBitCast(SV, Context.getIntegerType(SrcWidth), "tmp");
|
2009-01-31 02:28:54 +00:00
|
|
|
else if (isa<PointerType>(SV->getType()))
|
2009-02-03 19:41:50 +00:00
|
|
|
SV = Builder.CreatePtrToInt(SV, TD->getIntPtrType(), "tmp");
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-02-03 01:30:09 +00:00
|
|
|
// Zero extend or truncate the value if needed.
|
|
|
|
if (SV->getType() != AllocaType) {
|
|
|
|
if (SV->getType()->getPrimitiveSizeInBits() <
|
|
|
|
AllocaType->getPrimitiveSizeInBits())
|
2009-02-03 19:41:50 +00:00
|
|
|
SV = Builder.CreateZExt(SV, AllocaType, "tmp");
|
2009-02-03 01:30:09 +00:00
|
|
|
else {
|
|
|
|
// Truncation may be needed if storing more than the alloca can hold
|
|
|
|
// (undefined behavior).
|
2009-02-03 19:41:50 +00:00
|
|
|
SV = Builder.CreateTrunc(SV, AllocaType, "tmp");
|
2009-02-03 01:30:09 +00:00
|
|
|
SrcWidth = DestWidth;
|
|
|
|
SrcStoreWidth = DestStoreWidth;
|
|
|
|
}
|
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// If this is a big-endian system and the store is narrower than the
|
|
|
|
// full alloca type, we need to do a shift to get the right bits.
|
|
|
|
int ShAmt = 0;
|
|
|
|
if (TD->isBigEndian()) {
|
|
|
|
// On big-endian machines, the lowest bit is stored at the bit offset
|
|
|
|
// from the pointer given by getTypeStoreSizeInBits. This matters for
|
|
|
|
// integers with a bitwidth that is not a multiple of 8.
|
|
|
|
ShAmt = DestStoreWidth - SrcStoreWidth - Offset;
|
2008-02-29 07:03:13 +00:00
|
|
|
} else {
|
2009-01-31 02:28:54 +00:00
|
|
|
ShAmt = Offset;
|
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// Note: we support negative bitwidths (with shr) which are not defined.
|
|
|
|
// We do this to support (f.e.) stores off the end of a structure where
|
|
|
|
// only some bits in the structure are set.
|
|
|
|
APInt Mask(APInt::getLowBitsSet(DestWidth, SrcWidth));
|
|
|
|
if (ShAmt > 0 && (unsigned)ShAmt < DestWidth) {
|
2009-07-24 23:12:02 +00:00
|
|
|
SV = Builder.CreateShl(SV, ConstantInt::get(SV->getType(),
|
2009-07-03 19:42:02 +00:00
|
|
|
ShAmt), "tmp");
|
2009-01-31 02:28:54 +00:00
|
|
|
Mask <<= ShAmt;
|
|
|
|
} else if (ShAmt < 0 && (unsigned)-ShAmt < DestWidth) {
|
2009-07-24 23:12:02 +00:00
|
|
|
SV = Builder.CreateLShr(SV, ConstantInt::get(SV->getType(),
|
2009-07-03 19:42:02 +00:00
|
|
|
-ShAmt), "tmp");
|
2009-02-02 09:53:14 +00:00
|
|
|
Mask = Mask.lshr(-ShAmt);
|
2009-01-31 02:28:54 +00:00
|
|
|
}
|
2009-02-02 10:06:20 +00:00
|
|
|
|
2009-01-31 02:28:54 +00:00
|
|
|
// Mask out the bits we are about to insert from the old value, and or
|
|
|
|
// in the new bits.
|
|
|
|
if (SrcWidth != DestWidth) {
|
|
|
|
assert(DestWidth > SrcWidth);
|
2009-07-24 23:12:02 +00:00
|
|
|
Old = Builder.CreateAnd(Old, ConstantInt::get(Context, ~Mask), "mask");
|
2009-02-03 19:41:50 +00:00
|
|
|
SV = Builder.CreateOr(Old, SV, "ins");
|
2008-02-29 07:03:13 +00:00
|
|
|
}
|
|
|
|
return SV;
|
|
|
|
}
|
|
|
|
|
|
|
|
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
|
|
|
|
/// PointsToConstantGlobal - Return true if V (possibly indirectly) points to
|
|
|
|
/// some part of a constant global variable. This intentionally only accepts
|
|
|
|
/// constant expressions because we don't can't rewrite arbitrary instructions.
|
|
|
|
static bool PointsToConstantGlobal(Value *V) {
|
|
|
|
if (GlobalVariable *GV = dyn_cast<GlobalVariable>(V))
|
|
|
|
return GV->isConstant();
|
|
|
|
if (ConstantExpr *CE = dyn_cast<ConstantExpr>(V))
|
|
|
|
if (CE->getOpcode() == Instruction::BitCast ||
|
|
|
|
CE->getOpcode() == Instruction::GetElementPtr)
|
|
|
|
return PointsToConstantGlobal(CE->getOperand(0));
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// isOnlyCopiedFromConstantGlobal - Recursively walk the uses of a (derived)
|
|
|
|
/// pointer to an alloca. Ignore any reads of the pointer, return false if we
|
|
|
|
/// see any stores or other unknown uses. If we see pointer arithmetic, keep
|
|
|
|
/// track of whether it moves the pointer (with isOffset) but otherwise traverse
|
|
|
|
/// the uses. If we see a memcpy/memmove that targets an unoffseted pointer to
|
|
|
|
/// the alloca, and if the source pointer is a pointer to a constant global, we
|
|
|
|
/// can optimize this.
|
|
|
|
static bool isOnlyCopiedFromConstantGlobal(Value *V, Instruction *&TheCopy,
|
|
|
|
bool isOffset) {
|
|
|
|
for (Value::use_iterator UI = V->use_begin(), E = V->use_end(); UI!=E; ++UI) {
|
2009-01-28 20:16:43 +00:00
|
|
|
if (LoadInst *LI = dyn_cast<LoadInst>(*UI))
|
|
|
|
// Ignore non-volatile loads, they are always ok.
|
|
|
|
if (!LI->isVolatile())
|
|
|
|
continue;
|
|
|
|
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
if (BitCastInst *BCI = dyn_cast<BitCastInst>(*UI)) {
|
|
|
|
// If uses of the bitcast are ok, we are ok.
|
|
|
|
if (!isOnlyCopiedFromConstantGlobal(BCI, TheCopy, isOffset))
|
|
|
|
return false;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(*UI)) {
|
|
|
|
// If the GEP has all zero indices, it doesn't offset the pointer. If it
|
|
|
|
// doesn't, it does.
|
|
|
|
if (!isOnlyCopiedFromConstantGlobal(GEP, TheCopy,
|
|
|
|
isOffset || !GEP->hasAllZeroIndices()))
|
|
|
|
return false;
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
// If this is isn't our memcpy/memmove, reject it as something we can't
|
|
|
|
// handle.
|
2009-03-08 03:37:16 +00:00
|
|
|
if (!isa<MemTransferInst>(*UI))
|
If an alloca only has two types of uses: 1) reads 2) a memcpy/memmove that
copies from a constant global, then we can change the reads to read from the
global instead of from the alloca. This eliminates the alloca and the memcpy,
and promotes secondary optimizations (because the loads are now loads from
a constant global).
This is important for a common C idiom:
void foo() {
int A[] = {1,2,3,4,5,6,7,8,9...};
... only reads of A ...
}
For some reason, people forget to mark the array static or const.
This triggers on these multisource benchmarks:
JM/ldecode: block_pos, [3 x [4 x [4 x i32]]]
FreeBench/mason: m, [18 x i32], inlined 4 times
MiBench/office-stringsearch: search_strings, [1332 x i8*]
MiBench/office-stringsearch: find_strings, [1333 x i8*]
Prolangs-C++/city: dirs, [9 x i8*], inlined 4 places
and these spec benchmarks:
177.mesa: message, [8 x [32 x i8]]
186.crafty: bias_rl45, [64 x i32]
186.crafty: diag_sq, [64 x i32]
186.crafty: empty, [9 x i8]
186.crafty: xlate, [15 x i8]
186.crafty: status, [13 x i8]
186.crafty: bdinfo, [25 x i8]
445.gobmk: routines, [16 x i8*]
458.sjeng: piece_rep, [14 x i8*]
458.sjeng: t, [13 x i32], inlined 4 places.
464.h264ref: block8x8_idx, [3 x [4 x [4 x i32]]]
464.h264ref: block_pos, [3 x [4 x [4 x i32]]]
464.h264ref: j_off_tab, [12 x i32]
This implements Transforms/ScalarRepl/memcpy-from-global.ll
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@36429 91177308-0d34-0410-b5e6-96231b3b80d8
2007-04-25 06:40:51 +00:00
|
|
|
return false;
|
|
|
|
|
|
|
|
// If we already have seen a copy, reject the second one.
|
|
|
|
if (TheCopy) return false;
|
|
|
|
|
|
|
|
// If the pointer has been offset from the start of the alloca, we can't
|
|
|
|
// safely handle this.
|
|
|
|
if (isOffset) return false;
|
|
|
|
|
|
|
|
// If the memintrinsic isn't using the alloca as the dest, reject it.
|
|
|
|
if (UI.getOperandNo() != 1) return false;
|
|
|
|
|
|
|
|
MemIntrinsic *MI = cast<MemIntrinsic>(*UI);
|
|
|
|
|
|
|
|
// If the source of the memcpy/move is not a constant global, reject it.
|
|
|
|
if (!PointsToConstantGlobal(MI->getOperand(2)))
|
|
|
|
return false;
|
|
|
|
|
|
|
|
// Otherwise, the transform is safe. Remember the copy instruction.
|
|
|
|
TheCopy = MI;
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// isOnlyCopiedFromConstantGlobal - Return true if the specified alloca is only
|
|
|
|
/// modified by a copy from a constant global. If we can prove this, we can
|
|
|
|
/// replace any uses of the alloca with uses of the global directly.
|
|
|
|
Instruction *SROA::isOnlyCopiedFromConstantGlobal(AllocationInst *AI) {
|
|
|
|
Instruction *TheCopy = 0;
|
|
|
|
if (::isOnlyCopiedFromConstantGlobal(AI, TheCopy, false))
|
|
|
|
return TheCopy;
|
|
|
|
return 0;
|
|
|
|
}
|