RPCS3/llvm-mirror
1
0
Fork 0
mirror of https://github.com/RPCS3/llvm-mirror.git synced 2024-11-22 18:54:02 +01:00
Code Issues Projects Releases Wiki Activity

[LSV] Use make_range, and reformat a DEBUG message. NFC

Browse Source 
Summary:
The DEBUG message was hard to read because two Values were being printed
on the same line with only the delimiter "aliases".  This change makes
us print each Value on its own line.

Reviewers: asbirlea

Subscribers: llvm-commits, arsenm, mzolotukhin

Differential Revision: https://reviews.llvm.org/D22533

llvm-svn: 276055
This commit is contained in:
Justin Lebar 2016-07-19 23:19:18 +00:00
parent 79abe4d38e
commit 2a419b6afb
1 changed files with 15 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
View File

@ -431,14 +431,15 @@ unsigned Vectorizer::getVectorizablePrefixEndIdx(ArrayRef<Value *> Chain,
SmallVector<std::pair<Value *, unsigned>, 16> ChainInstrs;
unsigned InstrIdx = 0;
for (auto I = From; I != To; ++I, ++InstrIdx) {
for (Instruction &I : make_range(From, To)) {
++InstrIdx;
if (isa<LoadInst>(I) || isa<StoreInst>(I)) {
if (!is_contained(Chain, &*I))
MemoryInstrs.push_back({&*I, InstrIdx});
if (!is_contained(Chain, &I))
MemoryInstrs.push_back({&I, InstrIdx});
else
ChainInstrs.push_back({&*I, InstrIdx});
} else if (I->mayHaveSideEffects()) {
DEBUG(dbgs() << "LSV: Found side-effecting operation: " << *I << '\n');
ChainInstrs.push_back({&I, InstrIdx});
} else if (I.mayHaveSideEffects()) {
DEBUG(dbgs() << "LSV: Found side-effecting operation: " << I << '\n');
return 0;
}
}
@ -477,11 +478,13 @@ unsigned Vectorizer::getVectorizablePrefixEndIdx(ArrayRef<Value *> Chain,
Value *Ptr0 = getPointerOperand(M0);
Value *Ptr1 = getPointerOperand(M1);
dbgs() << "LSV: Found alias.\n"
dbgs() << "LSV: Found alias:\n"
" Aliasing instruction and pointer:\n"
<< *MemInstrValue << " aliases " << *Ptr0 << '\n'
<< " " << *MemInstrValue << '\n'
<< " " << *Ptr0 << '\n'
<< " Aliased instruction and pointer:\n"
<< *ChainInstrValue << " aliases " << *Ptr1 << '\n';
<< " " << *ChainInstrValue << '\n'
<< " " << *Ptr1 << '\n';
});
return ChainIdx;
@ -744,7 +747,7 @@ bool Vectorizer::vectorizeStoreChain(
DEBUG({
dbgs() << "LSV: Stores to vectorize:\n";
for (Value *V : Chain)
V->dump();
dbgs() << " " << *V << "\n";
});
// We won't try again to vectorize the elements of the chain, regardless of