[llvm] r369250 - [SLP] reduce duplicated code; NFC

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 19 04:39:56 PDT 2019


Author: spatel
Date: Mon Aug 19 04:39:56 2019
New Revision: 369250

URL: http://llvm.org/viewvc/llvm-project?rev=369250&view=rev
Log:
[SLP] reduce duplicated code; NFC

Modified:
    llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp

Modified: llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp?rev=369250&r1=369249&r2=369250&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/SLPVectorizer.cpp Mon Aug 19 04:39:56 2019
@@ -2176,11 +2176,13 @@ void BoUpSLP::buildTree_rec(ArrayRef<Val
     if (Res.second)
       UniqueValues.emplace_back(V);
   }
-  if (UniqueValues.size() == VL.size()) {
+  size_t NumUniqueScalarValues = UniqueValues.size();
+  if (NumUniqueScalarValues == VL.size()) {
     ReuseShuffleIndicies.clear();
   } else {
     LLVM_DEBUG(dbgs() << "SLP: Shuffle for reused scalars.\n");
-    if (UniqueValues.size() <= 1 || !llvm::isPowerOf2_32(UniqueValues.size())) {
+    if (NumUniqueScalarValues <= 1 ||
+        !llvm::isPowerOf2_32(NumUniqueScalarValues)) {
       LLVM_DEBUG(dbgs() << "SLP: Scalar used twice in bundle.\n");
       newTreeEntry(VL, None /*not vectorized*/, UserTreeIdx);
       return;




More information about the llvm-commits mailing list