================
@@ -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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits