================ @@ -199,6 +200,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 ---------------- kadircet wrote:
```suggestion // Deduplicate insertions when running in bulk fix mode. ``` 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