Change non_volatile to not is_volatile

Author: amd-eochoalo

mlir/include/mlir/Dialect/AMDGPU/IR/AMDGPU.td

@@ -1485,7 +1485,7 @@ def AMDGPU_TensorLoadToLDSOp :
   Arguments<(ins AMDGPU_TDMDescriptorType: $desc,
                  DefaultValuedOptionalAttr<AMDGPU_CacheScopeAttr, "CacheScope::Workgroup">: $cache_scope,
                  DefaultValuedOptionalAttr<AMDGPU_TemporalLoadHintsAttr, "TemporalLoadHints::RegularTemporal">: $temporal_hint,
-                 DefaultValuedOptionalAttr<BoolAttr, "false">: $non_volatile)> {
+                 DefaultValuedOptionalAttr<BoolAttr, "true">: $is_volatile)> {
   let summary = "Load tensors from global memory to LDS.";
   let description = [{
     Load tensors of up to five dimensions from global memory to LDS.
@@ -1503,7 +1503,7 @@ def AMDGPU_TensorStoreFromLDSOp :
   Arguments<(ins AMDGPU_TDMDescriptorType: $desc,
                  DefaultValuedOptionalAttr<AMDGPU_CacheScopeAttr, "CacheScope::Workgroup">: $cache_scope,
                  DefaultValuedOptionalAttr<AMDGPU_TemporalStoreHintsAttr, "TemporalStoreHints::RegularTemporal">: $temporal_hint,
-                 DefaultValuedOptionalAttr<BoolAttr, "false">: $non_volatile)> {
+                 DefaultValuedOptionalAttr<BoolAttr, "true">: $is_volatile)> {
 
   let summary = "Store tensors from LDS to global memory.";
   let description = [{

mlir/lib/Conversion/AMDGPUToROCDL/AMDGPUToROCDL.cpp

@@ -2723,9 +2723,9 @@ struct AMDGPUTensorLoadStoreOpLowering
 
     ValueRange desc = adaptor.getDesc();
     uint32_t temporalHint = static_cast<uint32_t>(op.getTemporalHint());
-    bool nonVolatile = static_cast<bool>(op.getNonVolatile());
+    bool isVolatile = !static_cast<bool>(op.getIsVolatile());
     uint32_t cacheScope = static_cast<uint32_t>(op.getCacheScope());
-    int32_t cachePolicy = cacheScope | temporalHint << 2 | nonVolatile << 5;
+    int32_t cachePolicy = cacheScope | temporalHint << 2 | isVolatile << 5;
 
     rewriter.replaceOpWithNewOp<TargetOp>(op, desc[0], desc[1], desc[2],
                                           desc[3], cachePolicy,

mlir/test/Conversion/AMDGPUToROCDL/gfx1250.mlir

@@ -485,10 +485,10 @@ func.func @tensor_load_to_lds(%desc: !amdgpu.tdm_descriptor) {
   amdgpu.tensor_load_to_lds %desc { temporal_hint = #amdgpu.temporal_load_hint<nontemporal_highpriority> } : !amdgpu.tdm_descriptor
 
   // CHECK: rocdl.tensor.load.to.lds %[[DGROUPS]]#0, %[[DGROUPS]]#1, %[[DGROUPS]]#2, %[[DGROUPS]]#3 cachepolicy 0 : vector<4xi32>, vector<8xi32>
-  amdgpu.tensor_load_to_lds %desc { non_volatile = false } : !amdgpu.tdm_descriptor
+  amdgpu.tensor_load_to_lds %desc { is_volatile = true } : !amdgpu.tdm_descriptor
 
   // CHECK: rocdl.tensor.load.to.lds %[[DGROUPS]]#0, %[[DGROUPS]]#1, %[[DGROUPS]]#2, %[[DGROUPS]]#3 cachepolicy 32 : vector<4xi32>, vector<8xi32>
-  amdgpu.tensor_load_to_lds %desc { non_volatile = true } : !amdgpu.tdm_descriptor
+  amdgpu.tensor_load_to_lds %desc { is_volatile = false } : !amdgpu.tdm_descriptor
 
   func.return
 }
@@ -537,10 +537,10 @@ func.func @tensor_store_from_lds(%desc: !amdgpu.tdm_descriptor) {
   amdgpu.tensor_store_from_lds %desc { temporal_hint = #amdgpu.temporal_store_hint<nontemporal_writeback> } : !amdgpu.tdm_descriptor
 
   // CHECK: rocdl.tensor.store.from.lds %[[DGROUPS]]#0, %[[DGROUPS]]#1, %[[DGROUPS]]#2, %[[DGROUPS]]#3 cachepolicy 0 : vector<4xi32>, vector<8xi32>
-  amdgpu.tensor_store_from_lds %desc { non_volatile = false } : !amdgpu.tdm_descriptor
+  amdgpu.tensor_store_from_lds %desc { is_volatile = true } : !amdgpu.tdm_descriptor
 
   // CHECK: rocdl.tensor.store.from.lds %[[DGROUPS]]#0, %[[DGROUPS]]#1, %[[DGROUPS]]#2, %[[DGROUPS]]#3 cachepolicy 32 : vector<4xi32>, vector<8xi32>
-  amdgpu.tensor_store_from_lds %desc { non_volatile = true } : !amdgpu.tdm_descriptor
+  amdgpu.tensor_store_from_lds %desc { is_volatile = false } : !amdgpu.tdm_descriptor
   func.return
 }