[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