[PATCH] D89721: [GWP-ASan] Cleanup (NFC)

Kostya Kortchinsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 12:39:19 PDT 2020


cryptoad added inline comments.


================
Comment at: compiler-rt/lib/gwp_asan/platform_specific/utilities_posix.cpp:31
 }
-#else  // __BIONIC__
+#else // __BIONIC__
 void Check(bool Condition, const char *Message) {
----------------
hctim wrote:
> my `clang-format --style=LLVM` uses doublespace after `#else` as well, but not after `#endif`. hmm...
I tried a few combinations, with a gLinux stock clang-format, a freshly compiled one from ToT, and they all end up with 1 space.
The one at the top of the file for `stdio` also has 1 space.

Not sure what's the deal with that pre-merge check.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89721



More information about the llvm-commits mailing list