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

Define isArrayCompTypeValueType method for Value Propagation #5909

Merged
merged 1 commit into from
Apr 12, 2021

Conversation

hzongaro
Copy link
Member

@hzongaro hzongaro commented Apr 9, 2021

This change adds a virtual method and base implementation of isArrayCompTypeValueType to OMR::ValuePropagation that can be used to determine whether the component type of an array is definitely, might be or is definitely not a value type.

The base implementation simply assumes any array's component type might be a value type if value type support is enabled, and that it is definitely not a value type otherwise.

This change adds a virtual method and base implementation of
isArrayCompTypeValueType to OMR::ValuePropagation that can be used to
determine whether the component type of an array is definitely, might be
or is definitely not a value type.

The base implementation simply assumes any array's component type might
be a value type if value type support is enabled, and that it is
definitely not a value type otherwise.

Signed-off-by:  Henry Zongaro <zongaro@ca.ibm.com>
@hzongaro
Copy link
Member Author

hzongaro commented Apr 9, 2021

FYI @Leonardo2718, @a7ehuo, @vijaysun-omr, @0xdaryl - this pull request replaces my recently withdrawn pull request #5904. Corrections to the implementation of isArrayCompTypeValueType will go directly to the appropriate downstream project.

@0xdaryl
Copy link
Contributor

0xdaryl commented Apr 12, 2021

Jenkins build all

@0xdaryl 0xdaryl self-assigned this Apr 12, 2021
@0xdaryl 0xdaryl merged commit 0928810 into eclipse:master Apr 12, 2021
@hzongaro hzongaro deleted the vp-array-comp-vt-check-base branch April 22, 2021 13:54
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