https://github.com/AlexVlx commented:
> Is this only relevant for the __builtin_amdgcn_atomic* builtins? Those are a > bit of a legacy kludge. Now that inc/dec have been properly added to > atomicrmw, we should probably go back and add those operations to the > standard set of atomic builtins Yes, it's only them (including fences); non-target BIs / ops are fine. Unfortunately, they are used in various headers (for example, in HIP), so we'd have to first add inc dec, then make sure that various headers are updated to remove the use of the target builtins, before we can stop relying on these. https://github.com/llvm/llvm-project/pull/154867 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits