Hi Reid, Thanks for the quick review! Committed revision 207132.
On Thu, Apr 24, 2014 at 7:26 PM, Reid Kleckner <[email protected]> wrote: > LGTM > > > On Thu, Apr 24, 2014 at 11:20 AM, Andrea Di Biagio > <[email protected]> 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 >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >> > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
