<div dir="ltr">We can remove them.  intrin.h pulls in ia32intrin.h.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 24, 2014 at 11:35 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Should lib/Headers/Intrin.h be updated in some way? I guess both rdtsc<br>
and rdtscp should be marked inline now, or maybe they should simply be<br>
removed from Intrin.h?<br>
<span class="HOEnZb"><font color="#888888"><br>
 - Hans<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Thu, Apr 24, 2014 at 11:26 AM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
> LGTM<br>
><br>
><br>
> On Thu, Apr 24, 2014 at 11:20 AM, Andrea Di Biagio<br>
> <<a href="mailto:andrea.dibiagio@gmail.com">andrea.dibiagio@gmail.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> This patch:<br>
>> 1. adds a definition for two new Builtinsx86:<br>
>>   __builtin_ia32_rdtsc;<br>
>>   __builtin_ia32_rdtscp.<br>
>><br>
>> 2. Replaces the already existing definition of intrinsics __rdtsc in<br>
>> ia32intrin.h with a simple call to the new GCC builtin<br>
>> '__builtin_ia32_rdtsc()'.<br>
>><br>
>> 3. Adds a definition for the new intrinsic __rdtscp in ia32intrin.h.<br>
>><br>
>> LLVM revision 207127 (<br>
>> <a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=207127" target="_blank">http://llvm.org/viewvc/llvm-project?view=revision&revision=207127</a>)<br>
>> added the two new x86 builtin intrinsics to the IR.<br>
>> This patch is simply adding the support for the two new intrinsics to<br>
>> Clang.<br>
>><br>
>> Please let me know if ok to submit.<br>
>><br>
>> Thanks,<br>
>> Andrea Di Biagio<br>
>> SN Systems - Sony Computer Entertainment Group.<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>
> 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></div>