This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Add future.md placeholder to PowerPC


Hi!

On Thu, Aug 01, 2019 at 03:55:09PM -0400, Michael Meissner wrote:
> Add future.md place holder to PowerPC.
> 
> This patch creates future.md.  Right now it is a place holder, and it takes the
> power9.md, and changes the names to be future, and the cpu to be future.
> 
> It is anticipated that in the future, this file will be updated to match the
> scheduling information for a possible 'future' machine.

> During our testing, we
> discovered that the code scheduled is somewhat whacky if you don't have
> automata that matches the machine.

This is the important part, what makes this patch useful now.

> 2019-08-01  Michael Meissner  <meissner@linux.ibm.com>
> 
> 	* config/rs6000/future.md: New file.
> 	* config/rs6000/rs6000.md (top level): Include future.md.

Just say
	* config/rs6000/rs6000.md: Include future.md.

> 	* config/rs6000/t-rs6000 (MD_INCLUDES): Add future.md

Period?

> --- gcc/config/rs6000/future.md	(revision 0)
> +++ gcc/config/rs6000/future.md	(working copy)
> @@ -0,0 +1,519 @@
> +;; Scheduling description for IBM POWER9 processor.
> +;; Copyright (C) 2016-2019 Free Software Foundation, Inc.
> +;;
> +;; Contributed by Pat Haugen (pthaugen@us.ibm.com).

Please change this, or clarify it, saying it is a copy of power9.md,
and that it is a placeholder.

Okay for trunk with those changes.  Thanks!


Segher


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]