On 11/30/2015 06:11 PM, Ulrich Weigand wrote: ... > However, I agree that UNSPEC_P_TO_BLK really should also get the length > as input, to make it have precisely defined semantics. Also, I'd rather > use a more descriptive name, like UNSPEC_REPLICATE_BYTE or the like. > > What would you think about something like the following? > > (define_insn "*setmem_long" > [(clobber (match_operand:<DBL> 0 "register_operand" "=d")) > (set (mem:BLK (subreg:P (match_operand:<DBL> 3 "register_operand" "0") 0)) > (unspec:BLK [(match_operand:P 2 "shift_count_or_setmem_operand" "Y") > (subreg:P (match_dup 3) 1)] UNSPEC_REPLICATE_BYTE)) > (use (match_operand:<DBL> 1 "register_operand" "d")) > (clobber (reg:CC CC_REGNUM))]
Fine with me. Thanks! Bye, -Andreas-