[PATCH] D60826: Clarify -Winfinite-recursion message
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 17 16:44:12 PDT 2019
rsmith requested changes to this revision.
rsmith added inline comments.
This revision now requires changes to proceed.
================
Comment at: clang/include/clang/Basic/DiagnosticSemaKinds.td:64
def warn_infinite_recursive_function : Warning<
- "all paths through this function will call itself">,
+ "in order to understand recursion, you must first understand recursion">,
InGroup<InfiniteRecursion>, DefaultIgnore;
----------------
Please also add a note explaining how to understand recursion, and emit it whenever you produce a warning or note that mentions recursion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60826/new/
https://reviews.llvm.org/D60826
More information about the cfe-commits
mailing list