[llvm] [mlir] [mlir][OpenMP] Add __atomic_store to AtomicInfo (PR #121055)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 13 07:17:21 PST 2025
NimishMishra wrote:
Hi @tblah,
It seems the issue in your crash is with `__atomic_load`; still it affects `__atomic_store`. I think the issue is the way the first argument of the libcall is generated. Right now, the `type` of the atomic variable is coming out as `ptr`, which is 8 bytes by default. We rather want to argument to the libcall to include the size of the allocated type.
I have made a branch off this PR here: https://github.com/NimishMishra/f18-llvm-project/commits/atomic_allocated_type_size/. Could you check if you are happy with it? I can submit it for review as a separate PR then, since the issue is with all of atomic read/write/update.
https://github.com/llvm/llvm-project/pull/121055
More information about the llvm-commits
mailing list