ilya-biryukov added a comment. Though I'm not a reviewer, I was looking through the emails and found this change interesting. This is not intended to block the review in any way, just trying to get a picture of the direction include-cleaner is taking.
Is this a digression from the initial idea of include-what-you-spell? I thought that the whole point was in trying to make the analysis more "local" and avoid relying on implicit instantiations. What is our strategy on balancing the number of false-positive removals we produce with the number of false-negatives where we don't remove something that's actually unused? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152747/new/ https://reviews.llvm.org/D152747 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
