[PATCH] D55851: Implement basic loop fusion pass

[Roman Tereshin via Phabricator via llvm-commits]

rtereshin added inline comments.


================
Comment at: llvm/trunk/lib/Transforms/Scalar/LoopFuse.cpp:324
+}
+#endif
+
----------------
It appears that if both `NDEBUG` and `LLVM_ENABLE_DUMP` are defined, the function will be compiled in, but all `LLVM_DEBUG` macro eliminated. Resulting
```lang=c++
static void
printFusionCandidates(const FusionCandidateCollection &FusionCandidates) {
  for (const auto &CandidateSet : FusionCandidates) {
  }
}
```
produces an "unused variable" `CandidateSet` warning. We probably don't need to wrap anything in `LLVM_DEBUG` here.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55851/new/

https://reviews.llvm.org/D55851