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

Add TR::vnolz and TR::vnotz evaluators in z/codegen #7137

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

dchopra001
Copy link
Contributor

This PR introduces evaluators for the TR::vnolz and TR::vnotz IL in the z/codegen. Support is only enabled for architectures with vector instruction support.

@dchopra001
Copy link
Contributor Author

Tagging @r30shah for review.

@dchopra001
Copy link
Contributor Author

The failure in x86-64 macOS is unrelated as the changes in this PR are confined to the z codegen.

@@ -2337,6 +2325,76 @@ OMR::Z::TreeEvaluator::lbitpermuteEvaluator(TR::Node *node, TR::CodeGenerator *c
return TR::TreeEvaluator::bitpermuteEvaluator(node, cg);
}

TR::Register*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use inlineVectorUnaryOp ? It does most of the work this two function does.

Copy link
Contributor

@r30shah r30shah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, can you squash the commits?

@r30shah
Copy link
Contributor

r30shah commented Oct 13, 2023

Jenkins build zlinux,zos

@r30shah
Copy link
Contributor

r30shah commented Oct 13, 2023

@hzongaro given Daryl is away, can I request you to review and merge this change ?

@hzongaro hzongaro self-assigned this Oct 16, 2023
@hzongaro hzongaro merged commit 906da98 into eclipse-omr:master Oct 17, 2023
8 checks passed
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