public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Turnoff prefetching for -march=znver1
@ 2016-02-02 20:28 Stepanyan, Victoria
  2016-02-02 20:49 ` Uros Bizjak
  0 siblings, 1 reply; 3+ messages in thread
From: Stepanyan, Victoria @ 2016-02-02 20:28 UTC (permalink / raw)
  To: 'gcc-patches@gcc.gnu.org'
  Cc: 'gerald@pfeifer.com', 'ubizjak@gmail.com',
	'rguenther@suse.de',
	Kumar, Venkataramanan

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

Hi Maintainers,

This patch disables prefetching for -march=znver1 which is turned on by default.

gcc/ChangeLog:

2016-02-02 Victoria Stepanyan <victoria.stepanyan@amd.com>

	* gcc/config/i386/x86-tune.def: Disable default prefetching for -march=znver1

Ok for trunk?

Victoria


[-- Attachment #2: turnoff_prefetch.patch.txt --]
[-- Type: text/plain, Size: 580 bytes --]

--- a/gcc/config/i386/x86-tune.def
+++ b/gcc/config/i386/x86-tune.def
@@ -218,7 +218,7 @@ DEF_TUNE (X86_TUNE_FOUR_JUMP_LIMIT, "four_jump_limit",
    at -O3.  For the moment, the prefetching seems badly tuned for Intel
    chips.  */
DEF_TUNE (X86_TUNE_SOFTWARE_PREFETCHING_BENEFICIAL, "software_prefetching_beneficial",
-          m_K6_GEODE | m_AMD_MULTIPLE)
+          m_K6_GEODE | m_ATHLON_K8 | m_AMDFAM10 | m_BDVER | m_BTVER)

/* X86_TUNE_LCP_STALL: Avoid an expensive length-changing prefix stall
    on 16-bit immediate moves into memory on Core2 and Corei7.  */

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Turnoff prefetching for -march=znver1
  2016-02-02 20:28 Turnoff prefetching for -march=znver1 Stepanyan, Victoria
@ 2016-02-02 20:49 ` Uros Bizjak
  2016-02-04  7:55   ` Kumar, Venkataramanan
  0 siblings, 1 reply; 3+ messages in thread
From: Uros Bizjak @ 2016-02-02 20:49 UTC (permalink / raw)
  To: Stepanyan, Victoria; +Cc: gcc-patches, gerald, rguenther, Kumar, Venkataramanan

On Tue, Feb 2, 2016 at 9:28 PM, Stepanyan, Victoria
<Victoria.Stepanyan@amd.com> wrote:
> Hi Maintainers,
>
> This patch disables prefetching for -march=znver1 which is turned on by default.
>
> gcc/ChangeLog:
>
> 2016-02-02 Victoria Stepanyan <victoria.stepanyan@amd.com>
>
>         * gcc/config/i386/x86-tune.def: Disable default prefetching for -march=znver1
>
> Ok for trunk?

OK.

Thanks,
Uros.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* RE: Turnoff prefetching for -march=znver1
  2016-02-02 20:49 ` Uros Bizjak
@ 2016-02-04  7:55   ` Kumar, Venkataramanan
  0 siblings, 0 replies; 3+ messages in thread
From: Kumar, Venkataramanan @ 2016-02-04  7:55 UTC (permalink / raw)
  To: Uros Bizjak, Stepanyan, Victoria; +Cc: gcc-patches, gerald, rguenther

Hi Uros,

> -----Original Message-----
> From: Uros Bizjak [mailto:ubizjak@gmail.com]
> Sent: Wednesday, February 03, 2016 2:20 AM
> To: Stepanyan, Victoria
> Cc: gcc-patches@gcc.gnu.org; gerald@pfeifer.com; rguenther@suse.de;
> Kumar, Venkataramanan
> Subject: Re: Turnoff prefetching for -march=znver1
> 
> On Tue, Feb 2, 2016 at 9:28 PM, Stepanyan, Victoria
> <Victoria.Stepanyan@amd.com> wrote:
> > Hi Maintainers,
> >
> > This patch disables prefetching for -march=znver1 which is turned on by
> default.
> >
> > gcc/ChangeLog:
> >
> > 2016-02-02 Victoria Stepanyan <victoria.stepanyan@amd.com>
> >
> >         * gcc/config/i386/x86-tune.def: Disable default prefetching for -
> march=znver1
> >
> > Ok for trunk?
> 
> OK.

Thank you. I up streamed it on behalf of Victoria.
https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=233127


> 
> Thanks,
> Uros.

Regards,
Venkat.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-02-04  7:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-02 20:28 Turnoff prefetching for -march=znver1 Stepanyan, Victoria
2016-02-02 20:49 ` Uros Bizjak
2016-02-04  7:55   ` Kumar, Venkataramanan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).