<div dir="ltr">Serguei,<div><br></div><div>glibc 2.5 was released 11 years ago, so I wonder what operating system you are using now.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 4, 2017 at 12:08 AM, Serguei Katkov via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.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 class="m_-5366148437303026918WordSection1">
<p class="MsoNormal">Hi All,<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">The landed patch <a href="https://reviews.llvm.org/D38481" target="_blank">https://reviews.llvm.org/<wbr>D38481</a> introduced the usage of CPU_COUNT defined in glibc sched.h header.<u></u><u></u></p>
<p class="MsoNormal">I failed to find this symbol in sched.h of glibc version 2.5-24, so compilation just fails.<u></u><u></u></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New";color:black">/home/dolphin/merge-from-<wbr>upstream-area/ws/pristine/lib/<wbr>Support/Threading.cpp: In function ‘unsigned int llvm::hardware_concurrency()’:<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New";color:black">/home/dolphin/merge-from-<wbr>upstream-area/ws/pristine/lib/<wbr>Support/Threading.cpp:80:26: error: ‘CPU_COUNT’ was not declared in this scope<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New";color:black">     return CPU_COUNT(&Set);<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:10.0pt;font-family:"Courier New";color:black">                          ^<u></u><u></u></span></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">It is buildable with newest version of glibc. <u></u><u></u></p>
<p class="MsoNormal">I tried to find a requirements for glibc version in LLVM documentation but failed.<u></u><u></u></p>
<p class="MsoNormal">So I wonder whether there is such requirement or not.<u></u><u></u></p>
<p class="MsoNormal">Could anyone point me to this documentation?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I'm trying to understand whether patch is wrong which relies on availability of library but does not check the symbol itself or this version of glibc is not supported.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Thank you,<u></u><u></u></p>
<p class="MsoNormal">Serguei.<u></u><u></u></p>
<p class="MsoNormal"><span lang="RU"><u></u> <u></u></span></p>
</div>
</div>

<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div>