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

Arithmetic/logical operator mixin for proxy references #429

Closed
bernhardmgruber opened this issue Dec 10, 2021 · 0 comments · Fixed by #430
Closed

Arithmetic/logical operator mixin for proxy references #429

bernhardmgruber opened this issue Dec 10, 2021 · 0 comments · Fixed by #430
Labels
enhancement New feature or request

Comments

@bernhardmgruber
Copy link
Member

With the advent of computed mappings returning proxy references, the user is much more restricted in what they can express on such proxy references. Adding arithmetic and logical operators to proxy references would alleviate some of this pain. Since proxy references have a small and common interface, we could provide these operators as a mixin.

@bernhardmgruber bernhardmgruber changed the title Arithmetic mixin for proxy references Arithmetic/logical operator mixin for proxy references Dec 10, 2021
@bernhardmgruber bernhardmgruber added the enhancement New feature or request label Dec 10, 2021
bernhardmgruber added a commit to bernhardmgruber/llama that referenced this issue Dec 10, 2021
* supplies compound assignment and increment/decrement operators
* mixin ProxyRefOpMixin into proxy references for ChangeTypeReference, BitPackedFloatRef, BitPackedIntRef and Bytesplit::Reference
* add unit tests for ProxyRefOpMixin and proxy references of mappings

Fixes alpaka-group#429
bernhardmgruber added a commit to bernhardmgruber/llama that referenced this issue Dec 10, 2021
* supplies compound assignment and increment/decrement operators
* mixin ProxyRefOpMixin into proxy references for ChangeTypeReference, BitPackedFloatRef, BitPackedIntRef and Bytesplit::Reference
* add unit tests for ProxyRefOpMixin and proxy references of mappings

Fixes alpaka-group#429
bernhardmgruber added a commit to bernhardmgruber/llama that referenced this issue Dec 10, 2021
* supplies compound assignment and increment/decrement operators
* mixin ProxyRefOpMixin into proxy references for ChangeTypeReference, BitPackedFloatRef, BitPackedIntRef and Bytesplit::Reference
* add unit tests for ProxyRefOpMixin and proxy references of mappings

Fixes alpaka-group#429
bernhardmgruber added a commit to bernhardmgruber/llama that referenced this issue Dec 10, 2021
* supplies compound assignment and increment/decrement operators
* mixin ProxyRefOpMixin into proxy references for ChangeTypeReference, BitPackedFloatRef, BitPackedIntRef and Bytesplit::Reference
* add unit tests for ProxyRefOpMixin and proxy references of mappings

Fixes alpaka-group#429
bernhardmgruber added a commit to bernhardmgruber/llama that referenced this issue Dec 10, 2021
* supplies compound assignment and increment/decrement operators
* mixin ProxyRefOpMixin into proxy references for ChangeTypeReference, BitPackedFloatRef, BitPackedIntRef and Bytesplit::Reference
* add unit tests for ProxyRefOpMixin and proxy references of mappings
* unit tests are compiled with precise FP model for icpx

Fixes alpaka-group#429
bernhardmgruber added a commit that referenced this issue Dec 11, 2021
* supplies compound assignment and increment/decrement operators
* mixin ProxyRefOpMixin into proxy references for ChangeTypeReference, BitPackedFloatRef, BitPackedIntRef and Bytesplit::Reference
* add unit tests for ProxyRefOpMixin and proxy references of mappings
* unit tests are compiled with precise FP model for icpx

Fixes #429
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant