<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Does your revert help with the build? Because we are still seeing compilation errors on macOS, presumably caused by r355231 <a href="https://github.com/llvm/llvm-project/commit/ddc4b7c1d6d4d4ffd8538b40c00c3fb16b1b1ab0" class="">https://github.com/llvm/llvm-project/commit/ddc4b7c1d6d4d4ffd8538b40c00c3fb16b1b1ab0</a><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Mar 1, 2019, at 16:41, Vlad Tsyrklevich <<a href="mailto:vlad@tsyrklevich.net" class="">vlad@tsyrklevich.net</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">I've reverted both of these changes in r355250. The sanitizer build fails on Linux and I have other failures to investigate. (For what it's worth Volodymyr, I took a brief look and it was unclear to me if either were correct. The Linux build was failing before and after your change.)</div><br class=""><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 1, 2019 at 4:07 PM Volodymyr Sapsai via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" class="">llvm-commits@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Martin, I’ve made a small fix in r355244. See inline for more details<br class="">
<br class="">
> On Mar 1, 2019, at 14:30, Martin Storsjo via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a>> wrote:<br class="">
> <br class="">
> Author: mstorsjo<br class="">
> Date: Fri Mar 1 14:30:14 2019<br class="">
> New Revision: 355236<br class="">
> <br class="">
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=355236&view=rev" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project?rev=355236&view=rev</a><br class="">
> Log:<br class="">
> [sanitizers] Don't use Windows Trace Logging on MinGW<br class="">
> <br class="">
> mingw-w64 currently is lacking the headers for this feature.<br class="">
> <br class="">
> Make the include lowercase at the same time. We consistently<br class="">
> use lowercase for windows header includes, as windows itself is<br class="">
> case insensitive, the SDK headers (in general, not necessarily<br class="">
> considering this particular header) aren't consistent among themselves<br class="">
> about what the proper canonical capitalization for headers are,<br class="">
> and MinGW uses all lowercase names for the headers (as it is often<br class="">
> used on case sensitive filesystems).<br class="">
> <br class="">
> In case mingw-w64 later gets this header, we can revert this<br class="">
> (but keep the include lowercased).<br class="">
> <br class="">
> Differential Revision: <a href="https://reviews.llvm.org/D58765" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/D58765</a><br class="">
> <br class="">
> Modified:<br class="">
> compiler-rt/trunk/lib/sanitizer_common/sanitizer_common.h<br class="">
> compiler-rt/trunk/lib/sanitizer_common/<a href="http://sanitizer_win.cc" class="">sanitizer_win.cc</a><br class="">
> <br class="">
> Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_common.h<br class="">
> URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_common.h?rev=355236&r1=355235&r2=355236&view=diff" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_common.h?rev=355236&r1=355235&r2=355236&view=diff</a><br class="">
> ==============================================================================<br class="">
> --- compiler-rt/trunk/lib/sanitizer_common/sanitizer_common.h (original)<br class="">
> +++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_common.h Fri Mar 1 14:30:14 2019<br class="">
> @@ -804,7 +804,13 @@ enum AndroidApiLevel {<br class="">
> <br class="">
> void WriteToSyslog(const char *buffer);<br class="">
> <br class="">
> -#if SANITIZER_MAC || SANITIZER_WINDOWS<br class="">
> +#if defined(SANITIZER_WINDOWS) && defined(_MSC_VER)<br class="">
> +#define SANITIZER_WIN_TRACE 1<br class="">
> +#else<br class="">
> +#define SANITIZER_WIN_TRACE 0<br class="">
> +#endif<br class="">
> +<br class="">
> +#if SANITIZER_LINUX || SANITIZER_WIN_TRACE<br class="">
I think changing SANITIZER_MAC to SANITIZER_LINUX was a mistake and was copy-pasted from the part of the file below. But I can be wrong and maybe we need #if SANITIZER_MAC || SANITIZER_LINUX || SANITIZER_WIN_TRACE<br class="">
<br class="">
> void LogFullErrorReport(const char *buffer);<br class="">
> #else<br class="">
> INLINE void LogFullErrorReport(const char *buffer) {}<br class="">
> @@ -818,7 +824,7 @@ INLINE void WriteOneLineToSyslog(const c<br class="">
> INLINE void LogMessageOnPrintf(const char *str) {}<br class="">
> #endif<br class="">
> <br class="">
> -#if SANITIZER_LINUX || SANITIZER_WINDOWS<br class="">
> +#if SANITIZER_LINUX || SANITIZER_WIN_TRACE<br class="">
> // Initialize Android logging. Any writes before this are silently lost.<br class="">
> void AndroidLogInit();<br class="">
> void SetAbortMessage(const char *);<br class="">
> <br class="">
> Modified: compiler-rt/trunk/lib/sanitizer_common/<a href="http://sanitizer_win.cc" class="">sanitizer_win.cc</a><br class="">
> URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_win.cc?rev=355236&r1=355235&r2=355236&view=diff" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_win.cc?rev=355236&r1=355235&r2=355236&view=diff</a><br class="">
> ==============================================================================<br class="">
> --- compiler-rt/trunk/lib/sanitizer_common/<a href="http://sanitizer_win.cc" class="">sanitizer_win.cc</a> (original)<br class="">
> +++ compiler-rt/trunk/lib/sanitizer_common/<a href="http://sanitizer_win.cc" class="">sanitizer_win.cc</a> Fri Mar 1 14:30:14 2019<br class="">
> @@ -20,7 +20,6 @@<br class="">
> #include <io.h><br class="">
> #include <psapi.h><br class="">
> #include <stdlib.h><br class="">
> -#include <TraceLoggingProvider.h><br class="">
> <br class="">
> #include "sanitizer_common.h"<br class="">
> #include "sanitizer_file.h"<br class="">
> @@ -32,6 +31,8 @@<br class="">
> #if defined(PSAPI_VERSION) && PSAPI_VERSION == 1<br class="">
> #pragma comment(lib, "psapi")<br class="">
> #endif<br class="">
> +#if SANITIZER_WIN_TRACE<br class="">
> +#include <traceloggingprovider.h><br class="">
> // Windows trace logging provider init<br class="">
> #pragma comment(lib, "advapi32.lib")<br class="">
> TRACELOGGING_DECLARE_PROVIDER(g_asan_provider);<br class="">
> @@ -39,6 +40,9 @@ TRACELOGGING_DECLARE_PROVIDER(g_asan_pro<br class="">
> TRACELOGGING_DEFINE_PROVIDER(g_asan_provider, "AddressSanitizerLoggingProvider",<br class="">
> (0x6c6c766d, 0x3846, 0x4e6a, 0xa4, 0xfb, 0x5b,<br class="">
> 0x53, 0x0b, 0xd0, 0xf3, 0xfa));<br class="">
> +#else<br class="">
> +#define TraceLoggingUnregister(x)<br class="">
> +#endif<br class="">
> <br class="">
> // A macro to tell the compiler that this part of the code cannot be reached,<br class="">
> // if the compiler supports this feature. Since we're using this in<br class="">
> @@ -1080,6 +1084,7 @@ u32 GetNumberOfCPUs() {<br class="">
> return sysinfo.dwNumberOfProcessors;<br class="">
> }<br class="">
> <br class="">
> +#if SANITIZER_WIN_TRACE<br class="">
> // TODO(mcgov): Rename this project-wide to PlatformLogInit<br class="">
> void AndroidLogInit(void) {<br class="">
> HRESULT hr = TraceLoggingRegister(g_asan_provider);<br class="">
> @@ -1103,6 +1108,7 @@ void LogFullErrorReport(const char *buff<br class="">
> TraceLoggingValue(buffer, "AsanReportContents"));<br class="">
> }<br class="">
> }<br class="">
> +#endif // SANITIZER_WIN_TRACE<br class="">
> <br class="">
> } // namespace __sanitizer<br class="">
> <br class="">
> <br class="">
> <br class="">
> _______________________________________________<br class="">
> llvm-commits mailing list<br class="">
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a><br class="">
> <a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="">
<br class="">
_______________________________________________<br class="">
llvm-commits mailing list<br class="">
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a><br class="">
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank" class="">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="">
</blockquote></div>
</div></blockquote></div><br class=""></body></html>