[llvm] r346858 - [VPlan] Update ifdef.

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 14 05:21:26 PST 2018


Author: fhahn
Date: Wed Nov 14 05:21:26 2018
New Revision: 346858

URL: http://llvm.org/viewvc/llvm-project?rev=346858&view=rev
Log:
[VPlan] Update ifdef.

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

Modified: llvm/trunk/lib/Transforms/Vectorize/VPlanSLP.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Vectorize/VPlanSLP.cpp?rev=346858&r1=346857&r2=346858&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Vectorize/VPlanSLP.cpp (original)
+++ llvm/trunk/lib/Transforms/Vectorize/VPlanSLP.cpp Wed Nov 14 05:21:26 2018
@@ -362,14 +362,14 @@ VPInstruction *VPlanSlp::buildGraph(Arra
   // If we already visited this instruction bundle, re-use the existing node
   auto I = BundleToCombined.find(to_vector<4>(Values));
   if (I != BundleToCombined.end()) {
-#ifdef NDEBUG
+#ifndef NDEBUG
     // Check that the resulting graph is a tree. If we re-use a node, this means
     // its values have multiple users. We only allow this, if all users of each
     // value are the same instruction.
     for (auto *V : Values) {
       auto UI = V->user_begin();
       auto *FirstUser = *UI++;
-      while (UI != V->use_end()) {
+      while (UI != V->user_end()) {
         assert(*UI == FirstUser && "Currently we only support SLP trees.");
         UI++;
       }




More information about the llvm-commits mailing list