[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