[PATCH] D54543: [stack-safety] Inter-Procedural Analysis implementation

Vitaly Buka via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 21 14:51:42 PST 2018


vitalybuka added inline comments.


================
Comment at: llvm/lib/Analysis/StackSafetyAnalysis.cpp:514
+  runDataFlow();
+  LLVM_DEBUG(verifyFixedPoint());
+
----------------
eugenis wrote:
> This makes verifyFixedPoint unused in no-assertions build. Move it under LLVM_DEBUG, too.
I'd rather keep unused function than introducing "#ifndef NDEBUG"


Repository:
  rL LLVM

https://reviews.llvm.org/D54543





More information about the llvm-commits mailing list