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