<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 9, 2015 at 4:32 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On Tue, Jun 9, 2015 at 3:32 AM, Daniel Sanders <span dir="ltr"><<a href="mailto:Daniel.Sanders@imgtec.com" target="_blank">Daniel.Sanders@imgtec.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
I'm not suitable to review the change but there's a couple things I can comment on to help you get your change reviewed.<br>
<span><br>
> That way tgmath can be used with, say, VS 2010, which is the one I am using,<br>
> so far without any problem.<br>
<br>
</span>I believe there was a policy decision to support only the last two major releases of GCC, Clang, and Visual Studio and our getting started page (<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_GettingStarted.html-23host-2Dc-2Dtoolchain-2Dboth-2Dcompiler-2Dand-2Dstandard-2Dlibrary&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=N09CH6gsZxeWTz0zwHeswOqrcFCouG-01F8Pfzm6HfY&s=VzSINQr_ZQtepyw1SfoHtE_z6J7Aqwn39BK6IoRN2xw&e=" target="_blank">http://llvm.org/docs/GettingStarted.html#host-c-toolchain-both-compiler-and-standard-library</a>) lists Visual Studio 2013 as being the minimum supported version. At the time the decision was made, this was necessary to be able to migrate to C++11.<br></blockquote><div><br></div></span><div>The tgmath.h in clang/lib/Headers/tgmath.h is one of clang's builtin headers. They are used while compiling users' code, not while compiling clang itself. Clang is the only compiler that is ever intended to see these headers.</div></div></div></div></blockquote><div><br></div><div>Yes, the configuration is clang targeting the MSVC 2010 runtime, which is supported relatively well. The MSVC 2010 runtime is lacking some math functions. I expect Javier will send along a diff which will make it easier to identify the issue.</div></div></div></div>