Omr: Move _ialoadUnneeded field to Z CodeGenerator

Created on 11 Apr 2021  路  6Comments  路  Source: eclipse/omr

The following field is unique to the Z CodeGenerator and is only used by it. It should be relocated to z/codegen/OMRCodeGenerator.hpp:

https://github.com/eclipse/omr/blob/d7a31696cdb2b56f5107d500fda2fbc9a1e8b69a/compiler/codegen/OMRCodeGenerator.hpp#L271

backlog compiler good first issue help wanted
Source
picture 0xdaryl

Most helpful comment

Sorry for the lag, still working on this.

picture rlkenny98 on 24 Apr 2021
👍2

All 6 comments

Hey @0xdaryl, Can I help out with this? Fairly new to open source contributions and I would like to go about it properly this time.

rlkenny98 picture rlkenny98 on 14 Apr 2021

Sure! You can start with the project's Contributor Guidelines. If you are not familiar with git/GitHub at all then our git crash course may be of use to you too. Please reach out if you have any questions. Thanks for your help!

0xdaryl picture 0xdaryl on 14 Apr 2021

Sounds Good!

rlkenny98 picture rlkenny98 on 14 Apr 2021

Sorry for the lag, still working on this.

rlkenny98 picture rlkenny98 on 24 Apr 2021
👍2

Will have this finished this week!

rlkenny98 picture rlkenny98 on 3 May 2021

Opened pull request. Thanks for your patience, enjoy the rest of your weekend!

rlkenny98 picture rlkenny98 on 9 May 2021
Was this page helpful?
0 / 5 - 0 ratings

Related issues

samolisov picture samolisov  路  5Comments
0xdaryl picture 0xdaryl  路  3Comments
0xdaryl picture 0xdaryl  路  3Comments
hzongaro picture hzongaro  路  4Comments
0xdaryl picture 0xdaryl  路  3Comments