================ @@ -53,6 +53,11 @@ class TargetOptions { /// Print verbose assembly bool asmVerbose = false; + + /// Atomic control options for AMD gpu + bool ignoreDenormalMode = false; + bool remoteMemory = false; + bool fineGrainedMemory = false; ---------------- skatrak wrote:
Nit: I think it makes sense to include "atomic" within variables names and also associated getters/setters and attributes, to avoid confusion as to what these flags apply to. ```suggestion bool atomicIgnoreDenormalMode = false; bool atomicRemoteMemory = false; bool atomicFineGrainedMemory = false; ``` https://github.com/llvm/llvm-project/pull/143441 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits