[PATCH] D137697: Move the isSelfContained function from clangd to libtooling.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 11 05:37:29 PST 2022


hokein added inline comments.


================
Comment at: clang/include/clang/Tooling/Inclusions/Header.h:21
+///
+/// A header is considered self-contained if either it has a proper header guard
+/// or it doesn't has dont-include-me-similar pattern.
----------------
kadircet wrote:
> kadircet wrote:
> > let's mention being `#import`d as well
> again comment seems to be the same.
you're looking at the old version, this file has been renamed, please see the https://reviews.llvm.org/D137697.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137697



More information about the cfe-commits mailing list