[PATCH] D72362: [clang-tidy] misc-no-recursion: a new check

[Eugene Zelenko via Phabricator via cfe-commits]

Eugene.Zelenko added a comment.

It'll be reasonable to add CERT alias.



================
Comment at: clang-tools-extra/clang-tidy/misc/NoRecursionCheck.cpp:213
+  CallGraphNode::CallRecord *Node = &EntryNode;
+  while (1) {
+    // Did we see this node before?
----------------
true


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/misc-no-recursion.rst:8
+that are loops), diagnoses each function in the cycle,
+and displays one example of possible call graph loop (recursion).
----------------
It'll be reasonable to add links to relevant coding guidelines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72362