+2014-03-11 Joern Rennecke <joern.rennecke@embecosm.com>
+
+ * config/arc/predicates.md (extend_operand): During/after reload,
+ allow const_int_operand.
+ * config/arc/arc.md (mulsidi3_700): Use extend_operand predicate.
+ (umulsidi3_700): Likewise. Change operand 2 constraint back to "cL".
+ (mulsi3_highpart): Change operand 2 constraint alternatives 2 and 3
+ to "i".
+ (umulsi3_highpart_i): Likewise.
+
2014-03-11 Richard Biener <rguenther@suse.de>
* tree-ssa-structalias.c (get_constraint_for_ptr_offset):
(define_insn_and_split "mulsidi3_700"
[(set (match_operand:DI 0 "register_operand" "=&r")
(mult:DI (sign_extend:DI (match_operand:SI 1 "register_operand" "%c"))
- (sign_extend:DI (match_operand:SI 2 "register_operand" "cL"))))]
+ (sign_extend:DI (match_operand:SI 2 "extend_operand" "cL"))))]
"TARGET_ARC700 && !TARGET_NOMPY_SET"
"#"
"&& reload_completed"
(lshiftrt:DI
(mult:DI
(sign_extend:DI (match_operand:SI 1 "register_operand" "%0,c, 0,c"))
- (sign_extend:DI (match_operand:SI 2 "extend_operand" "c,c, s,s")))
+ (sign_extend:DI (match_operand:SI 2 "extend_operand" "c,c, i,i")))
(const_int 32))))]
"TARGET_ARC700 && !TARGET_NOMPY_SET"
"mpyh%? %0,%1,%2"
(lshiftrt:DI
(mult:DI
(zero_extend:DI (match_operand:SI 1 "register_operand" "%0,c, 0,c"))
- (zero_extend:DI (match_operand:SI 2 "extend_operand" "c,c, s,s")))
+ (zero_extend:DI (match_operand:SI 2 "extend_operand" "c,c, i,i")))
(const_int 32))))]
"TARGET_ARC700 && !TARGET_NOMPY_SET"
"mpyhu%? %0,%1,%2"
(define_insn_and_split "umulsidi3_700"
[(set (match_operand:DI 0 "dest_reg_operand" "=&r")
(mult:DI (zero_extend:DI (match_operand:SI 1 "register_operand" "%c"))
- (zero_extend:DI (match_operand:SI 2 "register_operand" "c"))))]
-;; (zero_extend:DI (match_operand:SI 2 "register_operand" "rL"))))]
+ (zero_extend:DI (match_operand:SI 2 "extend_operand" "cL"))))]
"TARGET_ARC700 && !TARGET_NOMPY_SET"
"#"
"reload_completed"
(and (match_code "reg")
(match_test "REGNO (op) == (TARGET_BIG_ENDIAN ? 58 : 59)")))
+; Unfortunately, we can not allow a const_int_operand before reload, because
+; reload needs a non-void mode to guide it how to reload the inside of a
+; {sign_}extend.
(define_predicate "extend_operand"
- (ior (match_test "register_operand (op, mode)")
- (and (match_test "immediate_operand (op, mode)")
- (not (match_test "const_int_operand (op, mode)")))))
+ (ior (match_operand 0 "register_operand")
+ (and (match_operand 0 "immediate_operand")
+ (ior (not (match_operand 0 "const_int_operand"))
+ (match_test "reload_in_progress || reload_completed")))))
(define_predicate "millicode_store_operation"
(match_code "parallel")