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"
|
2009-08-23 04:37:46 +00:00
|
|
|
#include "llvm/Support/raw_ostream.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"
|
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 {
|
2009-09-02 06:11:42 +00:00
|
|
|
struct 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>();
|
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;
|
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
/// DeadInsts - Keep track of instructions we have made dead, so that
|
|
|
|
/// we can remove them after we are done working.
|
|
|
|
SmallVector<Value*, 32> DeadInsts;
|
|
|
|
|
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;
|
|
|
|
|
|
|
|
/// 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()
|
2010-01-21 23:05:53 +00:00
|
|
|
: isUnsafe(false), isMemCpySrc(false), isMemCpyDst(false) {}
|
2007-05-30 06:11:23 +00:00
|
|
|
};
|
|
|
|
|
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; }
|
|
|
|
|
2010-01-21 23:05:53 +00:00
|
|
|
bool isSafeAllocaToScalarRepl(AllocaInst *AI);
|
2007-05-30 06:11:23 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
void isSafeForScalarRepl(Instruction *I, AllocaInst *AI, uint64_t Offset,
|
2009-12-21 18:39:47 +00:00
|
|
|
AllocaInfo &Info);
|
2009-12-18 20:14:40 +00:00
|
|
|
void isSafeGEP(GetElementPtrInst *GEPI, AllocaInst *AI, uint64_t &Offset,
|
2009-12-21 18:39:47 +00:00
|
|
|
AllocaInfo &Info);
|
|
|
|
void isSafeMemAccess(AllocaInst *AI, uint64_t Offset, uint64_t MemSize,
|
|
|
|
const Type *MemOpType, bool isStore, AllocaInfo &Info);
|
2009-12-18 20:14:40 +00:00
|
|
|
bool TypeHasComponent(const Type *T, uint64_t Offset, uint64_t Size);
|
2009-12-19 06:53:17 +00:00
|
|
|
uint64_t FindElementAndOffset(const Type *&T, uint64_t &Offset,
|
|
|
|
const Type *&IdxTy);
|
2007-05-30 06:11:23 +00:00
|
|
|
|
2009-10-23 21:09:37 +00:00
|
|
|
void DoScalarReplacement(AllocaInst *AI,
|
|
|
|
std::vector<AllocaInst*> &WorkList);
|
2009-12-18 20:14:40 +00:00
|
|
|
void DeleteDeadInstructions();
|
2009-10-23 21:09:37 +00:00
|
|
|
AllocaInst *AddNewAlloca(Function &F, const Type *Ty, AllocaInst *Base);
|
2005-12-12 07:19:13 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
void RewriteForScalarRepl(Instruction *I, AllocaInst *AI, uint64_t Offset,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
|
|
|
void RewriteBitCast(BitCastInst *BC, AllocaInst *AI, uint64_t Offset,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
|
|
|
void RewriteGEP(GetElementPtrInst *GEPI, AllocaInst *AI, uint64_t Offset,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
|
|
|
void RewriteMemIntrinUserOfAlloca(MemIntrinsic *MI, Instruction *Inst,
|
2009-10-23 21:09:37 +00:00
|
|
|
AllocaInst *AI,
|
2009-01-07 07:18:45 +00:00
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
2009-10-23 21:09:37 +00:00
|
|
|
void RewriteStoreUserOfWholeAlloca(StoreInst *SI, AllocaInst *AI,
|
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
|
|
|
SmallVector<AllocaInst*, 32> &NewElts);
|
2009-10-23 21:09:37 +00:00
|
|
|
void RewriteLoadUserOfWholeAlloca(LoadInst *LI, AllocaInst *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);
|
2009-10-23 21:09:37 +00:00
|
|
|
static Instruction *isOnlyCopiedFromConstantGlobal(AllocaInst *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-08-19 18:22:18 +00:00
|
|
|
TD = getAnalysisIfAvailable<TargetData>();
|
|
|
|
|
2003-09-12 15:36:03 +00:00
|
|
|
bool Changed = performPromotion(F);
|
2009-08-19 18:22:18 +00:00
|
|
|
|
|
|
|
// FIXME: ScalarRepl currently depends on TargetData more than it
|
|
|
|
// theoretically needs to. It should be refactored in order to support
|
|
|
|
// target-independent IR. Until this is done, just skip the actual
|
|
|
|
// scalar-replacement portion of this pass.
|
|
|
|
if (!TD) return Changed;
|
|
|
|
|
2003-09-12 15:36:03 +00:00
|
|
|
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-11-23 03:50:44 +00:00
|
|
|
PromoteMemToReg(Allocas, DT, DF);
|
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) {
|
2009-10-23 21:09:37 +00:00
|
|
|
std::vector<AllocaInst*> WorkList;
|
2003-05-27 15:45:27 +00:00
|
|
|
|
|
|
|
// 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)
|
2009-10-23 21:09:37 +00:00
|
|
|
if (AllocaInst *A = dyn_cast<AllocaInst>(I))
|
2003-05-27 15:45:27 +00:00
|
|
|
WorkList.push_back(A);
|
|
|
|
|
|
|
|
// Process the worklist
|
|
|
|
bool Changed = false;
|
|
|
|
while (!WorkList.empty()) {
|
2009-10-23 21:09:37 +00:00
|
|
|
AllocaInst *AI = WorkList.back();
|
2003-05-27 15:45:27 +00:00
|
|
|
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)) {
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "Found alloca equal to global: " << *AI << '\n');
|
|
|
|
DEBUG(dbgs() << " memcpy = " << *TheCopy << '\n');
|
2009-02-03 01:30:09 +00:00
|
|
|
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
|
|
|
|
2009-08-17 05:37:31 +00:00
|
|
|
// Do not promote [0 x %struct].
|
|
|
|
if (AllocaSize == 0) continue;
|
|
|
|
|
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-08-17 05:37:31 +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.
|
2010-01-21 23:05:53 +00:00
|
|
|
if (isSafeAllocaToScalarRepl(AI)) {
|
2007-04-25 05:02:56 +00:00
|
|
|
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) {
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "CONVERT TO VECTOR: " << *AI << "\n TYPE = "
|
2009-08-23 04:37:46 +00:00
|
|
|
<< *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 {
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "CONVERT TO SCALAR INTEGER: " << *AI << "\n");
|
2009-02-03 01:30:09 +00:00
|
|
|
|
|
|
|
// Create and insert the integer alloca.
|
2009-08-13 21:58:54 +00:00
|
|
|
const Type *NewTy = IntegerType::get(AI->getContext(), 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.
|
2009-10-23 21:09:37 +00:00
|
|
|
void SROA::DoScalarReplacement(AllocaInst *AI,
|
|
|
|
std::vector<AllocaInst*> &WorkList) {
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "Found inst to SROA: " << *AI << '\n');
|
2007-04-25 05:02:56 +00:00
|
|
|
SmallVector<AllocaInst*, 32> ElementAllocas;
|
|
|
|
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(),
|
2009-07-30 17:37:43 +00:00
|
|
|
AI->getName() + "." + Twine(i), AI);
|
2007-04-25 05:02:56 +00:00
|
|
|
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(),
|
2009-07-30 17:37:43 +00:00
|
|
|
AI->getName() + "." + Twine(i), AI);
|
2007-04-25 05:02:56 +00:00
|
|
|
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
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
// Now that we have created the new alloca instructions, rewrite all the
|
|
|
|
// uses of the old alloca.
|
|
|
|
RewriteForScalarRepl(AI, AI, 0, ElementAllocas);
|
2009-12-14 05:11:02 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
// Now erase any instructions that were made dead while rewriting the alloca.
|
|
|
|
DeleteDeadInstructions();
|
2009-12-17 18:34:24 +00:00
|
|
|
AI->eraseFromParent();
|
2009-12-18 20:14:40 +00:00
|
|
|
|
2007-04-25 05:02:56 +00:00
|
|
|
NumReplaced++;
|
2003-05-27 15:45:27 +00:00
|
|
|
}
|
2009-12-14 05:11:02 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
/// DeleteDeadInstructions - Erase instructions on the DeadInstrs list,
|
|
|
|
/// recursively including all their operands that become trivially dead.
|
|
|
|
void SROA::DeleteDeadInstructions() {
|
|
|
|
while (!DeadInsts.empty()) {
|
|
|
|
Instruction *I = cast<Instruction>(DeadInsts.pop_back_val());
|
|
|
|
|
|
|
|
for (User::op_iterator OI = I->op_begin(), E = I->op_end(); OI != E; ++OI)
|
|
|
|
if (Instruction *U = dyn_cast<Instruction>(*OI)) {
|
|
|
|
// Zero out the operand and see if it becomes trivially dead.
|
|
|
|
// (But, don't add allocas to the dead instruction list -- they are
|
|
|
|
// already on the worklist and will be deleted separately.)
|
|
|
|
*OI = 0;
|
|
|
|
if (isInstructionTriviallyDead(U) && !isa<AllocaInst>(U))
|
|
|
|
DeadInsts.push_back(U);
|
2009-12-14 05:11:02 +00:00
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
|
|
|
|
I->eraseFromParent();
|
2009-12-14 05:11:02 +00:00
|
|
|
}
|
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
|
|
|
|
/// isSafeForScalarRepl - Check if instruction I is a safe use with regard to
|
|
|
|
/// performing scalar replacement of alloca AI. The results are flagged in
|
2009-12-21 18:39:47 +00:00
|
|
|
/// the Info parameter. Offset indicates the position within AI that is
|
|
|
|
/// referenced by this instruction.
|
2009-12-18 20:14:40 +00:00
|
|
|
void SROA::isSafeForScalarRepl(Instruction *I, AllocaInst *AI, uint64_t Offset,
|
2009-12-21 18:39:47 +00:00
|
|
|
AllocaInfo &Info) {
|
2009-12-18 20:14:40 +00:00
|
|
|
for (Value::use_iterator UI = I->use_begin(), E = I->use_end(); UI!=E; ++UI) {
|
|
|
|
Instruction *User = cast<Instruction>(*UI);
|
2009-12-16 20:09:53 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
if (BitCastInst *BC = dyn_cast<BitCastInst>(User)) {
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeForScalarRepl(BC, AI, Offset, Info);
|
2009-12-18 20:14:40 +00:00
|
|
|
} else if (GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(User)) {
|
|
|
|
uint64_t GEPOffset = Offset;
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeGEP(GEPI, AI, GEPOffset, Info);
|
2009-12-18 20:14:40 +00:00
|
|
|
if (!Info.isUnsafe)
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeForScalarRepl(GEPI, AI, GEPOffset, Info);
|
2009-12-18 20:14:40 +00:00
|
|
|
} else if (MemIntrinsic *MI = dyn_cast<MemIntrinsic>(UI)) {
|
|
|
|
ConstantInt *Length = dyn_cast<ConstantInt>(MI->getLength());
|
|
|
|
if (Length)
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeMemAccess(AI, Offset, Length->getZExtValue(), 0,
|
2009-12-18 20:14:40 +00:00
|
|
|
UI.getOperandNo() == 1, Info);
|
|
|
|
else
|
|
|
|
MarkUnsafe(Info);
|
|
|
|
} else if (LoadInst *LI = dyn_cast<LoadInst>(User)) {
|
|
|
|
if (!LI->isVolatile()) {
|
|
|
|
const Type *LIType = LI->getType();
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeMemAccess(AI, Offset, TD->getTypeAllocSize(LIType),
|
2009-12-18 20:14:40 +00:00
|
|
|
LIType, false, Info);
|
|
|
|
} else
|
|
|
|
MarkUnsafe(Info);
|
|
|
|
} else if (StoreInst *SI = dyn_cast<StoreInst>(User)) {
|
|
|
|
// Store is ok if storing INTO the pointer, not storing the pointer
|
|
|
|
if (!SI->isVolatile() && SI->getOperand(0) != I) {
|
|
|
|
const Type *SIType = SI->getOperand(0)->getType();
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeMemAccess(AI, Offset, TD->getTypeAllocSize(SIType),
|
2009-12-18 20:14:40 +00:00
|
|
|
SIType, true, Info);
|
|
|
|
} else
|
|
|
|
MarkUnsafe(Info);
|
|
|
|
} else {
|
|
|
|
DEBUG(errs() << " Transformation preventing inst: " << *User << '\n');
|
|
|
|
MarkUnsafe(Info);
|
|
|
|
}
|
|
|
|
if (Info.isUnsafe) return;
|
2009-12-17 18:34:24 +00:00
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
}
|
2009-12-16 10:56:17 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
/// isSafeGEP - Check if a GEP instruction can be handled for scalar
|
|
|
|
/// replacement. It is safe when all the indices are constant, in-bounds
|
|
|
|
/// references, and when the resulting offset corresponds to an element within
|
|
|
|
/// the alloca type. The results are flagged in the Info parameter. Upon
|
2009-12-21 18:39:47 +00:00
|
|
|
/// return, Offset is adjusted as specified by the GEP indices.
|
2009-12-18 20:14:40 +00:00
|
|
|
void SROA::isSafeGEP(GetElementPtrInst *GEPI, AllocaInst *AI,
|
2009-12-21 18:39:47 +00:00
|
|
|
uint64_t &Offset, AllocaInfo &Info) {
|
2009-12-18 20:14:40 +00:00
|
|
|
gep_type_iterator GEPIt = gep_type_begin(GEPI), E = gep_type_end(GEPI);
|
|
|
|
if (GEPIt == E)
|
|
|
|
return;
|
|
|
|
|
2008-08-23 05:21:06 +00:00
|
|
|
// Walk through the GEP type indices, checking the types that this indexes
|
|
|
|
// into.
|
2009-12-18 20:14:40 +00:00
|
|
|
for (; GEPIt != E; ++GEPIt) {
|
2008-08-23 05:21:06 +00:00
|
|
|
// Ignore struct elements, no extra checking needed for these.
|
2009-12-18 20:14:40 +00:00
|
|
|
if (isa<StructType>(*GEPIt))
|
2008-08-23 05:21:06 +00:00
|
|
|
continue;
|
2008-10-06 16:23:31 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
ConstantInt *IdxVal = dyn_cast<ConstantInt>(GEPIt.getOperand());
|
|
|
|
if (!IdxVal)
|
|
|
|
return MarkUnsafe(Info);
|
2008-08-23 05:21:06 +00:00
|
|
|
}
|
2009-12-16 20:09:53 +00:00
|
|
|
|
2009-12-22 06:57:14 +00:00
|
|
|
// Compute the offset due to this GEP and check if the alloca has a
|
|
|
|
// component element at that offset.
|
2009-12-21 18:39:47 +00:00
|
|
|
SmallVector<Value*, 8> Indices(GEPI->op_begin() + 1, GEPI->op_end());
|
|
|
|
Offset += TD->getIndexedOffset(GEPI->getPointerOperandType(),
|
|
|
|
&Indices[0], Indices.size());
|
2009-12-18 20:14:40 +00:00
|
|
|
if (!TypeHasComponent(AI->getAllocatedType(), Offset, 0))
|
|
|
|
MarkUnsafe(Info);
|
2009-12-16 20:09:53 +00:00
|
|
|
}
|
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
/// isSafeMemAccess - Check if a load/store/memcpy operates on the entire AI
|
|
|
|
/// alloca or has an offset and size that corresponds to a component element
|
|
|
|
/// within it. The offset checked here may have been formed from a GEP with a
|
|
|
|
/// pointer bitcasted to a different type.
|
2009-12-21 18:39:47 +00:00
|
|
|
void SROA::isSafeMemAccess(AllocaInst *AI, uint64_t Offset, uint64_t MemSize,
|
2009-12-18 20:14:40 +00:00
|
|
|
const Type *MemOpType, bool isStore,
|
|
|
|
AllocaInfo &Info) {
|
|
|
|
// Check if this is a load/store of the entire alloca.
|
2009-12-21 18:39:47 +00:00
|
|
|
if (Offset == 0 && MemSize == TD->getTypeAllocSize(AI->getAllocatedType())) {
|
2009-12-18 20:14:40 +00:00
|
|
|
bool UsesAggregateType = (MemOpType == AI->getAllocatedType());
|
|
|
|
// This is safe for MemIntrinsics (where MemOpType is 0), integer types
|
|
|
|
// (which are essentially the same as the MemIntrinsics, especially with
|
|
|
|
// regard to copying padding between elements), or references using the
|
|
|
|
// aggregate type of the alloca.
|
|
|
|
if (!MemOpType || isa<IntegerType>(MemOpType) || UsesAggregateType) {
|
|
|
|
if (!UsesAggregateType) {
|
|
|
|
if (isStore)
|
|
|
|
Info.isMemCpyDst = true;
|
|
|
|
else
|
|
|
|
Info.isMemCpySrc = true;
|
2009-12-17 18:34:24 +00:00
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
// Check if the offset/size correspond to a component within the alloca type.
|
|
|
|
const Type *T = AI->getAllocatedType();
|
2009-12-21 18:39:47 +00:00
|
|
|
if (TypeHasComponent(T, Offset, MemSize))
|
2009-12-18 20:14:40 +00:00
|
|
|
return;
|
2009-12-16 20:09:53 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
return MarkUnsafe(Info);
|
|
|
|
}
|
|
|
|
|
|
|
|
/// TypeHasComponent - Return true if T has a component type with the
|
|
|
|
/// specified offset and size. If Size is zero, do not check the size.
|
|
|
|
bool SROA::TypeHasComponent(const Type *T, uint64_t Offset, uint64_t Size) {
|
|
|
|
const Type *EltTy;
|
|
|
|
uint64_t EltSize;
|
|
|
|
if (const StructType *ST = dyn_cast<StructType>(T)) {
|
|
|
|
const StructLayout *Layout = TD->getStructLayout(ST);
|
|
|
|
unsigned EltIdx = Layout->getElementContainingOffset(Offset);
|
|
|
|
EltTy = ST->getContainedType(EltIdx);
|
|
|
|
EltSize = TD->getTypeAllocSize(EltTy);
|
|
|
|
Offset -= Layout->getElementOffset(EltIdx);
|
|
|
|
} else if (const ArrayType *AT = dyn_cast<ArrayType>(T)) {
|
|
|
|
EltTy = AT->getElementType();
|
|
|
|
EltSize = TD->getTypeAllocSize(EltTy);
|
2009-12-22 06:57:14 +00:00
|
|
|
if (Offset >= AT->getNumElements() * EltSize)
|
|
|
|
return false;
|
2009-12-18 20:14:40 +00:00
|
|
|
Offset %= EltSize;
|
|
|
|
} else {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
if (Offset == 0 && (Size == 0 || EltSize == Size))
|
|
|
|
return true;
|
|
|
|
// Check if the component spans multiple elements.
|
|
|
|
if (Offset + Size > EltSize)
|
|
|
|
return false;
|
|
|
|
return TypeHasComponent(EltTy, Offset, Size);
|
|
|
|
}
|
|
|
|
|
|
|
|
/// RewriteForScalarRepl - Alloca AI is being split into NewElts, so rewrite
|
|
|
|
/// the instruction I, which references it, to use the separate elements.
|
|
|
|
/// Offset indicates the position within AI that is referenced by this
|
|
|
|
/// instruction.
|
|
|
|
void SROA::RewriteForScalarRepl(Instruction *I, AllocaInst *AI, uint64_t Offset,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
|
|
|
for (Value::use_iterator UI = I->use_begin(), E = I->use_end(); UI!=E; ++UI) {
|
|
|
|
Instruction *User = cast<Instruction>(*UI);
|
|
|
|
|
|
|
|
if (BitCastInst *BC = dyn_cast<BitCastInst>(User)) {
|
|
|
|
RewriteBitCast(BC, AI, Offset, NewElts);
|
|
|
|
} else if (GetElementPtrInst *GEPI = dyn_cast<GetElementPtrInst>(User)) {
|
|
|
|
RewriteGEP(GEPI, AI, Offset, NewElts);
|
|
|
|
} else if (MemIntrinsic *MI = dyn_cast<MemIntrinsic>(User)) {
|
|
|
|
ConstantInt *Length = dyn_cast<ConstantInt>(MI->getLength());
|
|
|
|
uint64_t MemSize = Length->getZExtValue();
|
|
|
|
if (Offset == 0 &&
|
|
|
|
MemSize == TD->getTypeAllocSize(AI->getAllocatedType()))
|
|
|
|
RewriteMemIntrinUserOfAlloca(MI, I, AI, NewElts);
|
2009-12-19 06:53:17 +00:00
|
|
|
// Otherwise the intrinsic can only touch a single element and the
|
|
|
|
// address operand will be updated, so nothing else needs to be done.
|
2009-12-18 20:14:40 +00:00
|
|
|
} else if (LoadInst *LI = dyn_cast<LoadInst>(User)) {
|
|
|
|
const Type *LIType = LI->getType();
|
|
|
|
if (LIType == AI->getAllocatedType()) {
|
|
|
|
// 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
|
|
|
|
// (Also works for arrays instead of structs)
|
|
|
|
Value *Insert = UndefValue::get(LIType);
|
|
|
|
for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
|
|
|
|
Value *Load = new LoadInst(NewElts[i], "load", LI);
|
|
|
|
Insert = InsertValueInst::Create(Insert, Load, i, "insert", LI);
|
|
|
|
}
|
|
|
|
LI->replaceAllUsesWith(Insert);
|
|
|
|
DeadInsts.push_back(LI);
|
|
|
|
} else if (isa<IntegerType>(LIType) &&
|
|
|
|
TD->getTypeAllocSize(LIType) ==
|
|
|
|
TD->getTypeAllocSize(AI->getAllocatedType())) {
|
|
|
|
// If this is a load of the entire alloca to an integer, rewrite it.
|
|
|
|
RewriteLoadUserOfWholeAlloca(LI, AI, NewElts);
|
2009-01-08 05:42:05 +00:00
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
} else if (StoreInst *SI = dyn_cast<StoreInst>(User)) {
|
|
|
|
Value *Val = SI->getOperand(0);
|
|
|
|
const Type *SIType = Val->getType();
|
|
|
|
if (SIType == AI->getAllocatedType()) {
|
|
|
|
// 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
|
|
|
|
// (Also works for arrays instead of structs)
|
|
|
|
for (unsigned i = 0, e = NewElts.size(); i != e; ++i) {
|
|
|
|
Value *Extract = ExtractValueInst::Create(Val, i, Val->getName(), SI);
|
|
|
|
new StoreInst(Extract, NewElts[i], SI);
|
|
|
|
}
|
|
|
|
DeadInsts.push_back(SI);
|
|
|
|
} else if (isa<IntegerType>(SIType) &&
|
|
|
|
TD->getTypeAllocSize(SIType) ==
|
|
|
|
TD->getTypeAllocSize(AI->getAllocatedType())) {
|
|
|
|
// If this is a store of the entire alloca from an integer, rewrite it.
|
|
|
|
RewriteStoreUserOfWholeAlloca(SI, AI, NewElts);
|
2009-02-10 07:00:59 +00:00
|
|
|
}
|
2009-12-17 18:34:24 +00:00
|
|
|
}
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
/// RewriteBitCast - Update a bitcast reference to the alloca being replaced
|
|
|
|
/// and recursively continue updating all of its uses.
|
|
|
|
void SROA::RewriteBitCast(BitCastInst *BC, AllocaInst *AI, uint64_t Offset,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
|
|
|
RewriteForScalarRepl(BC, AI, Offset, NewElts);
|
|
|
|
if (BC->getOperand(0) != AI)
|
|
|
|
return;
|
2009-12-16 20:09:53 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
// The bitcast references the original alloca. Replace its uses with
|
|
|
|
// references to the first new element alloca.
|
|
|
|
Instruction *Val = NewElts[0];
|
|
|
|
if (Val->getType() != BC->getDestTy()) {
|
|
|
|
Val = new BitCastInst(Val, BC->getDestTy(), "", BC);
|
|
|
|
Val->takeName(BC);
|
|
|
|
}
|
|
|
|
BC->replaceAllUsesWith(Val);
|
|
|
|
DeadInsts.push_back(BC);
|
|
|
|
}
|
2009-12-16 20:09:53 +00:00
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
/// FindElementAndOffset - Return the index of the element containing Offset
|
|
|
|
/// within the specified type, which must be either a struct or an array.
|
|
|
|
/// Sets T to the type of the element and Offset to the offset within that
|
2009-12-19 06:53:17 +00:00
|
|
|
/// element. IdxTy is set to the type of the index result to be used in a
|
|
|
|
/// GEP instruction.
|
|
|
|
uint64_t SROA::FindElementAndOffset(const Type *&T, uint64_t &Offset,
|
|
|
|
const Type *&IdxTy) {
|
|
|
|
uint64_t Idx = 0;
|
2009-12-18 20:14:40 +00:00
|
|
|
if (const StructType *ST = dyn_cast<StructType>(T)) {
|
|
|
|
const StructLayout *Layout = TD->getStructLayout(ST);
|
|
|
|
Idx = Layout->getElementContainingOffset(Offset);
|
|
|
|
T = ST->getContainedType(Idx);
|
|
|
|
Offset -= Layout->getElementOffset(Idx);
|
2009-12-19 06:53:17 +00:00
|
|
|
IdxTy = Type::getInt32Ty(T->getContext());
|
|
|
|
return Idx;
|
2009-12-18 20:14:40 +00:00
|
|
|
}
|
2009-12-19 06:53:17 +00:00
|
|
|
const ArrayType *AT = cast<ArrayType>(T);
|
|
|
|
T = AT->getElementType();
|
|
|
|
uint64_t EltSize = TD->getTypeAllocSize(T);
|
|
|
|
Idx = Offset / EltSize;
|
|
|
|
Offset -= Idx * EltSize;
|
|
|
|
IdxTy = Type::getInt64Ty(T->getContext());
|
2009-12-18 20:14:40 +00:00
|
|
|
return Idx;
|
|
|
|
}
|
|
|
|
|
|
|
|
/// RewriteGEP - Check if this GEP instruction moves the pointer across
|
|
|
|
/// elements of the alloca that are being split apart, and if so, rewrite
|
|
|
|
/// the GEP to be relative to the new element.
|
|
|
|
void SROA::RewriteGEP(GetElementPtrInst *GEPI, AllocaInst *AI, uint64_t Offset,
|
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
|
|
|
uint64_t OldOffset = Offset;
|
|
|
|
SmallVector<Value*, 8> Indices(GEPI->op_begin() + 1, GEPI->op_end());
|
|
|
|
Offset += TD->getIndexedOffset(GEPI->getPointerOperandType(),
|
|
|
|
&Indices[0], Indices.size());
|
|
|
|
|
|
|
|
RewriteForScalarRepl(GEPI, AI, Offset, NewElts);
|
|
|
|
|
|
|
|
const Type *T = AI->getAllocatedType();
|
2009-12-19 06:53:17 +00:00
|
|
|
const Type *IdxTy;
|
|
|
|
uint64_t OldIdx = FindElementAndOffset(T, OldOffset, IdxTy);
|
2009-12-18 20:14:40 +00:00
|
|
|
if (GEPI->getOperand(0) == AI)
|
2009-12-19 06:53:17 +00:00
|
|
|
OldIdx = ~0ULL; // Force the GEP to be rewritten.
|
2009-12-18 20:14:40 +00:00
|
|
|
|
|
|
|
T = AI->getAllocatedType();
|
|
|
|
uint64_t EltOffset = Offset;
|
2009-12-19 06:53:17 +00:00
|
|
|
uint64_t Idx = FindElementAndOffset(T, EltOffset, IdxTy);
|
2009-12-18 20:14:40 +00:00
|
|
|
|
|
|
|
// If this GEP does not move the pointer across elements of the alloca
|
|
|
|
// being split, then it does not needs to be rewritten.
|
|
|
|
if (Idx == OldIdx)
|
|
|
|
return;
|
|
|
|
|
|
|
|
const Type *i32Ty = Type::getInt32Ty(AI->getContext());
|
|
|
|
SmallVector<Value*, 8> NewArgs;
|
|
|
|
NewArgs.push_back(Constant::getNullValue(i32Ty));
|
|
|
|
while (EltOffset != 0) {
|
2009-12-19 06:53:17 +00:00
|
|
|
uint64_t EltIdx = FindElementAndOffset(T, EltOffset, IdxTy);
|
|
|
|
NewArgs.push_back(ConstantInt::get(IdxTy, EltIdx));
|
2009-12-18 20:14:40 +00:00
|
|
|
}
|
|
|
|
Instruction *Val = NewElts[Idx];
|
|
|
|
if (NewArgs.size() > 1) {
|
|
|
|
Val = GetElementPtrInst::CreateInBounds(Val, NewArgs.begin(),
|
|
|
|
NewArgs.end(), "", GEPI);
|
|
|
|
Val->takeName(GEPI);
|
2009-12-16 20:09:53 +00:00
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
if (Val->getType() != GEPI->getType())
|
2010-01-27 19:46:52 +00:00
|
|
|
Val = new BitCastInst(Val, GEPI->getType(), Val->getName(), GEPI);
|
2009-12-18 20:14:40 +00:00
|
|
|
GEPI->replaceAllUsesWith(Val);
|
|
|
|
DeadInsts.push_back(GEPI);
|
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.
|
2009-12-18 20:14:40 +00:00
|
|
|
void SROA::RewriteMemIntrinUserOfAlloca(MemIntrinsic *MI, Instruction *Inst,
|
2009-10-23 21:09:37 +00:00
|
|
|
AllocaInst *AI,
|
2009-01-07 07:18:45 +00:00
|
|
|
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
|
2009-12-18 20:14:40 +00:00
|
|
|
if (Inst == MTI->getRawDest())
|
2009-03-08 03:37:16 +00:00
|
|
|
OtherPtr = MTI->getRawSource();
|
2009-01-07 07:18:45 +00:00
|
|
|
else {
|
2009-12-18 20:14:40 +00:00
|
|
|
assert(Inst == MTI->getRawSource());
|
2009-03-08 03:37:16 +00:00
|
|
|
OtherPtr = MTI->getRawDest();
|
2007-03-05 07:52:57 +00:00
|
|
|
}
|
2009-01-07 07:18:45 +00:00
|
|
|
}
|
2009-12-08 18:22:03 +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) {
|
2009-12-18 20:14:40 +00:00
|
|
|
|
|
|
|
// Remove bitcasts and all-zero GEPs from OtherPtr. This is an
|
|
|
|
// optimization, but it's also required to detect the corner case where
|
|
|
|
// both pointer operands are referencing the same memory, and where
|
|
|
|
// OtherPtr may be a bitcast or GEP that currently being rewritten. (This
|
|
|
|
// function is only called for mem intrinsics that access the whole
|
|
|
|
// aggregate, so non-zero GEPs are not an issue here.)
|
|
|
|
while (1) {
|
|
|
|
if (BitCastInst *BC = dyn_cast<BitCastInst>(OtherPtr)) {
|
|
|
|
OtherPtr = BC->getOperand(0);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
if (GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(OtherPtr)) {
|
|
|
|
// All zero GEPs are effectively bitcasts.
|
|
|
|
if (GEP->hasAllZeroIndices()) {
|
|
|
|
OtherPtr = GEP->getOperand(0);
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
break;
|
|
|
|
}
|
2010-01-19 04:32:48 +00:00
|
|
|
// Copying the alloca to itself is a no-op: just delete it.
|
|
|
|
if (OtherPtr == AI || OtherPtr == NewElts[0]) {
|
|
|
|
// This code will run twice for a no-op memcpy -- once for each operand.
|
|
|
|
// Put only one reference to MI on the DeadInsts list.
|
|
|
|
for (SmallVector<Value*, 32>::const_iterator I = DeadInsts.begin(),
|
|
|
|
E = DeadInsts.end(); I != E; ++I)
|
|
|
|
if (*I == MI) return;
|
|
|
|
DeadInsts.push_back(MI);
|
2009-12-18 20:14:40 +00:00
|
|
|
return;
|
2010-01-19 04:32:48 +00:00
|
|
|
}
|
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();
|
2009-12-18 20:14:40 +00:00
|
|
|
bool SROADest = MI->getRawDest() == Inst;
|
2009-01-07 07:18:45 +00:00
|
|
|
|
2009-08-13 21:58:54 +00:00
|
|
|
Constant *Zero = Constant::getNullValue(Type::getInt32Ty(MI->getContext()));
|
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;
|
|
|
|
|
2010-01-19 04:32:48 +00:00
|
|
|
if (OtherPtr) {
|
2009-08-13 21:58:54 +00:00
|
|
|
Value *Idx[2] = { Zero,
|
|
|
|
ConstantInt::get(Type::getInt32Ty(MI->getContext()), i) };
|
2009-12-18 20:14:40 +00:00
|
|
|
OtherElt = GetElementPtrInst::CreateInBounds(OtherPtr, Idx, Idx + 2,
|
2010-01-27 19:46:52 +00:00
|
|
|
OtherPtr->getName()+"."+Twine(i),
|
2009-12-18 20:14:40 +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-31 20:28:14 +00:00
|
|
|
StoreVal = Constant::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)
|
2010-01-27 19:46:52 +00:00
|
|
|
EltPtr = new BitCastInst(EltPtr, BytePtrTy, EltPtr->getName(), 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)
|
2010-01-27 19:46:52 +00:00
|
|
|
OtherElt = new BitCastInst(OtherElt, BytePtrTy, OtherElt->getName(), MI);
|
2009-01-07 07:18:45 +00:00
|
|
|
|
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
|
2009-08-13 21:58:54 +00:00
|
|
|
// Align
|
|
|
|
ConstantInt::get(Type::getInt32Ty(MI->getContext()), OtherEltAlign)
|
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
|
|
|
}
|
2009-12-18 20:14:40 +00:00
|
|
|
DeadInsts.push_back(MI);
|
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
|
|
|
|
2009-12-04 21:57:37 +00:00
|
|
|
/// RewriteStoreUserOfWholeAlloca - We found a store of an integer that
|
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
|
|
|
/// overwrites the entire allocation. Extract out the pieces of the stored
|
|
|
|
/// integer and store them individually.
|
2009-10-23 21:09:37 +00:00
|
|
|
void SROA::RewriteStoreUserOfWholeAlloca(StoreInst *SI, AllocaInst *AI,
|
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
|
|
|
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.
|
|
|
|
Value *SrcVal = SI->getOperand(0);
|
2009-12-18 20:14:40 +00:00
|
|
|
const Type *AllocaEltTy = AI->getAllocatedType();
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t AllocaSizeBits = TD->getTypeAllocSizeInBits(AllocaEltTy);
|
2009-01-07 07:18:45 +00:00
|
|
|
|
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-08-13 21:58:54 +00:00
|
|
|
IntegerType::get(SI->getContext(), 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
|
|
|
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "PROMOTING STORE TO WHOLE ALLOCA: " << *AI << '\n' << *SI
|
2009-09-15 07:08:25 +00:00
|
|
|
<< '\n');
|
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
|
|
|
|
|
|
|
// 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-08-13 21:58:54 +00:00
|
|
|
IntegerType::get(SI->getContext(), 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-29 22:17:13 +00:00
|
|
|
PointerType::getUnqual(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-08-13 21:58:54 +00:00
|
|
|
IntegerType::get(SI->getContext(),
|
|
|
|
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-29 22:17:13 +00:00
|
|
|
PointerType::getUnqual(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;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2009-12-18 20:14:40 +00:00
|
|
|
DeadInsts.push_back(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
|
|
|
}
|
|
|
|
|
2009-12-04 21:57:37 +00:00
|
|
|
/// RewriteLoadUserOfWholeAlloca - We found a load of the entire allocation to
|
2009-01-08 05:42:05 +00:00
|
|
|
/// an integer. Load the individual pieces to form the aggregate value.
|
2009-10-23 21:09:37 +00:00
|
|
|
void SROA::RewriteLoadUserOfWholeAlloca(LoadInst *LI, AllocaInst *AI,
|
2009-01-08 05:42:05 +00:00
|
|
|
SmallVector<AllocaInst*, 32> &NewElts) {
|
|
|
|
// Extract each element out of the NewElts according to its structure offset
|
|
|
|
// and form the result value.
|
2009-12-18 20:14:40 +00:00
|
|
|
const Type *AllocaEltTy = AI->getAllocatedType();
|
2009-05-09 07:06:46 +00:00
|
|
|
uint64_t AllocaSizeBits = TD->getTypeAllocSizeInBits(AllocaEltTy);
|
2009-01-08 05:42:05 +00:00
|
|
|
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "PROMOTING LOAD OF WHOLE ALLOCA: " << *AI << '\n' << *LI
|
2009-09-15 07:08:25 +00:00
|
|
|
<< '\n');
|
2009-01-08 05:42:05 +00:00
|
|
|
|
|
|
|
// 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
|
|
|
|
|
|
|
Value *ResultVal =
|
2009-08-13 21:58:54 +00:00
|
|
|
Constant::getNullValue(IntegerType::get(LI->getContext(), 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-08-13 21:58:54 +00:00
|
|
|
const IntegerType *FieldIntTy = IntegerType::get(LI->getContext(),
|
|
|
|
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-29 22:17:13 +00:00
|
|
|
PointerType::getUnqual(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);
|
2009-12-18 20:14:40 +00:00
|
|
|
DeadInsts.push_back(LI);
|
2009-01-08 05:42:05 +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.
|
2010-01-21 23:05:53 +00:00
|
|
|
bool SROA::isSafeAllocaToScalarRepl(AllocaInst *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;
|
|
|
|
|
2009-12-21 18:39:47 +00:00
|
|
|
isSafeForScalarRepl(AI, AI, 0, Info);
|
2009-12-18 20:14:40 +00:00
|
|
|
if (Info.isUnsafe) {
|
2010-01-05 01:27:09 +00:00
|
|
|
DEBUG(dbgs() << "Cannot transform: " << *AI << '\n');
|
2010-01-21 23:05:53 +00:00
|
|
|
return false;
|
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-12-18 20:14:40 +00:00
|
|
|
HasPadding(AI->getAllocatedType(), *TD))
|
2010-01-21 23:05:53 +00:00
|
|
|
return false;
|
2004-11-14 04:24:28 +00:00
|
|
|
|
2010-01-21 23:05:53 +00:00
|
|
|
return true;
|
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.
|
2009-08-13 21:58:54 +00:00
|
|
|
if (VecTy != Type::getVoidTy(Context)) { // either null or a vector type.
|
2009-02-03 01:30:09 +00:00
|
|
|
|
|
|
|
// 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;
|
|
|
|
}
|
2009-10-05 05:54:46 +00:00
|
|
|
} else if (In->isFloatTy() || In->isDoubleTy() ||
|
2009-02-03 01:30:09 +00:00
|
|
|
(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-29 22:17:13 +00:00
|
|
|
VecTy = VectorType::get(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.
|
2009-08-13 21:58:54 +00:00
|
|
|
VecTy = Type::getVoidTy(Context);
|
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-12-09 18:05:27 +00:00
|
|
|
/// its accesses to a single vector type, return true and set VecTy to
|
2009-02-03 01:30:09 +00:00
|
|
|
/// 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());
|
2009-12-18 20:14:40 +00:00
|
|
|
uint64_t GEPOffset = TD->getIndexedOffset(GEP->getPointerOperandType(),
|
2009-01-31 02:28:54 +00:00
|
|
|
&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-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());
|
2009-12-18 20:14:40 +00:00
|
|
|
uint64_t GEPOffset = TD->getIndexedOffset(GEP->getPointerOperandType(),
|
2009-01-31 02:28:54 +00:00
|
|
|
&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-12-22 19:33:28 +00:00
|
|
|
Instruction *Old = Builder.CreateLoad(NewAI, NewAI->getName()+".in");
|
2009-02-03 19:45:44 +00:00
|
|
|
Value *New = ConvertScalar_InsertValue(SI->getOperand(0), Old, Offset,
|
|
|
|
Builder);
|
|
|
|
Builder.CreateStore(New, NewAI);
|
|
|
|
SI->eraseFromParent();
|
2009-12-22 19:33:28 +00:00
|
|
|
|
|
|
|
// If the load we just inserted is now dead, then the inserted store
|
|
|
|
// overwrote the entire thing.
|
|
|
|
if (Old->use_empty())
|
|
|
|
Old->eraseFromParent();
|
2009-02-03 19:45:44 +00:00
|
|
|
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-11-29 21:17:48 +00:00
|
|
|
|
2009-12-22 19:33:28 +00:00
|
|
|
Instruction *Old = Builder.CreateLoad(NewAI, NewAI->getName()+".in");
|
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-12-22 19:33:28 +00:00
|
|
|
|
|
|
|
// If the load we just inserted is now dead, then the memset overwrote
|
|
|
|
// the entire thing.
|
|
|
|
if (Old->use_empty())
|
|
|
|
Old->eraseFromParent();
|
2009-04-21 16:52:12 +00:00
|
|
|
}
|
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.
|
2010-01-25 18:26:54 +00:00
|
|
|
AllocaInst *OrigAI = cast<AllocaInst>(Ptr->getUnderlyingObject(0));
|
2009-03-08 04:04:21 +00:00
|
|
|
|
2010-01-25 18:26:54 +00:00
|
|
|
if (MTI->getSource()->getUnderlyingObject(0) != OrigAI) {
|
2009-03-08 04:04:21 +00:00
|
|
|
// 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);
|
2010-01-25 18:26:54 +00:00
|
|
|
} else if (MTI->getDest()->getUnderlyingObject(0) != OrigAI) {
|
2009-03-08 04:04:21 +00:00
|
|
|
// 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-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-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-08-13 21:58:54 +00:00
|
|
|
Value *V = Builder.CreateExtractElement(FromVal, ConstantInt::get(
|
|
|
|
Type::getInt32Ty(FromVal->getContext()), Elt), "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-30 23:03:37 +00:00
|
|
|
Value *Res = UndefValue::get(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-30 23:03:37 +00:00
|
|
|
Value *Res = UndefValue::get(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-08-13 21:58:54 +00:00
|
|
|
Builder.CreateTrunc(FromVal, IntegerType::get(FromVal->getContext(),
|
|
|
|
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-08-13 21:58:54 +00:00
|
|
|
Builder.CreateZExt(FromVal, IntegerType::get(FromVal->getContext(),
|
|
|
|
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-08-13 21:58:54 +00:00
|
|
|
ConstantInt::get(Type::getInt32Ty(SV->getContext()), 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-08-13 21:58:54 +00:00
|
|
|
SV = Builder.CreateBitCast(SV,
|
|
|
|
IntegerType::get(SV->getContext(),SrcWidth), "tmp");
|
2009-01-31 02:28:54 +00:00
|
|
|
else if (isa<PointerType>(SV->getType()))
|
2009-08-13 21:58:54 +00:00
|
|
|
SV = Builder.CreatePtrToInt(SV, TD->getIntPtrType(SV->getContext()), "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.
|
2009-10-23 21:09:37 +00:00
|
|
|
Instruction *SROA::isOnlyCopiedFromConstantGlobal(AllocaInst *AI) {
|
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
|
|
|
Instruction *TheCopy = 0;
|
|
|
|
if (::isOnlyCopiedFromConstantGlobal(AI, TheCopy, false))
|
|
|
|
return TheCopy;
|
|
|
|
return 0;
|
|
|
|
}
|