[llvm-branch-commits] [clang] [Clang] define memory scopes as a builtin enum (PR #185408)

[Joseph Huber via llvm-branch-commits]

jhuber6 wrote:

> Do you see any problem with informing the user that the function `fence()` should be redefined with a `__memory_scope` argument instead of `int`?

You would need to update the current usage in libc, libclc, and openmp and probably wherever else someone uses this in a function. It's not intuitive because why should this warning apply to the memory scope and not the order? And we can't change the former because it would break compatibility with GCC.

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