Skip to content

AArch64: Add a parameter for generating relocatable code to loadAddressConstant - #7072

Merged
knn-k merged 1 commit into
eclipse-omr:masterfrom
Akira1Saitoh:aarch64EnhanceLoadAddrConst
Jul 25, 2023
Merged

AArch64: Add a parameter for generating relocatable code to loadAddressConstant#7072
knn-k merged 1 commit into
eclipse-omr:masterfrom
Akira1Saitoh:aarch64EnhanceLoadAddrConst

Conversation

@Akira1Saitoh

Copy link
Copy Markdown
Contributor

The existing loadAddressConstant function on AArch64 determines whether it should generate relocatable code by the answer returned by compileRelocatableCode() query. However, it does not return the correct answer under a remote compilation.
To support remote compilations, this commit adds an overloaded version of loadAddressConstant with a boolean parameter that allows the caller of the api to determine if the relocatable code is generated.

…ssConstant

The existing `loadAddressConstant` function on AArch64 determines
whether it should generate relocatable code by the answer returned
by `compileRelocatableCode()` query. However, it does not return
the correct answer under a remote compilation.
To support remote compilations, this commit adds an overloaded version
of `loadAddressConstant` with a boolean parameter that allows
the caller of the api to determine if the relocatable code is generated.

Signed-off-by: Akira Saitoh <saiaki@jp.ibm.com>
@Akira1Saitoh
Akira1Saitoh marked this pull request as ready for review July 24, 2023 23:57
@knn-k

knn-k commented Jul 25, 2023

Copy link
Copy Markdown
Contributor

Jenkins build aarch64,amac

@knn-k knn-k self-assigned this Jul 25, 2023
@knn-k
knn-k merged commit 436bad3 into eclipse-omr:master Jul 25, 2023
@Akira1Saitoh
Akira1Saitoh deleted the aarch64EnhanceLoadAddrConst branch July 25, 2023 00:46
@knn-k knn-k mentioned this pull request Jul 25, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants