================
@@ -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

Reply via email to