JDK-8185656 : [AArch64] Fix for 8154537 was missing one file
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 9,10
  • Priority: P4
  • Status: Closed
  • Resolution: Duplicate
  • CPU: aarch64
  • Submitted: 2017-08-01
  • Updated: 2017-11-13
  • Resolved: 2017-11-13
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 10
10Resolved
Related Reports
Duplicate :  
JDK-8187224 - aarch64: some inconsistency between aarch64_ad.m4 and aarch64.ad
Duplicate :  
JDK-8187224 - aarch64: some inconsistency between aarch64_ad.m4 and aarch64.ad
Relates :  
JDK-8154537 - AArch64: some integer rotate instructions are never emitted
Description
I'm looking at the webrev in
http://cr.openjdk.java.net/~roland/8154537/webrev.00/ and I see the
the changes were made to aarch64.ad but not to ad_aarch64.m4.  This is
problematic because some .m4 files are used to generate the .ad file,
and if anyone regenerates the .ad file the bug will regress.

I think this is the change we need to make.  It won't affect generated
code at all, but it is something of a ticking bomb.

http://cr.openjdk.java.net/~aph/8185656.patch
Comments
ILW = possible 8154537-some integer rotate instructions not emitted; No failure as of now, possible regression if .ad file regenerated, only for aarch64; No Workaround = MLH = P4
02-08-2017
Hi [~roland], please note assigning this task to you for now, since you worked for 8154537. Please re-assign if any issues. Thanks.
02-08-2017