[clang] [llvm] AMDGPU: Add builtin/intrinsic global_(load|store)_b128 (PR #172090)

[Matt Arsenault via cfe-commits]

arsenm wrote:

> I think using a string for a user-facing interface is a bad design. The Sema check also doesn't check whether the scope string is valid or not. This should be something like a memory order value for those GCC atomic builtins.

Unfortunately we're stuck with a mixed set of atomic builtins with an enum and others with a string scope name as it is. I'm not sure if it's better to double down on one direction or the other 

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