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

Ensure staticName() symbols represent static fields #3031

Merged
merged 1 commit into from
Oct 6, 2018

Conversation

yanluo7
Copy link
Contributor

@yanluo7 yanluo7 commented Oct 4, 2018

Caller of Symbol::staticName typically assumes the underlying symbol
represents a static field. However, the condition that guards the
call currently only checks for a staic symbol, which is insufficient
as other types like Constant Dynamic. Method Handle, Method Type all
use static symbols. Use the correct check Symbol::isStaticField which
precludes the above mentioned non-static field types.

Signed-off-by: Yan Luo [email protected]

@0xdaryl
Copy link
Contributor

0xdaryl commented Oct 5, 2018

Change looks ok, but I have a couple of comments:

  • your commit title misled me to what your fix actually was. I propose a title something like "Ensure staticName() symbols are static fields"

  • if the callers of staticName are assuming they're asking about a static field, perhaps staticName should be renamed to staticFieldName to prevent this kind of mistake from occurring in the future? Not for this PR, but might be worth an issue to track it.

@0xdaryl
Copy link
Contributor

0xdaryl commented Oct 5, 2018

@genie-omr build all

Caller of Symbol::staticName typically assumes the underlying symbol
represents a static field. However, the condition that guards the
call currently only checks for a staic symbol, which is insufficient
as other types like Constant Dynamic. Method Handle, Method Type all
use static symbols. Use the correct check Symbol::isStaticField which
precludes the above mentioned non-static field types.

Signed-off-by: Yan Luo <[email protected]>
@yanluo7 yanluo7 changed the title Fix jit crash in Symbol staticName Ensure staticName() symbols represent static fields Oct 5, 2018
@yanluo7
Copy link
Contributor Author

yanluo7 commented Oct 5, 2018

Open #3034 to track renaming staticName to staticFieldName.

@0xdaryl 0xdaryl self-assigned this Oct 6, 2018
@0xdaryl
Copy link
Contributor

0xdaryl commented Oct 6, 2018

@genie-omr build all

@0xdaryl 0xdaryl merged commit e800d76 into eclipse-omr:master Oct 6, 2018
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Nov 13, 2019
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Dec 5, 2019
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jan 24, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jan 24, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jan 24, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Feb 19, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Feb 20, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Mar 9, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Mar 9, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Mar 23, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 1, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 6, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 6, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 7, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 24, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 28, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 30, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request May 10, 2020
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp,
OMRExternalRelocationsKindToHeaderSizeMap.hpp. These files contain
RelocationRecord, RelocationRecordGroup,RelocationRecordBinaryTemplate
classes. The RelocationRecordGroup is an iterator for application of
relocations. The RelocationRecordBinaryTemplate is the class that has
RelocationRecord information. Those classes realize core relocations
storage functionality in OpenJ9, but are language independent and thus
are worth including into OMR. The
OMRExternalRelocationsKindToHeaderSizeMap.hpp is an attempt for
improving extension, similarly to codeGenPhases, by including headers
with relocation sizes from consuming projects. Compared to OpenJ9, the
RelocationRecord classes became extensible. Another proposed change is
in the creation of relocation records: the implementation of
TR::RelocationRecord::create function is proposed to be done as a
function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031
[skip ci]

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jul 16, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jul 16, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jul 16, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jul 16, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jul 16, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jul 16, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Oct 1, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Oct 1, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Oct 6, 2021
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Mar 25, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Mar 25, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Mar 25, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 1, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 1, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 23, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 23, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 23, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 23, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 23, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 23, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jun 26, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Aug 11, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Sep 29, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Oct 28, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Oct 28, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Nov 22, 2022
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Jan 11, 2023
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 11, 2023
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Apr 11, 2023
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
georgkrylov added a commit to CAS-Atlantic/omr that referenced this pull request Aug 19, 2023
The added files are: OMRRelocationRecord.hpp, OMRRelocationRecord.cpp.
These files contain RelocationRecord, RelocationRecordGroup,
RelocationRecordBinaryTemplate classes. The RelocationRecordGroup is an
iterator for application of relocations. The
RelocationRecordBinaryTemplate is the class that has RelocationRecord
information. Those classes realize core relocations storage functiona-
lity in OpenJ9, but are language independent and thus are worth inclu-
ding into OMR.
Compared to OpenJ9, the RelocationRecord classes became extensible.
Another proposed change is in the creation of relocation records:
the implementation of TR::RelocationRecord::create function is proposed
to be done as a function in corresponding relocationRecord class. The
RelocationRecordBinaryTemplate, has undergone changes to improve
readability and portability: the setSize() and setType(), as well as
x64 architecture specific setExtra() and their corresponding accessor
methods are now  RelocationTarget API calls, as advised in Issue eclipse-omr#3031

Signed-off-by: Georgiy Krylov <[email protected]>
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