From patchwork Wed Jun 25 22:24:42 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Richard Henderson X-Patchwork-Id: 1746 Received: (qmail 16116 invoked by alias); 25 Jun 2014 22:25:01 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 15972 invoked by uid 89); 25 Jun 2014 22:25:01 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.1 required=5.0 tests=AWL, BAYES_50, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, RCVD_IN_DNSWL_LOW, SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-qa0-f53.google.com X-Received: by 10.224.166.201 with SMTP id n9mr16932595qay.62.1403735097378; Wed, 25 Jun 2014 15:24:57 -0700 (PDT) From: Richard Henderson To: libc-alpha@sourceware.org Cc: azanella@linux.vnet.ibm.com, davem@davemloft.net, marcus.shawcroft@arm.com Subject: [PATCH 2/6] Removing HP_TIMING_ZERO as unused Date: Wed, 25 Jun 2014 15:24:42 -0700 Message-Id: <1403735086-21797-3-git-send-email-rth@twiddle.net> In-Reply-To: <1403735086-21797-1-git-send-email-rth@twiddle.net> References: <1403735086-21797-1-git-send-email-rth@twiddle.net> * sysdeps/alpha/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/generic/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/i386/i686/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/ia64/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/powerpc/powerpc32/power4/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/powerpc/powerpc64/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/sparc/sparc32/sparcv9/hp-timing.h (HP_TIMING_ZERO): Remove. * sysdeps/sparc/sparc64/hp-timing.h (HP_TIMING_ZERO): Remove. --- sysdeps/alpha/hp-timing.h | 5 ----- sysdeps/generic/hp-timing.h | 3 --- sysdeps/i386/i686/hp-timing.h | 5 ----- sysdeps/ia64/hp-timing.h | 6 ------ sysdeps/powerpc/powerpc32/power4/hp-timing.h | 5 ----- sysdeps/powerpc/powerpc64/hp-timing.h | 5 ----- sysdeps/sparc/sparc32/sparcv9/hp-timing.h | 2 -- sysdeps/sparc/sparc64/hp-timing.h | 2 -- 8 files changed, 33 deletions(-) diff --git a/sysdeps/alpha/hp-timing.h b/sysdeps/alpha/hp-timing.h index 90f9b9d..4cbee66 100644 --- a/sysdeps/alpha/hp-timing.h +++ b/sysdeps/alpha/hp-timing.h @@ -42,8 +42,6 @@ - hp_timing_t: This is the type for variables used to store the time values. - - HP_TIMING_ZERO: clear `hp_timing_t' object. - - HP_TIMING_NOW: place timestamp for current time in variable given as parameter. @@ -77,9 +75,6 @@ /* We use 32 bit values for the times. */ typedef unsigned int hp_timing_t; -/* Set timestamp value to zero. */ -#define HP_TIMING_ZERO(VAR) (VAR) = (0) - /* The "rpcc" instruction returns a 32-bit counting half and a 32-bit "virtual cycle counter displacement". Subtracting the two gives us a virtual cycle count. */ diff --git a/sysdeps/generic/hp-timing.h b/sysdeps/generic/hp-timing.h index eddc971..bf78e4e 100644 --- a/sysdeps/generic/hp-timing.h +++ b/sysdeps/generic/hp-timing.h @@ -39,8 +39,6 @@ - hp_timing_t: This is the type for variables used to store the time values. - - HP_TIMING_ZERO: clear `hp_timing_t' object. - - HP_TIMING_NOW: place timestamp for current time in variable given as parameter. @@ -68,7 +66,6 @@ #define HP_TIMING_AVAIL (0) #define HP_TIMING_INLINE (0) typedef int hp_timing_t; -#define HP_TIMING_ZERO(Var) #define HP_TIMING_NOW(var) #define HP_TIMING_DIFF_INIT() #define HP_TIMING_DIFF(Diff, Start, End) diff --git a/sysdeps/i386/i686/hp-timing.h b/sysdeps/i386/i686/hp-timing.h index 4a2006e..18e5b95 100644 --- a/sysdeps/i386/i686/hp-timing.h +++ b/sysdeps/i386/i686/hp-timing.h @@ -49,8 +49,6 @@ - hp_timing_t: This is the type for variables used to store the time values. - - HP_TIMING_ZERO: clear `hp_timing_t' object. - - HP_TIMING_NOW: place timestamp for current time in variable given as parameter. @@ -83,9 +81,6 @@ /* We use 64bit values for the times. */ typedef unsigned long long int hp_timing_t; -/* Set timestamp value to zero. */ -#define HP_TIMING_ZERO(Var) (Var) = (0) - /* That's quite simple. Use the `rdtsc' instruction. Note that the value 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 diff --git a/sysdeps/ia64/hp-timing.h b/sysdeps/ia64/hp-timing.h index bf97b47..e91326c 100644 --- a/sysdeps/ia64/hp-timing.h +++ b/sysdeps/ia64/hp-timing.h @@ -43,8 +43,6 @@ - hp_timing_t: This is the type for variables used to store the time values. - - HP_TIMING_ZERO: clear `hp_timing_t' object. - - HP_TIMING_NOW: place timestamp for current time in variable given as parameter. @@ -77,10 +75,6 @@ /* We use 64bit values for the times. */ typedef unsigned long int hp_timing_t; -/* Set timestamp value to zero. */ -#define HP_TIMING_ZERO(Var) (Var) = (0) - - /* The Itanium/Merced has a bug where the ar.itc register value read is not correct in some situations. The solution is to read again. For now we always do this until we know how to recognize a fixed diff --git a/sysdeps/powerpc/powerpc32/power4/hp-timing.h b/sysdeps/powerpc/powerpc32/power4/hp-timing.h index f1e3beb..2124e02 100644 --- a/sysdeps/powerpc/powerpc32/power4/hp-timing.h +++ b/sysdeps/powerpc/powerpc32/power4/hp-timing.h @@ -42,8 +42,6 @@ - hp_timing_t: This is the type for variables used to store the time values. - - HP_TIMING_ZERO: clear `hp_timing_t' object. - - HP_TIMING_NOW: place timestamp for current time in variable given as parameter. @@ -76,9 +74,6 @@ /* We use 64bit values for the times. */ typedef unsigned long long int hp_timing_t; -/* Set timestamp value to zero. */ -#define HP_TIMING_ZERO(Var) (Var) = (0) - /* That's quite simple. Use the `mftb' instruction. Note that the value 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 diff --git a/sysdeps/powerpc/powerpc64/hp-timing.h b/sysdeps/powerpc/powerpc64/hp-timing.h index f1efa12..c773541 100644 --- a/sysdeps/powerpc/powerpc64/hp-timing.h +++ b/sysdeps/powerpc/powerpc64/hp-timing.h @@ -42,8 +42,6 @@ - hp_timing_t: This is the type for variables used to store the time values. - - HP_TIMING_ZERO: clear `hp_timing_t' object. - - HP_TIMING_NOW: place timestamp for current time in variable given as parameter. @@ -76,9 +74,6 @@ /* We use 64bit values for the times. */ typedef unsigned long long int hp_timing_t; -/* Set timestamp value to zero. */ -#define HP_TIMING_ZERO(Var) (Var) = (0) - /* That's quite simple. Use the `mftb' instruction. Note that the value 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 diff --git a/sysdeps/sparc/sparc32/sparcv9/hp-timing.h b/sysdeps/sparc/sparc32/sparcv9/hp-timing.h index fd7e76e..fbfb861 100644 --- a/sysdeps/sparc/sparc32/sparcv9/hp-timing.h +++ b/sysdeps/sparc/sparc32/sparcv9/hp-timing.h @@ -29,8 +29,6 @@ typedef unsigned long long int hp_timing_t; -#define HP_TIMING_ZERO(Var) (Var) = (0) - #define HP_TIMING_NOW(Var) \ __asm__ __volatile__ ("rd %%tick, %L0\n\t" \ "srlx %L0, 32, %H0" \ diff --git a/sysdeps/sparc/sparc64/hp-timing.h b/sysdeps/sparc/sparc64/hp-timing.h index fa08cc8..0773efb 100644 --- a/sysdeps/sparc/sparc64/hp-timing.h +++ b/sysdeps/sparc/sparc64/hp-timing.h @@ -29,8 +29,6 @@ typedef unsigned long int hp_timing_t; -#define HP_TIMING_ZERO(Var) (Var) = (0) - #define HP_TIMING_NOW(Var) __asm__ __volatile__ ("rd %%tick, %0" : "=r" (Var)) #define HP_TIMING_DIFF_INIT() \