elliottt opened PR #5251 from trevor/sse-blend-pinned-regs
to main
:
Instead of always using
xmm0
, use regalloc constraints to constrain the mask register for the sse blend operations.
<!--Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
elliottt updated PR #5251 from trevor/sse-blend-pinned-regs
to main
.
elliottt updated PR #5251 from trevor/sse-blend-pinned-regs
to main
.
elliottt has marked PR #5251 as ready for review.
elliottt submitted PR review.
elliottt created PR review comment:
Another option here would be to use an
Option<Xmm>
field onXmmRmR
. Happy to rework with that approach if the additional constructor here seems like too much.
elliottt updated PR #5251 from trevor/sse-blend-pinned-regs
to main
.
elliottt updated PR #5251 from trevor/sse-blend-pinned-regs
to main
.
elliottt updated PR #5251 from trevor/sse-blend-pinned-regs
to main
.
elliottt edited PR #5251 from trevor/sse-blend-pinned-regs
to main
:
Instead of using
xmm0
explicitly for the mask argument to instructions likeblendpd
, use regalloc constraints to constrain it toxmm0
instead.
<!--Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
elliottt edited PR #5251 from trevor/sse-blend-pinned-regs
to main
:
Instead of using
xmm0
explicitly for the mask argument to instructions likeblendvpd
, use regalloc constraints to constrain it toxmm0
instead.
<!--Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
elliottt requested fitzgen for a review on PR #5251.
jameysharp submitted PR review.
jameysharp submitted PR review.
jameysharp created PR review comment:
Should this comment also be deleted now? If I'm reading the surrounding context correctly, I think
xmm0
was the last thing the comment referred to.
elliottt submitted PR review.
elliottt created PR review comment:
Yep, thanks for catching that!
elliottt updated PR #5251 from trevor/sse-blend-pinned-regs
to main
.
elliottt merged PR #5251.
Last updated: Dec 23 2024 at 12:05 UTC