mirror of
https://github.com/llvm/llvm-project.git
synced 2025-04-27 21:26:04 +00:00
ae6b400002
This patch adds the -fminimize-whitespace with the following effects: * If combined with -E, remove as much non-line-breaking whitespace as possible. * If combined with -E -P, removes as much whitespace as possible, including line-breaks. The motivation is to reduce the amount of insignificant changes in the preprocessed output with source files where only whitespace has been changed (add/remove comments, clang-format, etc.) which is in particular useful with ccache. A patch for ccache for using this flag has been proposed to ccache as well: https://github.com/ccache/ccache/pull/815, which will use -fnormalize-whitespace when clang-13 has been detected, and additionally uses -P in "unify_mode". ccache already had a unify_mode in an older version which was removed because of problems that using the preprocessor itself does not have (such that the custom tokenizer did not recognize C++11 raw strings). This patch slightly reorganizes which part is responsible for adding newlines that are required for semantics. It is now either startNewLineIfNeeded() or MoveToLine() but never both; this avoids the ShouldUpdateCurrentLine workaround and avoids redundant lines being inserted in some cases. It also fixes a mandatory newline not inserted after a _Pragma("...") that is expanded into a #pragma. Reviewed By: aaron.ballman Differential Revision: https://reviews.llvm.org/D104601
24 lines
512 B
C
24 lines
512 B
C
// RUN: %clang_cc1 -E -C %s | FileCheck -strict-whitespace %s
|
|
// RUN: %clang_cc1 -E -C -fminimize-whitespace %s | FileCheck -strict-whitespace %s
|
|
|
|
// foo
|
|
// CHECK: // foo
|
|
|
|
/* bar */
|
|
// CHECK: /* bar */
|
|
|
|
#if FOO
|
|
#endif
|
|
/* baz */
|
|
// CHECK: /* baz */
|
|
|
|
_Pragma("unknown") // after unknown pragma
|
|
// CHECK: #pragma unknown
|
|
// CHECK-NEXT: #
|
|
// CHECK-NEXT: // after unknown pragma
|
|
|
|
_Pragma("comment(\"abc\")") // after known pragma
|
|
// CHECK: #pragma comment("abc")
|
|
// CHECK-NEXT: #
|
|
// CHECK-NEXT: // after known pragma
|