[8u] RFR 8130150: Implement BigInteger.montgomeryMultiply intrinsic

Andrew Haley aph at redhat.com
Mon Feb 15 14:21:58 UTC 2016


On 02/15/2016 01:06 PM, Vladimir Kempik wrote:
> Do you have a clue what else could be missing for solaris x64?
> Is asm volatile safe for MACC and MACC2 or better to keep it volatile 
> only for solaris x64?

It really shouldn't matter, as long as all the inputs, outputs, and
clobbers are correct.  If "volatile" does make a difference, we're
looking at a compiler bug.

I've looked hard at the register and clobber lists in MACC and MACC2
and AFAICS they're fine.  Maybe someone else should look too.

One thing may be relevant: the asm in the "sub" function doesn't
clobber "cc".  I think it should.

Andrew.


More information about the hotspot-compiler-dev mailing list