RFR: 8259822: [PPC64] Support the prefixed instruction format added in POWER10 [v4]

Martin Doerr mdoerr at openjdk.java.net
Wed Feb 3 21:03:42 UTC 2021


On Wed, 3 Feb 2021 17:35:33 GMT, Corey Ashford <github.com+51754783+CoreyAshford at openjdk.org> wrote:

>> Kazunori Ogata has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Removed pla and psubi and adjusted spacing based on review comments
>
> Looks good!  Thanks for the additional formatting cleanup

Sorry that I didn't review it, yet. I didn't have enough time.

Please note that C2 already has a mechanism to handle alignment:

Insert "ins_alignment(n);" into the instruct. This allows C2 to insert up to n-1 nop instructions. 
Add a "compute_padding" function to determine the actual number of nops to insert given the current offset.
They are used in s390.ad. Please take a look.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2095


More information about the hotspot-compiler-dev mailing list