[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