[Openmp-commits] [openmp] [OpenMP][libomptarget] Enable automatic unified shared memory executi… (PR #75999)

[Johannes Doerfert via Openmp-commits]

================
@@ -2618,6 +2623,15 @@ struct AMDGPUDeviceTy : public GenericDeviceTy, AMDGenericDeviceTy {
     return Plugin::success();
   }
 
+  /// Returns true if auto zero-copy the best configuration for the current
+  /// arch.
+  bool useAutoZeroCopyImpl() override {
+    // XNACK can be enabled with with kernel boot parameter or with
+    // environment variable.
+    return (IsAPU &&
+            (utils::isXnackEnabledViaKernelParam() || HSAXnackEnv.get()));
----------------
jdoerfert wrote:

I would swap the latter two. The EnvVar is cheaper to check.

https://github.com/llvm/llvm-project/pull/75999