User Details
- User Since
- Apr 9 2023, 1:21 PM (25 w, 2 d)
Aug 12 2023
Aug 11 2023
The full output of ClangBuildAnalyzer with the removed template line highlighted is here
Aug 3 2023
Corresponding PR (https://github.com/google/googletest/pull/4330) and issue (https://github.com/google/googletest/issues/4329) for GoogleTest.
Jul 13 2023
@fdeazeve Thank you!
@MaskRay seems like ninja on its own after preparing lldb isn't enough to build tests. I've now built llvm, clang, flang, bolt, lldb, + tests - I'm hoping that this is it now!
Jul 11 2023
Jul 9 2023
Jul 8 2023
After the previous attempt, I believe only LLDB was failing to compile. These are the necessary changes needed to fix all of LLDB locally on main.
Run clang-format over changes
Jul 7 2023
Rebasing on main
@MaskRay Thanks. I have updated the title to use [Support] and I will land the additional includes separately to the removal. This was very good advice last attempt as it I did have to revert the change.
Fix XCOFFDump.cpp
Jul 6 2023
Fix CompressionTest.cpp
Jul 5 2023
Jun 27 2023
Jun 26 2023
Jun 25 2023
This was found with IncludeGuardian 0.0.8 and the recommendation was given here https://gist.github.com/IncludeGuardian/4132d5149576a55e0560fae6f8ff4a38#file-before-yaml-L4053-L4055
@MaskRay good idea, I have split into 2 commits and pushed to the main
Jun 24 2023
Jun 23 2023
Fix missing includes for Windows-only files
Add back all includes (bad arc change)
Fix missing include
@barannikov88 no it's not. I was going to commit separately to keep the change small, but it turns out that if I move this to the source file there are no additional changes needed. SmallVector.h has now been moved to Error.cpp as well.
Fix compile issues
Jun 21 2023
Jun 18 2023
This was found with IncludeGuardian 0.0.8.
This is good to review now. Phabricator has become a bit confused after rebasing on top of https://reviews.llvm.org/D151557, I don't know whether it's worth creating a new, cleaner review and close this on?
Jun 16 2023
Run clang-format over patch
Jun 15 2023
Jun 13 2023
Jun 12 2023
Fix missing includes in test, fix formatting, and rebase on top of https://reviews.llvm.org/D151557
Anyone want to give this a final look and approval?
Jun 6 2023
LGTM!
Jun 3 2023
May 23 2023
@kadircet Thanks I will run this through the formatter.
May 21 2023
May 20 2023
Merged as ac73c48e091cff69047ed2dfb972f69ef7a0e9a0
Add missing specialization for clang-include-cleaner/Types.h