[PATCH] D144828: [clang-tidy] Add misc-header-include-cycle check

Piotr Zegar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Mar 5 12:41:31 PST 2023


PiotrZSL marked 4 inline comments as done.
PiotrZSL added a comment.

"But maybe when we emit a warning for a specified include, we could put it in a set to not warn on that include again"
I don't think so, in such case we hide include paths, and most of the time header guards should help here in reduction of cyclic dependences.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144828



More information about the cfe-commits mailing list