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

Sun Jun 27 23:23:04 PDT 2021

Meinersbur added a comment.

In D104601#2842686 <https://reviews.llvm.org/D104601#2842686>, @dblaikie wrote:

> One other thing: This wouldn't be usable when using debug info, presumably, because it'd refer to the wrong lines, etc.

This is considered in the ccache patch <https://github.com/ccache/ccache/pull/815>. By default (unless a corresponding "sloppiness" option is used), only direct hits (by binary file comparison) are used if: There is compiler output, debug info is generated, or `.incbin` assembly instruction is used.

However, the is orthogonal to `-fnormalize-whitespace`. It is a mitigation of using `-E -P` as preprocessor intermediate.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104601/new/

https://reviews.llvm.org/D104601