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.

Sign up for GitHub

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

Jump to bottom

Support for LLVM memoryeffects attribute #422

Open

wsmoses opened this issue Jun 15, 2024 · 2 comments

Labels

api coverage good first issue

Contributor

wsmoses commented Jun 15, 2024

With readonly/readnone/writeonly being no longer permissible as llvm fn args in later LLVM's a nice way to query for this would be helpful

Owner

maleadt commented Jun 17, 2024

Needs something like llvm/llvm-project#75123 but for setMemoryEffects/getMemoryEffects.

maleadt added good first issue api coverage labels

Contributor Author

wsmoses commented Jun 17, 2024

Enzyme.jl now vendors its own version of this here to be able to not crash on 1.11: https://github.com/EnzymeAD/Enzyme.jl/blob/e9804331d9e5ef6c277982e68f9077203b6d1a37/src/compiler/utils.jl#L1

Happy to have that brought in upstream here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment