8337780: RISC-V: C2: Change C calling convention for sp to NS

[zifeihan]

Hi, Please help review this backport to riscv-port-jdk11u.
Clean backport of JDK-8337780 to reduce frame size by 16 bytes and improve performance for some C2 runtime stubs as we do not have to save sp on the method entry.

Testing

• Run tier1-3 tests on SOPHON SG2042 (release)

---

Progress

• Change must not contain extraneous whitespace
• Commit message must refer to an issue

Issue

• JDK-8337780: RISC-V: C2: Change C calling convention for sp to NS (Enhancement - P4)

Reviewers

• Fei Yang (@RealFYang - Reviewer)

Reviewing

Using git

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

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

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 26

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

Using diff file

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

Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back gcao! 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]

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

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

8337780: RISC-V: C2: Change C calling convention for sp to NS

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.

[mlbridge]

Webrevs

• 00: Full (bf19b1bf)

[RealFYang]

Looks good. I compared some of the C2 runtime stubs, it works as expected.

[zifeihan]

Thanks for the review.
/integrate

[openjdk]

Going to push as commit 1f90ddd.
Since your change was applied there have been 8 commits pushed to the riscv-port branch:

• f229b61: Merge branch 'master' into riscv-port
• a83d930: 8273135: java/awt/color/ICC_ColorSpace/MTTransformReplacedProfile.java crashes in liblcms.dylib with NULLSeek+0x7
• c67fe36: 8325022: Incorrect error message on client authentication
• 3df1852: 8210338: Better output for GenerationTests.java
• 9ba9dd2: 8224081: SOCKS v4 tests require IPv4
• d3ce405: 8206440: Remove javac -source/-target 6 from jdk regression tests
• 828d645: 8314614: jdk/jshell/ImportTest.java failed with "InternalError: Failed remote listen"
• cd4ae8c: 8316973: GC: Make TestDisableDefaultGC use createTestJvm

Your commit was automatically rebased without conflicts.

[openjdk]

@zifeihan Pushed as commit 1f90ddd.

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