[PATCH] D115103: Leak Sanitizer port to Windows
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 7 00:02:33 PST 2021
MyDeveloperDay added inline comments.
================
Comment at: compiler-rt/lib/lsan/lsan.h:20
+#elif SANITIZER_WINDOWS
+# include "lsan_win.h"
#endif
----------------
vitalybuka wrote:
> MyDeveloperDay wrote:
> > clang-format?
> @clemenswasser Can you please a separate tiny patch which clang-format nearby lines
Sorry, is there a reason we can't follow the current style in this patch? sorry did I miss something? Its not about fighting clang-format its about ensuring we follow the style from the lines around it which are following the clang-format style. I don't need you to clang-format the whole file (although that would be great because this directory has very low clang-format status)
{F20932130, size=full}
https://clang.llvm.org/docs/ClangFormattedStatus.html
All I'm asking is that we don't indent the preprocessor directives to a non LLVM style.
i.e.
```
#include "lsan_win.h"
#if !SANITIZER_WINDOWS
#endif
```
vs
```
# include "lsan_win.h"
# if !SANITIZER_WINDOWS
# endif
```
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115103/new/
https://reviews.llvm.org/D115103
More information about the cfe-commits
mailing list