Message ID | 1282903951.23926.25.camel@e102325-lin.cambridge.arm.com |
---|---|
State | New |
Headers | show |
On 08/27/2010 12:12 PM, Ramana Radhakrishnan wrote: > Verified that stage2 proceeds further with this. Committing as obvious > in the next few minutes unless someone has any objections to this. > (define_attr "enabled" "no,yes" > (if_then_else (eq_attr "insn_enabled" "yes") > - (attr "arch_enabled") > + (if_then_else (eq_attr "arch_enabled" "yes") > + (const_string "yes") > + (const_string "no")) No real objection, but IMO it would be nicer to make this work in the generator programs. Bernd
Index: trunk/gcc/config/arm/arm.md =================================================================== --- trunk/gcc/config/arm/arm.md (revision 163581) +++ trunk/gcc/config/arm/arm.md (working copy) @@ -236,7 +236,9 @@ ; Enable all alternatives that are both arch_enabled and insn_enabled. (define_attr "enabled" "no,yes" (if_then_else (eq_attr "insn_enabled" "yes") - (attr "arch_enabled") + (if_then_else (eq_attr "arch_enabled" "yes") + (const_string "yes") + (const_string "no")) (const_string "no"))) ; POOL_RANGE is how far away from a constant pool entry that this insn