User Details
- User Since
- Jul 19 2018, 7:14 AM (245 w, 2 d)
Mon, Mar 20
Oct 5 2022
Aug 11 2022
Aug 3 2022
May 19 2022
May 18 2022
May 16 2022
Address review comments.
May 15 2022
Also rebase on top of main.
Address review comments: the structure is a bit different but the bug is now
actually removed.
May 12 2022
Remove redundant comment.
Remove unwanted formatting changes.
May 9 2022
May 6 2022
Apr 21 2022
Resolve comment.
Resolve the comments.
Use strlen for performance.
Remove unused include.
Apr 20 2022
Thanks, this looks good; just few nits regarding the comments.
Apr 19 2022
Resolve comments.
@sammccall ping
Apr 12 2022
Oops, sorry, I linked the wrong revision; here's the prototype: we plan to start rolling it out gradually https://reviews.llvm.org/D122677 and then have a common library that we could share between both clangd and Clang-Tidy (and potentially other tools).
Fix the comment.
Resolve comments.
Apr 11 2022
Apr 8 2022
LG to me with the comments that Haojian made!
Apr 7 2022
Apr 6 2022
I think this is no longer [WIP] but rather review-ready, right?
Apr 5 2022
Fix the test case.
Address review comments.
Rebase correctly.
Rebase on top of main.
Switch to stable file UniqueIDs.
Apr 4 2022
This is required for D120306 which was landed but later reverted because of the failing Windows tests (different slash types for filenames requested during preamble parsing and in the main file).
Fix the behavior: get Real Path instead.
Mar 31 2022
Address review comments.
Mar 25 2022
Use better name for header recorder lambda.
Add some docs, rebase on top of main.
Fix a bug, add tests for diagnostics. This is ready for a review now.
Mar 24 2022
Switch to a different model of recording includes that differentiates between
user includes and public headers recored through IWYU pragmas.
Mar 14 2022
Hi! I'm glad you're excited about IncludeCleaner and having a tool to try it out seems like a cool idea!
Mar 8 2022
Feb 22 2022
Move suffix map to CanonicalIncludes (where it logically belongs), remove
component limit there.
The patch is working but I didn't add tests yet. Also, I think I should move the suffix mapping to CanonicalIncludes structure rather than Headers.
Feb 15 2022
Feb 10 2022
Feb 9 2022
LG, thanks!