<div dir="ltr">On Wed, Jul 2, 2014 at 3:07 PM, Yi Kong <span dir="ltr"><<a href="mailto:kongy.dev@gmail.com" target="_blank">kongy.dev@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Thanks for working on this. But should the intrinsics have<br>

__builtin_arm_ prefix instead of __builtin_?<br></blockquote><div><br></div><div>Don't mind me ... nothing to see here ... move along now.</div><div><br></div><div>Corrected in SVN r212253.</div><div><br></div><div>Thanks for pointing out this mistake!</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Cheers,<br>
Yi<br>
<div class=""><div class="h5"><br>
On 2 July 2014 18:58, Saleem Abdulrasool <<a href="mailto:compnerd@compnerd.org">compnerd@compnerd.org</a>> wrote:<br>
> On Mon, Jun 30, 2014 at 3:21 PM, Saleem Abdulrasool <<a href="mailto:abdulras@fb.com">abdulras@fb.com</a>> wrote:<br>
>><br>
>><br>
>> On Jun 30, 2014, at 11:06 AM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>><br>
>> > This should be doable by extending BuiltinsARM.def to use LANGBUILTIN<br>
>> > and adding LANGBUILTIN(__sevl, ..., ALL_MS_LANGUAGES).<br>
>><br>
>> Sounds good to me, Ill take a look and see if I can actually get this<br>
>> approach to work.<br>
><br>
><br>
> SVN r212196 should change the names to the __builtin_ prefixed versions.<br>
> MSVC compatibility enables the unprefixed names.<br>
><br>
> Ill try to do a follow up change where I add the aliases into arm_acle.h<br>
> along with a test to ensure that it can be included while retaining MSVC<br>
> compatibility.<br>
><br>
>><br>
>> ><br>
>> > On Mon, Jun 30, 2014 at 2:10 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>><br>
>> > wrote:<br>
>> > On 30 June 2014 10:00, Yi Kong <<a href="mailto:kongy.dev@gmail.com">kongy.dev@gmail.com</a>> wrote:<br>
>> > > I see MSVC has a long list of ARM Intrinsics, all named differently<br>
>> > > from Clang. It's not feasible to rename Clang's intrinsics to MSVC's.<br>
>> > > I think you should make __selv(and others) be MSVC specific and active<br>
>> > > only in -fms-extensions.<br>
>> ><br>
>> > I agree. OS specific stuff should be a localization issue, while ABI<br>
>> > stuff should be in the generic code.<br>
>> ><br>
>> > Yi, can you get the MSVC intrinsics into a class of their own? The<br>
>> > builtins ARM test should differentiate between Win and ABI versions,<br>
>> > as well.<br>
>> ><br>
>> > cheers,<br>
>> > --renato<br>
>> > _______________________________________________<br>
>> > cfe-commits mailing list<br>
>> > <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
>> ><br>
>> > _______________________________________________<br>
>> > cfe-commits mailing list<br>
>> > <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> ><br>
>> > <a href="https://urldefense.proofpoint.com/v1/url?u=http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits&k=ZVNjlDMF0FElm4dQtryO4A%3D%3D%0A&r=CchYc4lrV44%2BZqxZADw0BQ%3D%3D%0A&m=KVXydYXSqQatEoH80QyKFZwkyzZQVtPRzI2G0IZeBAI%3D%0A&s=4cc4ed16ac775e3b83ebaa285814697aeefd5d209ac227e1ab550457c673248f" target="_blank">https://urldefense.proofpoint.com/v1/url?u=http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits&k=ZVNjlDMF0FElm4dQtryO4A%3D%3D%0A&r=CchYc4lrV44%2BZqxZADw0BQ%3D%3D%0A&m=KVXydYXSqQatEoH80QyKFZwkyzZQVtPRzI2G0IZeBAI%3D%0A&s=4cc4ed16ac775e3b83ebaa285814697aeefd5d209ac227e1ab550457c673248f</a><br>

>><br>
>> --<br>
>> Saleem Abdulrasool<br>
>> abdulras (at) fb (dot) com<br>
>><br>
>><br>
>><br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
><br>
><br>
><br>
> --<br>
> Saleem Abdulrasool<br>
> compnerd (at) compnerd (dot) org<br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>