[PATCH][X86] Add Clang support for intrinsics __rdtsc and __rdtscp.

[Hans Wennborg]

On Thu, Apr 24, 2014 at 11:38 AM, Reid Kleckner <rnk at google.com> wrote:
> We can remove them.  intrin.h pulls in ia32intrin.h.

Great. r207134


> On Thu, Apr 24, 2014 at 11:35 AM, Hans Wennborg <hans at chromium.org> wrote:
>>
>> Should lib/Headers/Intrin.h be updated in some way? I guess both rdtsc
>> and rdtscp should be marked inline now, or maybe they should simply be
>> removed from Intrin.h?
>>
>>  - Hans
>>
>> On Thu, Apr 24, 2014 at 11:26 AM, Reid Kleckner <rnk at google.com> wrote:
>> > LGTM
>> >
>> >
>> > On Thu, Apr 24, 2014 at 11:20 AM, Andrea Di Biagio
>> > <andrea.dibiagio at gmail.com> wrote:
>> >>
>> >> Hi,
>> >>
>> >> This patch:
>> >> 1. adds a definition for two new Builtinsx86:
>> >>   __builtin_ia32_rdtsc;
>> >>   __builtin_ia32_rdtscp.
>> >>
>> >> 2. Replaces the already existing definition of intrinsics __rdtsc in
>> >> ia32intrin.h with a simple call to the new GCC builtin
>> >> '__builtin_ia32_rdtsc()'.
>> >>
>> >> 3. Adds a definition for the new intrinsic __rdtscp in ia32intrin.h.
>> >>
>> >> LLVM revision 207127 (
>> >> http://llvm.org/viewvc/llvm-project?view=revision&revision=207127)
>> >> added the two new x86 builtin intrinsics to the IR.
>> >> This patch is simply adding the support for the two new intrinsics to
>> >> Clang.
>> >>
>> >> Please let me know if ok to submit.
>> >>
>> >> Thanks,
>> >> Andrea Di Biagio
>> >> SN Systems - Sony Computer Entertainment Group.
>> >>
>> >> _______________________________________________
>> >> cfe-commits mailing list
>> >> cfe-commits at cs.uiuc.edu
>> >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>> >>
>> >
>> >
>> > _______________________________________________
>> > cfe-commits mailing list
>> > cfe-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>> >
>
>