Disable opt_enabled attribute.

2014-06-05  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

        * config/arm/arm.md (enabled): Disable opt_enabled attribute.

From-SVN: r211256

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 8cb47dd34e769d80e1dba0e20a6350d9ee18540c..ca24af05a79601da4a3e8ad411f3d201bd07a87c 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,7 @@
+2014-06-05  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>
+
+       * config/arm/arm.md (enabled): Disable opt_enabled attribute.
+
 2014-06-05  Marek Polacek  <polacek@redhat.com>
 
        PR c/49706

diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md
index bec889a954fd46c58c89175fd61dec3bfee5e39c..f58a79bc44bd45e123f07c2efd5308bc5d987101 100644 (file)
--- a/gcc/config/arm/arm.md
+++ b/gcc/config/arm/arm.md
@@ -201,6 +201,17 @@
         (const_string "no")))
 
 ; Enable all alternatives that are both arch_enabled and insn_enabled.
+; FIXME:: opt_enabled has been temporarily removed till the time we have
+; an attribute that allows the use of such alternatives.
+; This depends on caching of speed_p, size_p on a per
+; alternative basis. The problem is that the enabled attribute
+; cannot depend on any state that is not cached or is not constant
+; for a compilation unit. We probably need a generic "hot/cold"
+; alternative which if implemented can help with this. We disable this
+; until such a time as this is implemented and / or the improvements or
+; regressions with removing this attribute are double checked.
+; See ashldi3_neon and <shift>di3_neon in neon.md.
+
  (define_attr "enabled" "no,yes"
    (cond [(and (eq_attr "predicable_short_it" "no")
               (and (eq_attr "predicated" "yes")
@@ -216,9 +227,6 @@
          (const_string "no")
 
          (eq_attr "arch_enabled" "no")
-         (const_string "no")
-
-         (eq_attr "opt_enabled" "no")
          (const_string "no")]
         (const_string "yes")))