forked from OSchip/llvm-project
2fec8860d8
Code patterns like this are common, `#` at the line beginning (https://google.github.io/styleguide/cppguide.html#Preprocessor_Directives), one space indentation for if/elif/else directives. ``` #if SANITIZER_LINUX # if defined(__aarch64__) # endif #endif ``` However, currently clang-format wants to reformat the code to ``` #if SANITIZER_LINUX #if defined(__aarch64__) #endif #endif ``` This significantly harms readability in my review. Use `IndentPPDirectives: AfterHash` to defeat the diagnostic. clang-format will now suggest: ``` #if SANITIZER_LINUX # if defined(__aarch64__) # endif #endif ``` Unfortunately there is no clang-format option using indent with 1 for just preprocessor directives. However, this is still one step forward from the current behavior. Reviewed By: #sanitizers, vitalybuka Differential Revision: https://reviews.llvm.org/D100238 |
||
|---|---|---|
| .. | ||
| .clang-format | ||
| CMakeLists.txt | ||
| lsan.cpp | ||
| lsan.h | ||
| lsan_allocator.cpp | ||
| lsan_allocator.h | ||
| lsan_common.cpp | ||
| lsan_common.h | ||
| lsan_common_fuchsia.cpp | ||
| lsan_common_linux.cpp | ||
| lsan_common_mac.cpp | ||
| lsan_flags.inc | ||
| lsan_fuchsia.cpp | ||
| lsan_fuchsia.h | ||
| lsan_interceptors.cpp | ||
| lsan_linux.cpp | ||
| lsan_mac.cpp | ||
| lsan_malloc_mac.cpp | ||
| lsan_posix.cpp | ||
| lsan_posix.h | ||
| lsan_preinit.cpp | ||
| lsan_thread.cpp | ||
| lsan_thread.h | ||
| weak_symbols.txt | ||