On 11/16/2016 08:22 PM, Aaron Sawdey wrote:
I've also changed the pattern to indicate that operand 3 may be clobbered (if it happens to be in cx already).
That part looks like a meaningless change since the pattern in that define_expand is never used (there's a DONE at the end).
I don't think this is really a concern anyway (the pattern checks for CX in fixed_regs) and I suspect we wouldn't get a hard register in this expander in other cases.
Other than that it looks OK to me, please wait a day or two to see if Uros has objections.
Bernd
