RFR: 8328998: Encoding support for Intel APX extended general-purpose registers [v2]

Vladimir Kozlov kvn at openjdk.org
Wed Apr 17 02:14:47 UTC 2024


On Wed, 17 Apr 2024 00:44:09 GMT, Steve Dohrmann <duke at openjdk.org> wrote:

>> Got it - it is for few registers check now. What is common between rsp, r12, r20, r28 registers (except encoding)?
>> R12 is used for heap base in compressed oops  and RSP is RSP. What are r20 and r28? Why they can't be used in this addressing mode?
>> 
>> Please add comments for all lines where you replaced checks for `r*->encoding()` to say for which registers you do a check and why.
>
> The reason registers 12, 20, and 28 are asserted out of that else at line 668 is they are handled earlier in an else around line 646.
> 
> `    } else if ((base_enc & 0x7) == 4) {
>       // [rsp + disp]
> `
> 
> I added a comment to this effect and have also added comments in the 3 other places in the function where the replacement was done, indicating the registers involved.

Good.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18476#discussion_r1568121201


More information about the hotspot-compiler-dev mailing list