PiotrZSL added a comment. This change is dangerous, it depends that elements/iterators won't be re-allocated/invalidated during insertion process. Part with removing double lookup in IncludeSorter::addInclude looks fine, but part that involve IncludeBucket is problematic. I will try to push part that involve IncludeLocations optimizations.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117460/new/ https://reviews.llvm.org/D117460 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits