[PATCH] D65906: [clang-scan-deps] Fix edge cases in the minimizer

Alexandre Ganea via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 8 11:24:37 PDT 2019


aganea added a comment.

In D65906#1620034 <https://reviews.llvm.org/D65906#1620034>, @arphaman wrote:

> Regarding the invisible characters before `#ifdef`, are you sure that's the right behavior?


Should we then copy these invisible characters to the minimized output? Believe it or not, these characters are there in our codebase since 2013, and never clang has complained about it :)


Repository:
  rC Clang

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

https://reviews.llvm.org/D65906





More information about the cfe-commits mailing list