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

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 20 06:47:31 PDT 2021


RKSimon added inline comments.


================
Comment at: clang/lib/Frontend/PrintPreprocessedOutput.cpp:680
+    if (RequireSpace || (!MinimizeWhitespace && Tok.hasLeadingSpace()) ||
+        ((EmittedTokensOnThisLine || EmittedTokensOnThisLine) &&
+         AvoidConcat(PrevPrevTok, PrevTok, Tok)))
----------------
@Meinersbur Static analysis is warning that these are both the same - should one be EmittedDirectiveOnThisLine ?


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