Skip to content

Remove setting of Vector Supports in Linkage - #15546

Merged
joransiu merged 1 commit into
eclipse-openj9:masterfrom
r30shah:removeVectorSupportCheck
Jul 22, 2022
Merged

Remove setting of Vector Supports in Linkage#15546
joransiu merged 1 commit into
eclipse-openj9:masterfrom
r30shah:removeVectorSupportCheck

Conversation

@r30shah

@r30shah r30shah commented Jul 14, 2022

Copy link
Copy Markdown
Contributor

Previously support of vector registers were checked while creating a
private linkage, which would not be a good place to do so. It should be
set while initializing Code Generator Object. This PR removes that part
of code from OpenJ9.

Signed-off-by: Rahil Shah rahil@ca.ibm.com

@r30shah r30shah changed the title Remove setting of Vector Supports in Linkage WIP: Remove setting of Vector Supports in Linkage Jul 14, 2022
Previously support of vector registers were checked while creating a
private linkage, which would not be a good place to do so. It should be
set while initializing Code Generator Object. This PR removes that part
of code from OpenJ9.

Signed-off-by: Rahil Shah <rahil@ca.ibm.com>
@r30shah
r30shah force-pushed the removeVectorSupportCheck branch from b957b93 to 3e43147 Compare July 18, 2022 20:09
@r30shah r30shah changed the title WIP: Remove setting of Vector Supports in Linkage Remove setting of Vector Supports in Linkage Jul 19, 2022
@r30shah

r30shah commented Jul 20, 2022

Copy link
Copy Markdown
Contributor Author

@joransiu Once we get changes in eclipse-omr/omr#6606 merged, we can clean-up the code that sets up things in Linkage through this PR. Can I please get your review on this one? Build ids I posted in eclipse-omr/omr#6606 (comment) tests out this changes as well.

@joransiu

Copy link
Copy Markdown
Member

jenkins test sanity zlinux jdk11,jdk17 depends eclipse-omr/omr#6606

@joransiu
joransiu merged commit 9b8f3c6 into eclipse-openj9:master Jul 22, 2022
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