diff mbox

[AArch64] Make gentune.sh also generate "generic_sched" attribute

Message ID 1418812093-24406-1-git-send-email-james.greenhalgh@arm.com
State New
Headers show

Commit Message

James Greenhalgh Dec. 17, 2014, 10:28 a.m. UTC
Hi,

Reviving this thread:
  https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00143.html

Actually, this is over-engineering. We don't need the generic_sched
attribute at all. We don't yet have a generic schedule model, and when
we do get one, we have other ways of handling it through
aarch64-cores.def.

This patch removes the generic_sched attribute.

Tested by building a compiler.

OK for trunk?

Cheers,
James

---
2014-12-17  James Greenhalgh  <james.greenhalgh@arm.com>

	* config/aarch64/aarch64.md (generic_sched): Delete it.

Comments

Richard Earnshaw Dec. 17, 2014, 5:37 p.m. UTC | #1
On 17/12/14 10:28, James Greenhalgh wrote:
> 
> Hi,
> 
> Reviving this thread:
>   https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00143.html
> 
> Actually, this is over-engineering. We don't need the generic_sched
> attribute at all. We don't yet have a generic schedule model, and when
> we do get one, we have other ways of handling it through
> aarch64-cores.def.
> 
> This patch removes the generic_sched attribute.
> 
> Tested by building a compiler.
> 
> OK for trunk?
> 
> Cheers,
> James
> 
> ---
> 2014-12-17  James Greenhalgh  <james.greenhalgh@arm.com>
> 
> 	* config/aarch64/aarch64.md (generic_sched): Delete it.
> 
> 

OK.

R.
diff mbox

Patch

diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index 97d7009c20f38071f006f33978985b5e6c889794..a0ee362e391d938fd1cb6b5edb6041e8f5e5ebf2 100644
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -188,14 +188,6 @@  (define_attr "enabled" "no,yes"
 ;; Processor types.
 (include "aarch64-tune.md")
 
-;; True if the generic scheduling description should be used.
-
-(define_attr "generic_sched" "yes,no"
-  (const (if_then_else
-          (eq_attr "tune" "cortexa53,cortexa15,thunderx")
-          (const_string "no")
-          (const_string "yes"))))
-
 ;; Scheduling
 (include "../arm/cortex-a53.md")
 (include "../arm/cortex-a15.md")