[PATCH] Tune for lakemont

[PATCH] Tune for lakemont

[Yulia Koval]

[-- Attachment #1: Type: text/plain, Size: 299 bytes --]

Hi,

The patch below contains some tuning changes for Lakemont, introduced
by H.J. Lu. Bootstraped/regtested for Linux/x86_64. Ok for trunk?

        * gcc/config/i386/x86-tune.def (X86_TUNE_USE_BT): Enable
        for Lakemont.
        (X86_TUNE_ZERO_EXTEND_WITH_AND): Disable for Lakemont.

Julia

[-- Attachment #2: 0001-gcc-config-i386-x86-tune.def-X86_TUNE_USE_BT-Enable.patch --]
[-- Type: application/octet-stream, Size: 1473 bytes --]

From d83c2d6943c148a1f6eb5460e347d904c4aad073 Mon Sep 17 00:00:00 2001
From: "H.J. Lu" <hjl.tools@gmail.com>
Date: Thu, 9 Jul 2015 18:42:45 -0700
Subject: [PATCH] 	* gcc/config/i386/x86-tune.def (X86_TUNE_USE_BT):
 Enable 	 for Lakemont. 	(X86_TUNE_ZERO_EXTEND_WITH_AND): Disable for
 Lakemont.

---
 gcc/config/i386/x86-tune.def | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gcc/config/i386/x86-tune.def b/gcc/config/i386/x86-tune.def
index 3b3f149..213c604 100644
--- a/gcc/config/i386/x86-tune.def
+++ b/gcc/config/i386/x86-tune.def
@@ -292,7 +292,7 @@ DEF_TUNE (X86_TUNE_USE_CLTD, "use_cltd",
 /* X86_TUNE_USE_BT: Enable use of BT (bit test) instructions.  */
 DEF_TUNE (X86_TUNE_USE_BT, "use_bt",
           m_CORE_ALL | m_BONNELL | m_SILVERMONT | m_KNL | m_INTEL
-	  | m_AMD_MULTIPLE | m_GENERIC)
+	  | m_LAKEMONT | m_AMD_MULTIPLE | m_GENERIC)
 
 /*****************************************************************************/
 /* 387 instruction selection tuning                                          */
@@ -448,7 +448,7 @@ DEF_TUNE (X86_TUNE_SHIFT1, "shift1", ~m_486)
 /* X86_TUNE_ZERO_EXTEND_WITH_AND: Use AND instruction instead
    of mozbl/movwl.  */
 DEF_TUNE (X86_TUNE_ZERO_EXTEND_WITH_AND, "zero_extend_with_and",
-	  m_486 | m_PENT | m_LAKEMONT)
+	  m_486 | m_PENT)
 
 /* X86_TUNE_PROMOTE_HIMODE_IMUL: Modern CPUs have same latency for HImode
    and SImode multiply, but 386 and 486 do HImode multiply faster.  */
-- 
1.8.3.1

Re: [PATCH] Tune for lakemont

[Uros Bizjak]

On Thu, Oct 1, 2015 at 2:37 PM, Yulia Koval <vaalfreja@gmail.com> wrote:
> Hi,
>
> The patch below contains some tuning changes for Lakemont, introduced
> by H.J. Lu. Bootstraped/regtested for Linux/x86_64. Ok for trunk?
>
>         * gcc/config/i386/x86-tune.def (X86_TUNE_USE_BT): Enable
>         for Lakemont.
>         (X86_TUNE_ZERO_EXTEND_WITH_AND): Disable for Lakemont.

Non-algorithmic tuning of various parameters, especially the ones in
x86-tune.def, is always OK and pre-approved as an "obvious" patch.

Thanks,
Uros.

Re: [PATCH] Tune for lakemont

[H.J. Lu]

On Thu, Oct 1, 2015 at 5:42 AM, Uros Bizjak <ubizjak@gmail.com> wrote:
> On Thu, Oct 1, 2015 at 2:37 PM, Yulia Koval <vaalfreja@gmail.com> wrote:
>> Hi,
>>
>> The patch below contains some tuning changes for Lakemont, introduced
>> by H.J. Lu. Bootstraped/regtested for Linux/x86_64. Ok for trunk?
>>
>>         * gcc/config/i386/x86-tune.def (X86_TUNE_USE_BT): Enable
>>         for Lakemont.
>>         (X86_TUNE_ZERO_EXTEND_WITH_AND): Disable for Lakemont.
>
> Non-algorithmic tuning of various parameters, especially the ones in
> x86-tune.def, is always OK and pre-approved as an "obvious" patch.
>

Checked in.

Thanks.

-- 
H.J.