[Mlir-commits] [clang] [flang] [mlir] [flang][flang-driver][mlir][OpenMP] atomic control support (PR #143441)

[Sergio Afonso]

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