[llvm-branch-commits] [clang] [Clang][NFC] Introduce LanguageID::HIP_LANG and reclassify AtomicBuitins (PR #185403)

[Sameer Sahasrabuddhe via llvm-branch-commits]

ssahasra wrote:

> > Yes. I do realize now that I didn't implement/verify that check. Wondering what the fallout would be!
> 
> This would be the best case, and similarly not defining the memory scopes if these builtins are not available. But I can imagine that may be a little too destructive to work out.

If `__hip_atomic*` builtins are only allowed when the HIP language is enabled, AND those builtins disallow the general `__memory_scope` enum, then that's a start. It decouples the question of deprecating `__hip_atomic` builtins from doing the right thing regarding Clang scoped atomics.

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