[aarch64-port-dev ] RFR: aarch64: 8133842: C2 generates illegal instructions with int shifts >=32
Edward Nevill
edward.nevill at gmail.com
Thu Aug 20 10:47:44 UTC 2015
Hi,
The following webrev
http://cr.openjdk.java.net/~enevill/8133842/webrev.01/
fixes a problem reported by one of our partners whereby C2 can generate illegal instructions on certain partners HW.
JIRA issue here
https://bugs.openjdk.java.net/browse/JDK-8133842
The problem occurs when you have a logical or arithmetic instruction with a RHS which is shifted by a constant where (const & 32) != 0, ie the constant is 32..63 or 96..127 etc.
For example the following
res = i | (j >> 53)
generates the instruction
orrw Rd, Rn, Rm, ASR #53
This instruction has a 6 bit field for the shift so this would appear to be a legal encoding. However certain partner HW treats this as an undefined instruction generating a SIGILL.
The problem was that the rules in aarch64.ad were always anding the constant with 0x3f for both ints and longs.
The above webrev fixes this to mask with 0x1f for ints and 0x3f for longs.
Tested with hotspot and langtools. Results the same in both cases.
Hotspot: Test results: passed: 863; failed: 2; error: 10
Langtools: Test results: passed: 3,263
Thanks for your help with this review,
Ed.
More information about the aarch64-port-dev
mailing list