8299168: RISC-V: Fix MachNode size mismatch for MacroAssembler::_verify_oops*

[DingliZhang]

Please review this backport to riscv-port-jdk11u.
Backport of JDK-8299168.
Applies not cleanly due to context difference, but it is easy to resolve them manually.

Testing:

• Tier1 passed without new failure on lp4a (release).
• Tier1 hotspot passed without new failure on lp4a (fastdebug).

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• Change must not contain extraneous whitespace
• Commit message must refer to an issue

Issue

• JDK-8299168: RISC-V: Fix MachNode size mismatch for MacroAssembler::_verify_oops* (Bug - P4)

Reviewers

• Fei Yang (@RealFYang - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/riscv-port-jdk11u.git pull/42/head:pull/42
$ git checkout pull/42

Update a local copy of the PR:
$ git checkout pull/42
$ git pull https://git.openjdk.org/riscv-port-jdk11u.git pull/42/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 42

View PR using the GUI difftool:
$ git pr show -t 42

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/riscv-port-jdk11u/pull/42.diff

Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back dzhang! A progress list of the required criteria for merging this PR into riscv-port will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

@DingliZhang This change now passes all automated pre-integration checks.

After integration, the commit message for the final commit will be:

8299168: RISC-V: Fix MachNode size mismatch for MacroAssembler::_verify_oops*

Reviewed-by: fyang

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been no new commits pushed to the riscv-port branch. If another commit should be pushed before you perform the /integrate command, your PR will be automatically rebased. If you prefer to avoid any potential automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the riscv-port branch, type /integrate in a new comment.

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[DingliZhang]

The test is in progress and will be updated later with the results.

[mlbridge]

Webrevs

• 01: Full (c0a92f66)
• 00: Full (78495b64)

[RealFYang]

Looks good.

[andreas-schwab]

/home/abuild/rpmbuild/BUILD/jdk11u-jdk-11.0.25-9/src/hotspot/cpu/riscv/macroAssembler_riscv.cpp:391:5: error: 'IncompressibleRegion' was not declared in this scope; did you mean 'CompressibleRegion'?

[DingliZhang]

/home/abuild/rpmbuild/BUILD/jdk11u-jdk-11.0.25-9/src/hotspot/cpu/riscv/macroAssembler_riscv.cpp:391:5: error: 'IncompressibleRegion' was not declared in this scope; did you mean 'CompressibleRegion'?

Sorry for the confusion. This PR requires a pre-patch . We need to merge these PR in order, so please be patient.

[andreas-schwab]

Thanks, I can confirm that this together with refs/pull/43/head fixes the issue.

[openjdk]

@DingliZhang Please do not rebase or force-push to an active PR as it invalidates existing review comments. Note for future reference, the bots always squash all changes into a single commit automatically as part of the integration. See OpenJDK Developers’ Guide for more information.

[DingliZhang]

Thanks for the review.
/integrate

[openjdk]

Going to push as commit ccbb663.

[openjdk]

@DingliZhang Pushed as commit ccbb663.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.