6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2025-08-09 11:25:55 +00:00
Code Issues Projects Releases Wiki Activity

R600: Don't promote allocas when one of the users is a ptrtoint instruction

Browse Source 
We need to figure out how to track ptrtoint values all the
way until result is converted back to a pointer in order
to correctly rewrite the pointer type.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@220997 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Tom Stellard
2014-10-31 20:52:04 +00:00
parent 5d6cee5e65
commit b5c86504a0
2 changed files with 38 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
lib/Target/R600/AMDGPUPromoteAlloca.cpp
View File

@@ -234,7 +234,8 @@ static bool tryPromoteAllocaToVector(AllocaInst *Alloca) {
return true; return true;
} }
static void collectUsesWithPtrTypes(Value *Val, std::vector<Value*> &WorkList) { static bool collectUsesWithPtrTypes(Value *Val, std::vector<Value*> &WorkList) {
bool Success = true;
for (User *User : Val->users()) { for (User *User : Val->users()) {
if(std::find(WorkList.begin(), WorkList.end(), User) != WorkList.end()) if(std::find(WorkList.begin(), WorkList.end(), User) != WorkList.end())
continue; continue;
@@ -242,11 +243,20 @@ static void collectUsesWithPtrTypes(Value *Val, std::vector<Value*> &WorkList) {
WorkList.push_back(User); WorkList.push_back(User);
continue; continue;
} }
// FIXME: Correctly handle ptrtoint instructions.
Instruction *UseInst = dyn_cast<Instruction>(User);
if (UseInst && UseInst->getOpcode() == Instruction::PtrToInt)
return false;
if (!User->getType()->isPointerTy()) if (!User->getType()->isPointerTy())
continue; continue;
WorkList.push_back(User); WorkList.push_back(User);
collectUsesWithPtrTypes(User, WorkList);
Success &= collectUsesWithPtrTypes(User, WorkList);
} }
return Success;
} }
void AMDGPUPromoteAlloca::visitAlloca(AllocaInst &I) { void AMDGPUPromoteAlloca::visitAlloca(AllocaInst &I) {
@@ -274,6 +284,13 @@ void AMDGPUPromoteAlloca::visitAlloca(AllocaInst &I) {
return; return;
} }
std::vector<Value*> WorkList;
if (!collectUsesWithPtrTypes(&I, WorkList)) {
DEBUG(dbgs() << " Do not know how to convert all uses\n");
return;
}
DEBUG(dbgs() << "Promoting alloca to local memory\n"); DEBUG(dbgs() << "Promoting alloca to local memory\n");
LocalMemAvailable -= AllocaSize; LocalMemAvailable -= AllocaSize;
@@ -320,10 +337,6 @@ void AMDGPUPromoteAlloca::visitAlloca(AllocaInst &I) {
I.replaceAllUsesWith(Offset); I.replaceAllUsesWith(Offset);
I.eraseFromParent(); I.eraseFromParent();
std::vector<Value*> WorkList;
collectUsesWithPtrTypes(Offset, WorkList);
for (std::vector<Value*>::iterator i = WorkList.begin(), for (std::vector<Value*>::iterator i = WorkList.begin(),
e = WorkList.end(); i != e; ++i) { e = WorkList.end(); i != e; ++i) {
Value *V = *i; Value *V = *i;

19
test/CodeGen/R600/private-memory.ll
View File

@@ -290,3 +290,22 @@ entry:
ret void ret void
} }
; AMDGPUPromoteAlloca does not know how to handle ptrtoint. When it
; finds one, it should stop trying to promote.
; FUNC-LABEL: ptrtoint:
; SI-NOT: DS_WRITE
; SI: BUFFER_STORE_DWORD v{{[0-9]+}}, v{{[0-9]+}}, s[{{[0-9]+:[0-9]+}}], s{{[0-9]+}} offen
; SI: BUFFER_LOAD_DWORD v{{[0-9]+}}, v{{[0-9]+}}, s[{{[0-9]+:[0-9]+}}], s{{[0-9]+}} offen offset:0x5
define void @ptrtoint(i32 addrspace(1)* %out, i32 %a, i32 %b) {
%alloca = alloca [16 x i32]
%tmp0 = getelementptr [16 x i32]* %alloca, i32 0, i32 %a
store i32 5, i32* %tmp0
%tmp1 = ptrtoint [16 x i32]* %alloca to i32
%tmp2 = add i32 %tmp1, 5
%tmp3 = inttoptr i32 %tmp2 to i32*
%tmp4 = getelementptr i32* %tmp3, i32 %b
%tmp5 = load i32* %tmp4
store i32 %tmp5, i32 addrspace(1)* %out
ret void
}