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  <meiss...@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 (pthau...@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

Reply via email to