[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