Segher Boessenkool <seg...@kernel.crashing.org> writes:
> Hi!
>
> On Tue, Sep 06, 2022 at 08:27:56PM +0800, Jiufu Guo wrote:
>> + if (can_create_pseudo_p ())
>> + {
>> + /* pli A,H + pli B,L + rldimi A,B,32,0. */
>
> A is the low one and B is the high one (the one that needs to be
> shifted). rl[wd]imi is one of our very few insns that has the output
> register as one of the input registers as well. "Good" to see not just
> the compiler has problems with this, but humans do as well! ;-)
Yes, A is the low part and B is the high.
Thanks for catch this!
>
> Otherwise looks good to me. Okay for trunk with that fixed somehow.
> Thanks!
Thanks again!
BR,
Jeff(Jiufu)
>
>
> Segher
