On Tue, Nov 13, 2018 at 05:37:07AM -0600, Segher Boessenkool wrote: > On Tue, Nov 13, 2018 at 01:35:42PM +1030, Alan Modra wrote: > > This catches a few places where move insn patterns don't slightly > > disparage CTR, LR and VRSAVE regs. Also fixes the doc for the rs6000 > > h constraint, and removes an r->cl alternative covered by r->h. > > > > Segher okayed a patch adding "*" like this patch a long time ago. > > Somehow I never committed it. This one does a few more things as > > well, but I think it's sufficiently obvious to commit as such. > > Bootstrapped etc. powerpc64le-linux and committed rev 266044. > > Thanks. > > I'd like to move to "cl" instead of "h" though :-) > > vrsave is not an allocatable register (it's a fixed register). We do > not have to handle it in the general mov patterns at all, just needs a > separate pattern just for this.
Yes, I had the same thought, but then it wouldn't have been quite such an obvious patch, and I figure that getting there from where we are now is not worse than before the patch I committed. I'll write you a patch to get rid of "h" if it's less work for you to review my patch than it is to write it yourself. What would you like? -- Alan Modra Australia Development Lab, IBM
