<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, May 29, 2015 at 5:56 AM, İsmail Dönmez <span dir="ltr"><<a href="mailto:ismail@donmez.ws" target="_blank">ismail@donmez.ws</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, May 28, 2015 at 11:48 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
> On Thu, May 28, 2015 at 5:09 AM İsmail Dönmez <<a href="mailto:ismail@donmez.ws">ismail@donmez.ws</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> On Thu, May 28, 2015 at 4:52 AM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>><br>
>> wrote:<br>
>> > Author: chandlerc<br>
>> > Date: Wed May 27 20:52:38 2015<br>
>> > New Revision: 238389<br>
>> ><br>
>> > URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D238389-26view-3Drev&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=m-2c6Zn9tpU6HkCxYclvcn3zWpwMk0Venl12SZrxlCU&s=YuP7Is_VUAoBcJMHpqrO3u4Ngv3XkFs7N417mjAPMm8&e=" target="_blank">http://llvm.org/viewvc/llvm-project?rev=238389&view=rev</a><br>
>> > Log:<br>
>> > [omp] Re-work Clang's handling of -fopenmp and undo r237769.<br>
>><br>
>><br>
>> This seems to break openmp tests with<br>
>> -DCLANG_DEFAULT_OPENMP_RUNTIME=libiomp5 see:<br>
><br>
><br>
> Yea, I wanted to have test coverage of the default, but it doesn't work well<br>
> when the default is overridden.<br>
><br>
> I looked into how to actually have test coverage of the default and track<br>
> which default was selected, but its really hard and complex. Yuck.<br>
><br>
> I'll look for a way to get at least *some* coverage of the default without<br>
> making it break folks that are overriding it. Sorry for the trouble.<br>
<br>
</div></div>Tests pass now but there is still one grave problem. With<br>
CLANG_DEFAULT_OPENMP_RUNTIME=libiomp5 if you do:<br>
<br>
λ clang -fopenmp=libgomp omp_hello.c<br>
λ ./a.out<br>
Hello World from thread = 0<br>
Number of threads = 1<br>
<br>
No OpenMP, lets go back to defaults:<br>
<br>
λ clang -fopenmp=libiomp5 omp_hello.c<br>
λ ./a.out<br>
Hello World from thread = 0<br>
Hello World from thread = 4<br>
Hello World from thread = 3<br>
Hello World from thread = 2<br>
Hello World from thread = 6<br>
Hello World from thread = 1<br>
Hello World from thread = 5<br>
Hello World from thread = 7<br>
Number of threads = 8<br></blockquote></div><br></div><div class="gmail_extra">OK, and what's the one grave problem?</div></div>