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

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 19 12:44:43 PDT 2020


hctim 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) {
----------------
cryptoad wrote:
> 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.
Maybe me and the mergebot have our binaries from a bad revision. Let me 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