From 2dd8e58cc533ee840d37725b11bc0dc0308a5dc0 Mon Sep 17 00:00:00 2001 From: "H.J. Lu" Date: Sun, 21 Oct 2018 00:37:11 -0700 Subject: x86: Don't include Use __builtin_ia32_rdtsc directly since including makes building glibc very slow. On Intel Core i5-6260U, this patch reduces x86-64 build time from 8 minutes 33 seconds to 3 minutes 48 seconds with "make -j4" and GCC 8.2.1. * sysdeps/x86/hp-timing.h: Don't include . (HP_TIMING_NOW): Replace _rdtsc with __builtin_ia32_rdtsc. --- ChangeLog | 5 +++++ sysdeps/x86/hp-timing.h | 9 +++++---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index eb1b0dbb1c..c3fbad184e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +2018-10-21 H.J. Lu + + * sysdeps/x86/hp-timing.h: Don't include . + (HP_TIMING_NOW): Replace _rdtsc with __builtin_ia32_rdtsc. + 2018-10-19 Joseph Myers [BZ #23794] diff --git a/sysdeps/x86/hp-timing.h b/sysdeps/x86/hp-timing.h index 1c20e9d828..77a1360748 100644 --- a/sysdeps/x86/hp-timing.h +++ b/sysdeps/x86/hp-timing.h @@ -22,8 +22,6 @@ #include #if MINIMUM_ISA == 686 || MINIMUM_ISA == 8664 -# include - /* We always assume having the timestamp register. */ # define HP_TIMING_AVAIL (1) # define HP_SMALL_TIMING_AVAIL (1) @@ -38,8 +36,11 @@ typedef unsigned long long int hp_timing_t; might not be 100% accurate since there might be some more instructions running in this moment. This could be changed by using a barrier like 'cpuid' right before the `rdtsc' instruciton. But we are not interested - in accurate clock cycles here so we don't do this. */ -# define HP_TIMING_NOW(Var) ((Var) = _rdtsc ()) + in accurate clock cycles here so we don't do this. + + NB: Use __builtin_ia32_rdtsc directly since including + makes building glibc very slow. */ +# define HP_TIMING_NOW(Var) ((Var) = __builtin_ia32_rdtsc ()) # include #else -- cgit 1.4.1