[PATCH] D145894: [IndVars] Option verify-indvars is broken (and always has been), delete it
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 14 01:37:10 PDT 2023
nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:2162
+#ifndef NDEBUG
// Check a post-condition.
----------------
You can drop the ifndef -- we generally don't place VerifyMemorySSA behind NDEBUG, because you need to enable it via a flag anyway.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D145894/new/
https://reviews.llvm.org/D145894
More information about the llvm-commits
mailing list