address review comment

Author: gtong-nv

tools/render-test/render-test-main.cpp

@@ -1566,35 +1566,33 @@ static SlangResult _innerMain(
             }
             else
             {
-                auto result = rhi::getRHI()->createDevice(desc, rhiDevice.writeRef());
-                if (SLANG_FAILED(result))
+                auto res = rhi::getRHI()->createDevice(desc, rhiDevice.writeRef());
+                if (SLANG_FAILED(res))
                 {
                     rhiDevice = nullptr;
+                    // We need to be careful here about SLANG_E_NOT_AVAILABLE. This return value means
+                    // that the renderer couldn't be created because it required *features* that were
+                    // *not available*. It does not mean the renderer in general couldn't be
+                    // constructed.
+                    //
+                    // Returning SLANG_E_NOT_AVAILABLE will lead to the test infrastructure ignoring
+                    // this test.
+                    //
+                    // We also don't want to output the 'Unable to create renderer' error, as this isn't
+                    // an error.
+                    if (res == SLANG_E_NOT_AVAILABLE)
+                    {
+                        return res;
+                    }
+                    if (!options.onlyStartup)
+                    {
+                        fprintf(stderr, "Unable to create renderer %s\n", rendererName.getBuffer());
+                    }
+                    return res;
                 }
             }
-            if (!rhiDevice)
-            {
-                // We need to be careful here about SLANG_E_NOT_AVAILABLE. This return value means
-                // that the renderer couldn't be created because it required *features* that were
-                // *not available*. It does not mean the renderer in general couldn't be
-                // constructed.
-                //
-                // Returning SLANG_E_NOT_AVAILABLE will lead to the test infrastructure ignoring
-                // this test.
-                //
-                // We also don't want to output the 'Unable to create renderer' error, as this isn't
-                // an error.
-                SlangResult res =
-                    SLANG_E_NOT_AVAILABLE; // Default to not available if device creation fails
-                if (!options.onlyStartup)
-                {
-                    fprintf(stderr, "Unable to create renderer %s\n", rendererName.getBuffer());
-                }
-
-                return res;
-            }
+            SLANG_ASSERT(rhiDevice);
             deviceWrapper = CachedDeviceWrapper(rhiDevice);
-            SLANG_ASSERT(deviceWrapper);
         }
 
         for (const auto& feature : requiredFeatureList)

tools/render-test/slang-test-device-cache.cpp

@@ -102,7 +102,7 @@ Slang::ComPtr<rhi::IDevice> DeviceCache::acquireDevice(const rhi::DeviceDesc& de
     key.deviceType = desc.deviceType;
     key.enableValidation = desc.enableValidation;
     key.enableRayTracingValidation = desc.enableRayTracingValidation;
-    key.profileName = desc.slang.targetProfile ? desc.slang.targetProfile : "";
+    key.profileName = desc.slang.targetProfile ? desc.slang.targetProfile : "Unknown";
 
     // Add required features to key
     for (int i = 0; i < desc.requiredFeatureCount; ++i)

tools/slang-test/slang-test-main.cpp

@@ -4776,6 +4776,11 @@ static bool shouldRunTest(TestContext* context, String filePath)
     {
         if (filePath.startsWith(excludePrefix))
         {
+            if (context->options.verbosity == VerbosityLevel::Verbose)
+            {
+                context->getTestReporter()->messageFormat(
+                    TestMessageType::Info, "%s file is excluded from the test because it is found from the exclusion list\n", filePath.getBuffer());
+            }
             return false;
         }
     }