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

DPCT1098 for store operations ? #1980

Closed
jinz2014 opened this issue May 14, 2024 · 3 comments
Closed

DPCT1098 for store operations ? #1980

jinz2014 opened this issue May 14, 2024 · 3 comments

Comments

@jinz2014
Copy link
Contributor

I think the description in DPCT1098 may be confusing for the store operations:
__stcg

__stcs

__stwb

__stwt

Thanks for your clarification.

@tomflinda
Copy link
Contributor

@jinz2014
The content of the DPCT1098 msg is The <expression text> expression is used instead of the <function name> call. These two expressions do not provide the exact same functionality. Check the generated code for potential precision and/or performance issues.
For more details, pls refer to https://www.intel.com/content/www/us/en/docs/dpcpp-compatibility-tool/developer-guide-reference/2024-1/dpct1098.html.

So, I'm not sure which part of the message is confusing. Could you provide more information?

@jinz2014
Copy link
Contributor Author

For the load operations, the link indicates that we may change the codes generated by SYClomatic. For the store operations, we don't need to make further changes. Is that right ?

@tomflinda
Copy link
Contributor

@jinz2014 The current migration mapping has the same semantics as the CUDA side. While the implementation of the CUDA side leverages cache operations, it has performance advantages; the SYCL side may not use cache operations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants