Skip to content

Commit 09f5b9a

Browse files
committed
Revert "[SLP]Do not include copyable data to the same user twice"
This reverts commit 758c685 to fix buildbot https://lab.llvm.org/buildbot/#/builders/195/builds/13298
1 parent d7a29e5 commit 09f5b9a

File tree

2 files changed

+0
-69
lines changed

2 files changed

+0
-69
lines changed

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5336,7 +5336,6 @@ class BoUpSLP {
53365336
ArrayRef<Value *> Op = EI.UserTE->getOperand(EI.EdgeIdx);
53375337
const auto *It = find(Op, I);
53385338
assert(It != Op.end() && "Lane not set");
5339-
SmallPtrSet<Instruction *, 4> Visited;
53405339
do {
53415340
int Lane = std::distance(Op.begin(), It);
53425341
assert(Lane >= 0 && "Lane not set");
@@ -5346,10 +5345,6 @@ class BoUpSLP {
53465345
assert(Lane < static_cast<int>(EI.UserTE->Scalars.size()) &&
53475346
"Couldn't find extract lane");
53485347
auto *In = cast<Instruction>(EI.UserTE->Scalars[Lane]);
5349-
if (!Visited.insert(In).second) {
5350-
It = find(make_range(std::next(It), Op.end()), I);
5351-
continue;
5352-
}
53535348
ScheduleCopyableDataMapByInstUser
53545349
.try_emplace(std::make_pair(std::make_pair(In, EI.EdgeIdx), I))
53555350
.first->getSecond()

llvm/test/Transforms/SLPVectorizer/X86/schedule-same-user-with-copyable.ll

Lines changed: 0 additions & 64 deletions
This file was deleted.

0 commit comments

Comments
 (0)