Request for review (XS): 6893554: SPECjvm2008 mpegaudio fails with SecurityException

Christian Thalinger Christian.Thalinger at Sun.COM
Fri Oct 23 10:10:21 PDT 2009

On Fri, 2009-10-23 at 09:45 -0700, Vladimir Kozlov wrote:
> Christian,
> Thank you for finding the cause so fast.
> The fix looks good. I am only concern about
> using #ifdef inside ins_encode and format.
> Could you verify the generated code for both
> 32- and 64-bit VM?

I don't like it either.  We could use the 64-bit code for 32-bit too,
but it's one (useless) instruction more.

The generated code with the patch is OK for 32- and 64-bit.  I tested
that with my initial testcase.

That leads me to another question: as I would have noticed the bug when
I wouldn't have forgotten to actually run the testcase on 64-bit SPARC,
aren't we running the compiler tests somewhere on a nightly basis?

-- Christian

More information about the hotspot-compiler-dev mailing list