================
@@ -199,6 +199,9 @@ void IncludeCleanerCheck::check(const
MatchFinder::MatchResult &Result) {
tooling::HeaderIncludes HeaderIncludes(getCurrentMainFile(), Code,
FileStyle->IncludeStyle);
+ // `tooling::HeaderIncludes::insert` will not modify `ExistingIncludes`. We
+ // should handle repeat include here
+ std::set<const std::string> InsertedHeader{};
----------------
danix800 wrote:
`std::set<const std::string>` won't compile, please take a look at the failed
CI checking. `llvm::StringSet` might be better option if `std::string` could be
switched to `StringRef`.
https://github.com/llvm/llvm-project/pull/65431
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
