hokein added a comment. In D84811#2199510 <https://reviews.llvm.org/D84811#2199510>, @kbobyrev wrote:
> Even despite `FileFilter` not being fully implemented yet (an issue for a > separate patch) I think this change should still be correct and is probably > OK to land, WDYT @hokein? Yes, personally no objection on landing this, but landing it doesn't seem to help much for your remote-index case (STL symbols are not filtered out)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84811/new/ https://reviews.llvm.org/D84811 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
