[clang] [Clang] Introduce scoped variants of GNU atomic functions (PR #72280)

Joseph Huber via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 16 17:07:43 PST 2023


jhuber6 wrote:

> > The underlying implementation is a string literal in the LLVM syncscope argument, but the problem is that this isn't standardized at all and varies between backends potentially
> 
> We don't have to use the same set of strings as syncscope if that doesn't make sense.

I don't think there's much of a point to making them strings if it's not directly invoking the syncscope name for the backend. Realistically as long as we give them descriptive names we can just ignore ones that don't apply on various targets. Like right now you can use these scoped variants in x64 code but it has no effect. Either that or we could use logic to go to the next heirarchy level that makes sense. As always, naming stuff is hard.

https://github.com/llvm/llvm-project/pull/72280


More information about the cfe-commits mailing list