diff mbox

[AArch64] Fix type of add_losym_<mode>

Message ID 53C3EA5B.6070900@arm.com
State New
Headers show

Commit Message

Richard Earnshaw July 14, 2014, 2:34 p.m. UTC 
Noticed while browsing the code.  The add_losym_<mode> instruction takes
an immediate field as the third operand.  Hence it's type is alu_imm,
not alu_reg.

Probably doesn't have any major effect on current pipeline models, but
could do one day...

Committed to trunk.

2014-07-14  Richard Earnshaw  <rearnsha@arm.com>

	* aarch64.md (add_losym_<mode>): Set type to alu_imm.

Comments

Kyrylo Tkachov July 14, 2014, 3:24 p.m. UTC | #1 
On 14/07/14 15:34, Richard Earnshaw wrote:
> Noticed while browsing the code.  The add_losym_<mode> instruction takes
> an immediate field as the third operand.  Hence it's type is alu_imm,
> not alu_reg.
>
> Probably doesn't have any major effect on current pipeline models, but
> could do one day...
>
> Committed to trunk.
>
> 2014-07-14  Richard Earnshaw  <rearnsha@arm.com>
>
> 	* aarch64.md (add_losym_<mode>): Set type to alu_imm.
Hi Richard,

Path should be config/aarch64/aarch64.md

Kyrill
diff mbox

Patch

diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index 3eb783c..ded49a4 100644
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -3790,7 +3790,7 @@ 
 		  (match_operand 2 "aarch64_valid_symref" "S")))]
   ""
   "add\\t%<w>0, %<w>1, :lo12:%a2"
-  [(set_attr "type" "alu_reg")]
+  [(set_attr "type" "alu_imm")]
 )
 
 (define_insn "ldr_got_small_<mode>"