<div dir="auto">Hi Bruno,<div dir="auto"><br></div><div dir="auto">Thank you for the fix. I couldn't get to it today. I'll watch the sanitizer bots more carefully in future.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mar 13, 2017 10:26 PM, "Bruno Cardoso Lopes" <<a href="mailto:bruno.cardoso@gmail.com">bruno.cardoso@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Eric,<br>
<br>
I fixed the build for darwin in r297703, let me know if you have any comments.<br>
<br>
Thanks,<br>
<br>
On Mon, Mar 13, 2017 at 3:04 PM, Bruno Cardoso Lopes<br>
<<a href="mailto:bruno.cardoso@gmail.com">bruno.cardoso@gmail.com</a>> wrote:<br>
> Hi Eric,<br>
><br>
>>  if (APPLE AND (LIBCXX_CXX_ABI_LIBNAME STREQUAL "libcxxabi" OR<br>
>> @@ -62,12 +66,7 @@ if (APPLE AND LLVM_USE_SANITIZER)<br>
>>      message(WARNING "LLVM_USE_SANITIZER=${LLVM_<wbr>USE_SANITIZER} is not supported on OS X")<br>
>>    endif()<br>
>>    if (LIBFILE)<br>
>> -    execute_process(COMMAND ${CMAKE_CXX_COMPILER} -print-file-name=lib OUTPUT_VARIABLE LIBDIR RESULT_VARIABLE Result)<br>
>> -    if (NOT ${Result} EQUAL "0")<br>
>> -      message(FATAL "Failed to find library resource directory")<br>
>> -    endif()<br>
>> -    string(STRIP "${LIBDIR}" LIBDIR)<br>
>> -    set(LIBDIR "${LIBDIR}/darwin/")<br>
>> +    find_compiler_rt_dir(LIBDIR)<br>
><br>
> Seems like this broke ASAN+UBSAN bot in green-dragon:<br>
> <a href="http://green.lab.llvm.org/green/job/clang-stage2-cmake-RgSan_build/3812" rel="noreferrer" target="_blank">http://green.lab.llvm.org/<wbr>green/job/clang-stage2-cmake-<wbr>RgSan_build/3812</a><br>
><br>
> Perhaps it's missing the "${LIBDIR}/darwin/" part from above?<br>
><br>
> --<br>
> Bruno Cardoso Lopes<br>
> <a href="http://www.brunocardoso.cc" rel="noreferrer" target="_blank">http://www.brunocardoso.cc</a><br>
<br>
<br>
<br>
--<br>
Bruno Cardoso Lopes<br>
<a href="http://www.brunocardoso.cc" rel="noreferrer" target="_blank">http://www.brunocardoso.cc</a><br>
</blockquote></div></div>