[Offload][AMDGPU] fix double inversion of coarse grain usm map logic

hidekisaito · hidekisaito · commit 43671d11aa7e · 2025-01-16T00:50:15.000-06:00
diff --git a/offload/plugins-nextgen/amdgpu/src/rtl.cpp b/offload/plugins-nextgen/amdgpu/src/rtl.cpp
@@ -4342,7 +4342,7 @@ struct AMDGPUDeviceTy : public GenericDeviceTy, AMDGenericDeviceTy {
   }
 
   bool IsGfx90aCoarseGrainUsmMapEnabledImpl() override final {
-    return !EnableGFX90ACoarseGrainUsmMaps;
+    return EnableGFX90ACoarseGrainUsmMaps;
   }
 
   bool hasAPUDeviceImpl() override final { return IsAPU; }
diff --git a/offload/src/OpenMP/Mapping.cpp b/offload/src/OpenMP/Mapping.cpp
@@ -274,8 +274,7 @@ TargetPointerResultTy MappingInfoTy::getTargetPointer(
       // memory as coarse-grained. The usage of coarse-grained memory can be
       // overriden by setting the env-var OMPX_DISABLE_USM_MAPS=1.
       if (Device.RTL->is_gfx90a(Device.DeviceID) && HstPtrBegin &&
-          (!Device.RTL->is_gfx90a_coarse_grain_usm_map_enabled(
-              Device.DeviceID))) {
+          Device.RTL->is_gfx90a_coarse_grain_usm_map_enabled(Device.DeviceID)) {
         Device.RTL->set_coarse_grain_mem_region(Device.DeviceID, HstPtrBegin,
                                                 Size);
         INFO(OMP_INFOTYPE_MAPPING_CHANGED, Device.DeviceID,

Original file line number	Diff line number	Diff line change
`@@ -4342,7 +4342,7 @@ struct AMDGPUDeviceTy : public GenericDeviceTy, AMDGenericDeviceTy {`
`4342`	`4342`	`}`
`4343`	`4343`
`4344`	`4344`	`bool IsGfx90aCoarseGrainUsmMapEnabledImpl() override final {`
`4345`		`- return !EnableGFX90ACoarseGrainUsmMaps;`
	`4345`	`+ return EnableGFX90ACoarseGrainUsmMaps;`
`4346`	`4346`	`}`
`4347`	`4347`
`4348`	`4348`	`bool hasAPUDeviceImpl() override final { return IsAPU; }`