On Fri, May 25, 2018 at 12:07:34PM +0800, wei.guo.si...@gmail.com wrote:
> +     /* save and restore cr0 */
> +     mfocrf  r5,64
> +     EXIT_VMX_OPS
> +     mtocrf  64,r5
> +     b       .LcmpAB_lightweight

That's cr1, not cr0.  You can use mcrf instead, it is cheaper (esp. if
you have it in a non-volatile CR field before so you need only one, if any).

> +     vcmpequb.  v7,v9,v10
> +     bnl     cr6,.Ldiffoffset_vmx_diff_found

In other places you say  bf 24,...  Dunno which is more readable, but
please pick one?


Segher

Reply via email to