>>> On 27.06.19 at 12:20, <ubiz...@gmail.com> wrote: > On Thu, Jun 27, 2019 at 10:57 AM Jan Beulich <jbeul...@suse.com> wrote: >> >> - the affine transformations are not commutative (the two source >> operands have entirely different meaning) >> - there's no need for three alternatives >> - the nonimmediate_operand/Bm combination can better be vector_operand/m >> >> gcc/ >> 2019-06-27 Jan Beulich <jbeul...@suse.com> >> >> * config/i386/sse.md (vgf2p8affineinvqb_<mode><mask_name>, >> vgf2p8affineqb_<mode><mask_name>): Drop % constraint modifier. >> Eliminate redundant alternative. Use vector_operand plus "m" >> constraint. > > Please just drop % modifier and use vector_operand here. IIRC, > register allocator operates on constraints, it doesn't care for > predicates. But predicates shouldn't be more constrained than > constraints. So, having "m" instead of "Bm" is a bad idea with > vector_operand.
Well, putting back the Bm is easy (if that's really needed). But do you also mean me to put back to redundant 3rd alternative? Jan
