<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">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 class=""><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><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><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> If nobody spots a bug in the following days I may upload the file, is it right?<br>
<br>
</span>Generally speaking, you'll need to wait for someone to review your patch and approve it. You'll need to send a patch to cfe-commits rather than attach your modified file though. At the moment, it's difficult to tell what changes you made.<br>
<br>
Please see <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_DeveloperPolicy.html-23making-2Dand-2Dsubmitting-2Da-2Dpatch&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=N09CH6gsZxeWTz0zwHeswOqrcFCouG-01F8Pfzm6HfY&s=CybvOxVITKfxTbvvwSCOBDNTWv7NR5JUhTBi66JiFzE&e=" target="_blank">http://llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch</a> and <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_docs_DeveloperPolicy.html-23code-2Dreviews&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=CnzuN65ENJ1H9py9XLiRvC_UQz6u3oG6GUNn7_wosSM&m=N09CH6gsZxeWTz0zwHeswOqrcFCouG-01F8Pfzm6HfY&s=CfSB2RJqp3LSXe3TZ2nkhOeypSYvJa1DUDokR9Q1-MM&e=" target="_blank">http://llvm.org/docs/DeveloperPolicy.html#code-reviews</a> for more information.<br>
<br>
Hope that helps<br>
<div><div class="h5"><br>
> -----Original Message-----<br>
> From: <a href="mailto:cfe-dev-bounces@cs.uiuc.edu">cfe-dev-bounces@cs.uiuc.edu</a> [mailto:<a href="mailto:cfe-dev-bounces@cs.uiuc.edu">cfe-dev-bounces@cs.uiuc.edu</a>]<br>
> On Behalf Of Javier Múgica<br>
> Sent: 09 June 2015 09:14<br>
> To: Clang development community<br>
> Subject: [cfe-dev] tgmath.h<br>
><br>
> Hello:<br>
><br>
> I have made some changes to tgmath.h. There could be seen repetitive<br>
> definitions, so I defined some macros in order to add some structure there<br>
> and make the file smaller and, in my opinion, more readable. I also added a<br>
> test to see if the implementatinos suports complex numbers and surrounded<br>
> the code that uses those types under the corresponding #ifdef #endif.<br>
><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>
> If nobody spots a bug in the following days I may upload the file, is it right?<br>
><br>
> Regards,<br>
><br>
> Javier A. Múgica<br>
<br>
</div></div>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
</blockquote></div><br></div></div>