[PATCH] D72184: [BPF] support atomic instructions
Brendan Jackman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 23 08:54:07 PST 2020
jackmanb added a comment.
> I did not see kernel has atomic_store, do you mean atomic_set?
Sorry yep I meant `atomic_set`
> Do you suggest we also implement atomic_set? There is no need for 64-bit architecture like x64, right?
Yeah actually now I think about it, `atomic_set` is pretty pointless, I think it's only there in the kernel to support strong type-checking of `atomic_t`; It doesn't imply any barriers.
I think we can do without it, it makes more sense to implement alongside barrier instructions.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72184/new/
https://reviews.llvm.org/D72184
More information about the cfe-commits
mailing list