[PATCH] D104601: [Preprocessor] Implement -fminimize-whitespace.
Michael Kruse via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 1 17:34:54 PDT 2021
Meinersbur added a comment.
@mstorsjo I've taken all steps required for the resolution suggested @aaron.ballman. llvm.org/PR51300 suberseedes llvm.org/PR51261 (release-13.x branch is under @tstellard's control). D107183 <https://reviews.llvm.org/D107183> has been pushed to main to fix the outstanding issue discovered by @romanovvlad (unfortunately without pre-commit review, but given that this might be blocking them, the patch is small and you would like the situation to be resolved by then, I think a post-commit review is acceptable).
@alexfh Happy to fix any additional issues you may find. The -fminimize-whitespace feature unfortunately required some restructuring of the the code since the decision on when to insert newline were done in many different places using different logics.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104601/new/
https://reviews.llvm.org/D104601
More information about the cfe-commits
mailing list