RFR: 8255550: x86: Assembler::cmpq(Address dst, Register src) encoding is incorrect [v2]

Aleksey Shipilev shade at openjdk.java.net
Thu Oct 29 12:32:50 UTC 2020


On Thu, 29 Oct 2020 07:28:29 GMT, Erik Ă–sterlund <eosterlund at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two commits:
>> 
>>  - Merge branch 'master' into JDK-8255550-cmpq-incorrect
>>  - 8255550: x86: Assembler::cmpq(Address dst, Register src) encoding is incorrect
>
> Thanks for poking me. I would prefer to change to the cmpq instruction that has the opposite order in the stack watermark barrier instead. Everywhere in the code I talk about the condition being sp being "above" watermark. Changing it to less makes me twist my head in ways that heads should not twist.

Dropped the `safepoint_poll` hunk after #924 integration. @fisk, this must be fine with you then?

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

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


More information about the hotspot-compiler-dev mailing list