[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