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

rustc: add support for nontemporal_load intrinsic #46618

Closed
gwenn opened this issue Dec 10, 2017 · 2 comments
Closed

rustc: add support for nontemporal_load intrinsic #46618

gwenn opened this issue Dec 10, 2017 · 2 comments
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. C-feature-request Category: A feature request, i.e: not implemented / a PR. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@gwenn
Copy link

gwenn commented Dec 10, 2017

For stdarch, some intrinsics need this.
It should be similar to the nontemporal_store introduced by #45367.

@pietroalbini pietroalbini added C-feature-request Category: A feature request, i.e: not implemented / a PR. A-SIMD Area: SIMD (Single Instruction Multiple Data) labels Jan 23, 2018
@workingjubilee workingjubilee added the A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. label Feb 9, 2021
@workingjubilee
Copy link
Member

workingjubilee commented Feb 9, 2021

I have been informed that the required memory fencing to implement these is not supported by LLVM in a cross-platform way yet, so this is likely blocked on further codegen backend support in order to implement them safely, at least for now. https://lists.llvm.org/pipermail/llvm-dev/2020-April/141279.html

Though these may be technically usable but "just" really hideously unsafe and of dangerous soundness concerns! Unsure. :D Something to pay careful attention to.

@workingjubilee workingjubilee removed the A-SIMD Area: SIMD (Single Instruction Multiple Data) label Oct 5, 2021
@Noratrieb Noratrieb added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Apr 5, 2023
@Mark-Simulacrum
Copy link
Member

I'm going to close this - we have some intrinsics exposing this (e.g., _mm_stream_ps), and while there are some issues with this currently (#114582) I don't know that we expect to do more here. I doubt we'd want a stable cross-platform API for something this niche.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. C-feature-request Category: A feature request, i.e: not implemented / a PR. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

5 participants