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

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