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