No need to kill NaNs if the source is not a floating point format

Author: hkr

PostProcessing/Runtime/PostProcessLayer.cs

@@ -309,7 +309,10 @@ void OnPreRender()
         void BuildCommandBuffers()
         {
             var context = m_CurrentContext;
-            var sourceFormat = m_Camera.allowHDR ? RuntimeUtilities.defaultHDRRenderTextureFormat : RenderTextureFormat.Default; 
+            var sourceFormat = m_Camera.allowHDR ? RuntimeUtilities.defaultHDRRenderTextureFormat : RenderTextureFormat.Default;
+
+            if (!RuntimeUtilities.isFloatingPointFormat(sourceFormat))
+                m_NaNKilled = true;
 
             context.Reset();
             context.camera = m_Camera;
@@ -425,7 +428,8 @@ void BuildCommandBuffers()
             int tempRt = m_TargetPool.Get();
             context.GetScreenSpaceTemporaryRT(m_LegacyCmdBuffer, tempRt, 0, sourceFormat, RenderTextureReadWrite.sRGB);
             m_LegacyCmdBuffer.Blit(cameraTarget, tempRt, RuntimeUtilities.copyStdMaterial, stopNaNPropagation ? 1 : 0);
-            m_NaNKilled = stopNaNPropagation;
+            if (!m_NaNKilled)
+                m_NaNKilled = stopNaNPropagation;
 
             context.command = m_LegacyCmdBuffer;
             context.source = tempRt;

PostProcessing/Runtime/Utils/RuntimeUtilities.cs

@@ -305,6 +305,14 @@ public static RenderTextureFormat defaultHDRRenderTextureFormat
             }
         }
 
+        public static bool isFloatingPointFormat(RenderTextureFormat format)
+        {
+            return format == RenderTextureFormat.DefaultHDR || format == RenderTextureFormat.ARGBHalf || format == RenderTextureFormat.ARGBFloat ||
+                   format == RenderTextureFormat.RGFloat || format == RenderTextureFormat.RGHalf ||
+                   format == RenderTextureFormat.RFloat || format == RenderTextureFormat.RHalf ||
+                   format == RenderTextureFormat.RGB111110Float;
+        }
+
         public static void Destroy(UnityObject obj)
         {
             if (obj != null)