<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 28, 2015 at 2:27 AM, Sumanth Gundapaneni <span dir="ltr"><<a href="mailto:sgundapa@codeaurora.org" target="_blank">sgundapa@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="#0563C1" vlink="#954F72"><div><p class="MsoNormal">Hi Rafael,<u></u><u></u></p><p class="MsoNormal">                  I looked at the code which you pushed a while ago to check for the gold linker.<u></u><u></u></p><p class="MsoNormal">Code below:<u></u><u></u></p><p class="MsoNormal">execute_process(<u></u><u></u></p><p class="MsoNormal">    COMMAND ${CMAKE_C_COMPILER} -Wl,--version<u></u><u></u></p><p class="MsoNormal">    OUTPUT_VARIABLE stdout <u></u><u></u></p><p class="MsoNormal">    ERROR_QUIET)<u></u><u></u></p><p class="MsoNormal">  if("${stdout}" MATCHES "GNU gold")<u></u><u></u></p><p class="MsoNormal">    set(LLVM_LINKER_IS_GOLD ON)<u></u><u></u></p><p class="MsoNormal">  endif()<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">I was trying to build runtime libraries (compiler-rt) for ARM using “clang” which includes this code.<u></u><u></u></p><p class="MsoNormal">A while ago, I saw the error message until geek4civic fixed it with “ERROR_QUIET”.<u></u><u></u></p><p class="MsoNormal">I usually specify my sysroot and tool chain(linker) through CFLAGS. <u></u><u></u></p><p class="MsoNormal">To fortify the logic, “${CMAKE_C_COMPILER} ${CMAKE_C_FLAGS}  -Wl, --version” would be ideal than “${CMAKE_C_COMPILER}  -Wl, --version” which works mostly for gnu compilers.<u></u><u></u></p><p class="MsoNormal">Let me know, I can push a patch <u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">execute_process(<u></u><u></u></p><p class="MsoNormal">-      COMMAND ${CMAKE_C_COMPILER} -Wl,--version<u></u><u></u></p><p class="MsoNormal">+    COMMAND “${CMAKE_C_COMPILER}  ${CMAKE_C_FLAGS}” -Wl,--version<u></u><u></u></p><p class="MsoNormal">    OUTPUT_VARIABLE stdout <u></u><u></u></p><p class="MsoNormal">    ERROR_QUIET)<u></u><u></u></p><p class="MsoNormal">  if("${stdout}" MATCHES "GNU gold")<u></u><u></u></p><p class="MsoNormal">    set(LLVM_LINKER_IS_GOLD ON)<u></u><u></u></p><p class="MsoNormal">  endif()</p></div></div></blockquote><div><br></div><div>While this may be convenient I don't think it's strictly correct.</div><div><br></div><div>CFLAGS != LDFLAGS</div><div><br></div><div>I think if extra flags are going to be passed it should be LD specific since that's what this is intended to check.</div></div></div></div>