From 0efc782e693fa710e217b375be60d3674e65ce76 Mon Sep 17 00:00:00 2001
From: Tao Sang <tao.sang@amd.com>
Date: Wed, 27 Jul 2022 12:06:23 -0400
Subject: [PATCH] SWDEV-347733 -  Remove luxmark workaround for gfx11

Make gfx11 always use LC by default.
Fix switch order of Navi3x.
Change-Id: I5774563edf4d1aed29a4e92723630adbfb75c48c


[ROCm/clr commit: 8c582a02ef2063ddedc31255decac2edb0690382]
---
 .../clr/rocclr/device/pal/palsettings.cpp     | 24 +++++++++----------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/projects/clr/rocclr/device/pal/palsettings.cpp b/projects/clr/rocclr/device/pal/palsettings.cpp
index f5ad8a1987..1ae4ba70f8 100644
--- a/projects/clr/rocclr/device/pal/palsettings.cpp
+++ b/projects/clr/rocclr/device/pal/palsettings.cpp
@@ -199,6 +199,11 @@ bool Settings::create(const Pal::DeviceProperties& palProp,
   amd::Os::getAppPathAndFileName(appName, appPathAndName);
 
   switch (palProp.revision) {
+    // Fall through for Navi3x ...
+    case Pal::AsicRevision::Navi33:
+    case Pal::AsicRevision::Navi32:
+    case Pal::AsicRevision::Navi31:
+    // Fall through for Navi2x ...
     case Pal::AsicRevision::Phoenix1:
     case Pal::AsicRevision::Raphael:
     case Pal::AsicRevision::Rembrandt:
@@ -214,23 +219,18 @@ bool Settings::create(const Pal::DeviceProperties& palProp,
           useWavefront64 = true;
         }
       }
+    // Fall through for Navi1x ...
     case Pal::AsicRevision::Navi14:
     case Pal::AsicRevision::Navi12:
     case Pal::AsicRevision::Navi10:
     case Pal::AsicRevision::Navi10_A0:
-    case Pal::AsicRevision::Navi31:
-    case Pal::AsicRevision::Navi32:
-    case Pal::AsicRevision::Navi33:
       gfx10Plus_ = true;
-      // Force luxmark to use HSAIL
-      {
-        if ((appName == "luxmark.exe") ||
-            (appName == "luxmark")) {
-          useLightning_ = flagIsDefault(GPU_ENABLE_LC) ? false : GPU_ENABLE_LC;
-        }
-        else {
-          useLightning_ = GPU_ENABLE_LC;
-        }
+      useLightning_ = GPU_ENABLE_LC;
+      // Force luxmark to use HSAIL for gfx10 if GPU_ENABLE_LC isn't set in ENV
+      if (flagIsDefault(GPU_ENABLE_LC) &&
+          palProp.gfxLevel < Pal::GfxIpLevel::GfxIp11_0 &&
+          (appName == "luxmark.exe" || appName == "luxmark")) {
+        useLightning_ = false;
       }
       enableWgpMode_ = GPU_ENABLE_WGP_MODE;
       if (useLightning_) {