[PATCH] D103157: Disable misc-no-recursion checking in Clang

2021-05-27 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman closed this revision. aaron.ballman added a comment. Thanks! I've committed in 767d34e3bdddef6c1871006dd0a2d06a4e1bcd5d . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.or

[PATCH] D103157: Disable misc-no-recursion checking in Clang

2021-05-26 Thread Erich Keane via Phabricator via cfe-commits
erichkeane accepted this revision. erichkeane added a comment. This revision is now accepted and ready to land. I like this, since the entire CFE does a ton of recursion on purpose for all AST related things. Please give others time to disagree however :) Repository: rG LLVM Github Monorepo

[PATCH] D103157: Disable misc-no-recursion checking in Clang

2021-05-26 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman created this revision. aaron.ballman added reviewers: alexfh, njames93, rsmith. aaron.ballman requested review of this revision. Herald added a subscriber: aheejin. Herald added a project: clang. We currently enable `misc-no-recursion`, but Clang uses recursion intentionally in a fa