<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 18, 2013 at 2:55 AM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im"><br>
On Jan 17, 2013, at 11:30 AM, Anna Zaks <<a href="mailto:ganna@apple.com">ganna@apple.com</a>> wrote:<br>
<br>
> Kostya,<br>
><br>
> I am getting a makefile build failure (looks like it's caused by this commit):<br>
><br>
>  COMPILE:   clang_darwin/asan_osx_dynamic/x86_64: /Volumes/Data/ws/llvmgit/llvm/projects/compiler-rt/lib/asan/asan_interceptors.cc<br>
> /Volumes/Data/ws/llvmgit/llvm/projects/compiler-rt/lib/asan/asan_interceptors.cc:78:13: error: unused<br>
>      function 'SetThreadName' [-Werror,-Wunused-function]<br>
> /Volumes/Data/ws/llvmgit/llvm/projects/compiler-rt/lib/asan/asan_interceptors.cc:78:13: static void SetThreadName(const char *name) {<br>
> error:             ^<br>
> unused<br>
>      function 'SetThreadName' [-Werror,-Wunused-function]<br>
> static void SetThreadName(const char *name) {<br>
>            ^<br>
> 1 error generated.<br>
<br>
</div>I reverted this patch along with the ones that depended on it to get the buildbots going again.<br>
<br>
Kostya, go ahead and recommit once you've fixed the warning.<br></blockquote><div><br></div><div style>Sorry for the breakage and thanks for the revert! </div><div style><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
BTW, in a project with that many #ifdefs, it is probably not a good idea to build with -Werror.<br></blockquote><div><br></div><div style>Before the commits I build everything on linux with </div><div style><div><br></div>
<div>cmake -DCMAKE_BUILD_TYPE=Release -DLLVM_ENABLE_ASSERTIONS=ON \</div><div> -DCMAKE_C_COMPILER=/home/kcc/host_clang/bin/clang \</div><div> -DCMAKE_CXX_COMPILER=/home/kcc/host_clang/bin/clang++ \</div><div> -DCMAKE_EXPORT_COMPILE_COMMANDS=ON /home/kcc/llvm/ </div>
<div style>make ...</div><div style><br></div><div style>and then I rely on our (unfortunately internal) bots to catch regressions </div><div style>in the gcc build and on non-linux platforms. </div><div style><br></div><div style>
My fault, I did not notice this failure. </div><div style><br></div><div style>--kcc </div><div><br></div><div><br></div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
Thanks,<br>
/jakob<br>
<br>
</blockquote></div><br></div></div>