From edc42b2dc18c1bf413fc393eeb05abd6076d8bbe Mon Sep 17 00:00:00 2001 From: Fangrui Song Date: Fri, 27 Dec 2024 21:01:08 -0800 Subject: [PATCH] [SLP] Migrate away from PointerUnion::get --- .../Transforms/Vectorize/SLPVectorizer.cpp | 32 +++++++++---------- 1 file changed, 15 insertions(+), 17 deletions(-) diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index 7f4c3d44b0ec..f52ddfda5e64 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -10776,23 +10776,21 @@ public: } if (ForExtracts) { // No need to add vectors here, already handled them in adjustExtracts. - assert( - InVectors.size() == 1 && isa(InVectors.front()) && - !CommonMask.empty() && - all_of(enumerate(CommonMask), - [&](auto P) { - Value *Scalar = - InVectors.front().get()->getOrdered( - P.index()); - if (P.value() == PoisonMaskElem) - return P.value() == Mask[P.index()] || - isa(Scalar); - if (isa(V1)) - return true; - auto *EI = cast(Scalar); - return EI->getVectorOperand() == V1; - }) && - "Expected only tree entry for extractelement vectors."); + assert(InVectors.size() == 1 && isa(InVectors[0]) && + !CommonMask.empty() && + all_of(enumerate(CommonMask), + [&](auto P) { + Value *Scalar = cast(InVectors[0]) + ->getOrdered(P.index()); + if (P.value() == PoisonMaskElem) + return P.value() == Mask[P.index()] || + isa(Scalar); + if (isa(V1)) + return true; + auto *EI = cast(Scalar); + return EI->getVectorOperand() == V1; + }) && + "Expected only tree entry for extractelement vectors."); return; } assert(!InVectors.empty() && !CommonMask.empty() &&