[PATCH] D104601: [Preprocessor] Implement -fnormalize-whitespace.

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 21 15:17:11 PDT 2021


dblaikie added a comment.

This is probably more @aaron.ballman 's wheelhouse, but for my money this seems pretty problematic - will make quoted text in compiler diagnostics weird/difficult to read, etc.

Do you have a particular use case that has exceptionally frequent whitespace-only changes?
Or is this something you think would be generally applicable? How applicable, do you think? Do you have some data showing a % of rebuilds that would be avoided in practice/based on real code bases, etc?


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