[x265] [PATCH] analysis: removed gcc warnings from RD 0/4 performance improvement series patches

[Steve Borho]

On Mon, Apr 20, 2015 at 7:49 AM,  <ashok at multicorewareinc.com> wrote:
> # HG changeset patch
> # User Ashok Kumar Mishra<ashok at multicorewareinc.com>
> # Date 1429529880 -19800
> #      Mon Apr 20 17:08:00 2015 +0530
> # Node ID 03773bdde765179f9114fcc8d14c235b1b376ded
> # Parent  8fb7e41279ea1390d4e21ce0fa1e14257be0cdf0
> analysis: removed gcc warnings from RD 0/4 performance improvement series patches
> This follow on patch has to be applied on the patch series which I sent on 13th April.

this needs to be rolled into the appropriate patch in the series, and
the series needs to be rebased onto the latest tip since fine-grained
AQ modified some key logic in these functions.

> diff -r 8fb7e41279ea -r 03773bdde765 source/encoder/analysis.cpp
> --- a/source/encoder/analysis.cpp       Mon Apr 20 17:08:00 2015 +0530
> +++ b/source/encoder/analysis.cpp       Mon Apr 20 17:08:00 2015 +0530
> @@ -966,7 +966,9 @@
>                          checkBestMode(md.pred[PRED_INTRA], depth);
>                      }
>                      else
> +                    {
>                          ProfileCounter(parentCTU, skippedIntraCU[cuGeom.depth]);
> +                    }
>                  }
>              }
>              else
> @@ -990,7 +992,9 @@
>                              md.bestMode = &md.pred[PRED_INTRA];
>                      }
>                      else
> +                    {
>                          ProfileCounter(parentCTU, skippedIntraCU[cuGeom.depth]);
> +                    }
>                  }
>
>                  /* finally code the best mode selected by SA8D costs:
> @@ -1114,8 +1118,7 @@
>          CUData& cu = md.bestMode->cu;
>          PartSize partSize = (PartSize)cu.m_partSize[0];
>          uint32_t numPU = (partSize == SIZE_2Nx2N) ? 1 : 2;
> -        uint32_t depth = cu.m_cuDepth[0];
> -        uint32_t puOffset = (g_puOffset[uint32_t(partSize)] << (g_unitSizeDepth - depth) * 2) >> 4;
> +        uint32_t puOffset = (g_puOffset[uint32_t(partSize)] << (g_unitSizeDepth - cu.m_cuDepth[0]) * 2) >> 4;
>          refMask = 0;
>          for (uint32_t puIdx = 0, subPartIdx = 0; puIdx < numPU; puIdx++, subPartIdx += puOffset)
>          {
> _______________________________________________
> x265-devel mailing list
> x265-devel at videolan.org
> https://mailman.videolan.org/listinfo/x265-devel



-- 
Steve Borho