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