[9] RFR(S): 8075136: Unnecessary sign extension for byte array access

Andrew Haley aph at redhat.com
Wed Mar 18 09:37:24 UTC 2015


On 17/03/15 16:32, Andrew Dinn wrote:
> Doh, apologies -- I'm afraid I am still one step behind!
> 
> The two new rules both specify an unscaled 32-bit integer register as
> index and, of course, the scale value is installed as 0 in the rule. So,
> the correct patch to function storeLoad is to include these two switch
> cases in the set which specify a sign-extend:

Hi Andrew,

I've been trying to follow this and I don't understand what you're
asking Tobias to do.  Can you please let us know what you think his
patches should be like?

Thanks,
Andrew.


More information about the hotspot-compiler-dev mailing list