RFA: 8074349: AARCH64: C2 generates poor code for some byte and character stores

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Mar 3 18:16:32 UTC 2015


Looks good.

Since it is platform specific code new rule allow you to push changes yourself.

Make sure you have jcheck enabled - ~/.hgrc has

[extensions]
jcheck=/<your path to jcheck>/jcheck.py
[hooks]
pretxnchangegroup=python:jcheck.hook
pretxncommit=python:jcheck.hook

See http://openjdk.java.net/projects/code-tools/jcheck/

And, please, make sure changeset has correct header:

8074349: AARCH64: C2 generates poor code for some byte and character stores
Summary: Use iRegIorL2I as the source operand for all stores of int size or less.
Reviewed-by: kvn

See http://openjdk.java.net/guide/producingChangeset.html

Thanks,
Vladimir

On 3/3/15 10:01 AM, Andrew Haley wrote:
> We should use iRegIorL2I as the source operand for all stores of int
> size or less.
>
> http://cr.openjdk.java.net/~aph/8074349/
>
> Andrew.
>


More information about the hotspot-dev mailing list