This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG43974333dd09: [include-cleaner] Unify always_keep with rest
of the keep logic (authored by kadircet).
Repository:
rG LLVM Github Monorepo
CHANGES
VitaNuo added a comment.
Thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156123/new/
https://reviews.llvm.org/D156123
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/
kadircet added inline comments.
Comment at: clang-tools-extra/include-cleaner/lib/Record.cpp:248
+ if (Top.SeenAtFile == SM.getMainFileID() && IncludedFile)
+Out->ShouldKeep.insert(IncludedFile->getUniqueID());
}
VitaNuo wrote:
> If I understand
kadircet updated this revision to Diff 546070.
kadircet added a comment.
- Add tests for pragmas on stdlib headers
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156123/new/
https://reviews.llvm.org/D156123
Files:
clang-tools-extra/clang-tidy/mis
VitaNuo added inline comments.
Comment at: clang-tools-extra/include-cleaner/lib/Record.cpp:248
+ if (Top.SeenAtFile == SM.getMainFileID() && IncludedFile)
+Out->ShouldKeep.insert(IncludedFile->getUniqueID());
}
If I understand correctly, you sh
kadircet created this revision.
kadircet added a reviewer: VitaNuo.
Herald added subscribers: PiotrZSL, carlosgalvezp, arphaman.
Herald added a reviewer: njames93.
Herald added a project: All.
kadircet requested review of this revision.
Herald added a project: clang-tools-extra.
Herald added a subs