-
Notifications
You must be signed in to change notification settings - Fork 415
Add msplats Vector operation #7785
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
Conversation
|
Minor fixes. |
|
After some thinking and discussions, we would like to propose that This will make it easier to splat a -1 value to all mask elements on platforms that don't have Mask registers. On those platforms, all mask operations happen bit-wise and therefore It should also help platforms that have Mask registers since the passed long value will represent the whole mask. If other values are passed, the behavior is undefined. |
|
On the second thought, the child type can be TR::Int8. Codegen can always do b2l (usually noop) if needed. We would like to keep the opcode's semantics to a minimum. |
|
I support that thought. It seemed a bit unnatural to require a 64-bit value to pass in |
|
Updated according to the comments above. |
|
Jenkins build ppc64le |
|
jenkins build plinux |
|
Improved opcode description. |
|
Improved commit message |
|
Fixed typo |
- msplats opcode takes 'true' or 'false' value and splats it into all lanes of the mask - 'true' must be represented as -1 and 'false' as 0 of type TR::Int8 - operation is undefined for any other value - implement Power evaluator
|
Jenkins build all |
|
Build failures seem unrelated to me. |
Uh oh!
There was an error while loading. Please reload this page.