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

Refactor code to use new attribute getters on CallSite for NoCapture and ByVal.

Browse Source 
Suggested in code review by Eli.

That code in InstCombine looks kinda suspicious.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@145013 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Nick Lewycky
2011-11-20 19:09:04 +00:00
parent 742e5cf612
commit 173862e546
8 changed files with 19 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Transforms/Scalar/MemCpyOptimizer.cpp
View File

@@ -950,7 +950,7 @@ bool MemCpyOpt::iterateOnFunction(Function &F) {
RepeatInstruction = processMemMove(M);
else if (CallSite CS = (Value*)I) {
for (unsigned i = 0, e = CS.arg_size(); i != e; ++i)
if (CS.paramHasAttr(i+1, Attribute::ByVal))
if (CS.isByValArgument(i))
MadeChange |= processByValArgument(CS, i);
}

5
lib/Transforms/Scalar/ScalarReplAggregates.cpp
View File

@@ -2534,13 +2534,12 @@ isOnlyCopiedFromConstantGlobal(Value *V, MemTransferInst *&TheCopy,
// ignore it if we know that the value isn't captured.
unsigned ArgNo = CS.getArgumentNo(UI);
if (CS.onlyReadsMemory() &&
(CS.getInstruction()->use_empty() ||
CS.paramHasAttr(ArgNo+1, Attribute::NoCapture)))
(CS.getInstruction()->use_empty() || CS.doesNotCapture(ArgNo)))
continue;
// If this is being passed as a byval argument, the caller is making a
// copy, so it is only a read of the alloca.
if (CS.paramHasAttr(ArgNo+1, Attribute::ByVal))
if (CS.isByValArgument(ArgNo))
continue;
}