8252866: MemoryAccess.setAddress should take Addressable, not MemoryAddress (follow up)

[mcimadamore]

This small patch fixes an oversight - MemoryAccess.setAddressAtOffset should call address() on the incoming value, before dispatching on the var handle.

---

Progress

• Change must not contain extraneous whitespace
• Change must be properly reviewed

Issue

• JDK-8252866: MemoryAccess.setAddress should take Addressable, not MemoryAddress ⚠️ Title mismatch between PR and JBS.

Reviewers

• Athijegannathan Sundararajan (@sundararajana - Committer)

Download

$ git fetch https://git.openjdk.java.net/panama-foreign pull/315/head:pull/315
$ git checkout pull/315

[bridgekeeper]

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

[mcimadamore]

/integrate

[openjdk]

@mcimadamore This change now passes all automated pre-integration checks. In addition to the automated checks, the change must also fulfill all project specific requirements

After integration, the commit message will be:

8252866: MemoryAccess.setAddress should take Addressable, not MemoryAddress (follow up)

Reviewed-by: sundar

• If you would like to add a summary, use the /summary command.
• To credit additional contributors, use the /contributor command.
• To add additional solved issues, use the /issue command.

There are currently no new commits on the foreign-memaccess branch since the last update of the source branch of this PR. If another commit should be pushed before you perform the /integrate command, your PR will be automatically rebased. If you would like to avoid potential automatic rebasing, specify the current head hash when integrating, like this: /integrate 424c71b3d6f545a9245545cdd9fd50ce249a6e1e.

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

[openjdk]

@mcimadamore Pushed as commit bb4f95a.

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

[mlbridge]

Webrevs

• 00: Full (117c946)