<div dir="ltr"><div class="gmail_quote">On Thu, May 28, 2015 at 5:09 AM İsmail Dönmez <<a href="mailto:ismail@donmez.ws">ismail@donmez.ws</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
On Thu, May 28, 2015 at 4:52 AM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> 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=aHhSLiZkEtgLOvSPOUlgE3q3QjhFovOCzJ12ymA8GnQ&s=CGu-nIxtphAdK1HSPPQQbMTZXCZlhPm7iXsUB9SBfWs&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></blockquote><div><br></div><div>Yea, I wanted to have test coverage of the default, but it doesn't work well when the default is overridden.</div><div><br></div><div>I looked into how to actually have test coverage of the default and track which default was selected, but its really hard and complex. Yuck.</div><div><br></div><div>I'll look for a way to get at least *some* coverage of the default without making it break folks that are overriding it. Sorry for the trouble.</div></div></div>