Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overload incRefCountForOpaquePseudoRegister in OpenJ9 #5881

Merged
merged 1 commit into from
May 29, 2019

Conversation

wbh123456
Copy link
Contributor

This patch overloads J9::Z::CodeGenerator::incRefCountForOpaquePseudoRegister in
OpenJ9 with less parameters. This is the 2/6 step to simpify function incRefCountForOpaquePseudoRegister.

Issue: eclipse-omr/omr#1855
Signed-off-by: Bohao(Aaron) Wang aaronwang0407@gmail.com

@wbh123456 wbh123456 changed the title WIP: Overload incRefCountForOpaquePseudoRegister in OpenJ9 Overload incRefCountForOpaquePseudoRegister in OpenJ9 May 27, 2019
@wbh123456 wbh123456 changed the title Overload incRefCountForOpaquePseudoRegister in OpenJ9 WIP: Overload incRefCountForOpaquePseudoRegister in OpenJ9 May 27, 2019
@wbh123456 wbh123456 changed the title WIP: Overload incRefCountForOpaquePseudoRegister in OpenJ9 Overload incRefCountForOpaquePseudoRegister in OpenJ9 May 27, 2019
@fjeremic
Copy link
Contributor

@wbh123456 I think something went wrong here judging by the number of foreign commits. Perhaps a rebase will help?

This patch overloads J9::Z::CodeGenerator::incRefCountForOpaquePseudoRegister in
OpenJ9 with less parameters. This is the 2/6 step to simpify this
function.

Issue: eclipse-omr/omr#1855
Signed-off-by: Bohao(Aaron) Wang <aaronwang0407@gmail.com>
@wbh123456
Copy link
Contributor Author

@wbh123456 I think something went wrong here judging by the number of foreign commits. Perhaps a rebase will help?

I did not perform a rebase properly. Now it should be good.

@fjeremic
Copy link
Contributor

Jenkins compile zlinux JDK8

@fjeremic fjeremic merged commit 7d34cb5 into eclipse-openj9:master May 29, 2019
@wbh123456 wbh123456 deleted the overload_inc branch May 30, 2019 12:56
@0xdaryl 0xdaryl added arch:z and removed comp:jit:z labels Jun 24, 2019
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.

3 participants