Fix atan2 argument order for HLSL (#1712)

Signed-off-by: Doug Walker <[email protected]> Signed-off-by: Doug Walker <[email protected]> (cherry picked from commit 8a93946)
AcademySoftwareFoundation · Jan 5, 2023 · f86ef4e · f86ef4e
1 parent ffeda7d
commit f86ef4e
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/OpenColorIO/GpuShaderUtils.cpp b/src/OpenColorIO/GpuShaderUtils.cpp
@@ -1072,8 +1072,10 @@ std::string GpuShaderText::atan2(const std::string & y,
         }
         case GPU_LANGUAGE_HLSL_DX11:
         {
-            // note: operand order is swapped in HLSL
-            kw << "atan2(" << x << ", " << y << ")";
+            // note: Various internet sources claim that the x & y arguments need to be
+            // swapped for HLSL (relative to GLSL).  However, recent testing on Windows
+            // has revealed that the argument order needs to be the same as GLSL.
+            kw << "atan2(" << y << ", " << x << ")";
             break;
         }
         case LANGUAGE_OSL_1: