Commit ee20354b authored by Timothy Gu's avatar Timothy Gu

pixblockdsp: Use AV_COPY128U for get_pixels_16_c

Before:
  15543 decicycles in get_pixels, 4193214 runs,   1090 skips
After:
   5713 decicycles in get_pixels, 8387564 runs,   1044 skips
parent 507b01f8
......@@ -20,15 +20,42 @@
#include "config.h"
#include "libavutil/attributes.h"
#include "libavutil/intreadwrite.h"
#include "avcodec.h"
#include "pixblockdsp.h"
#define BIT_DEPTH 16
#include "pixblockdsp_template.c"
#undef BIT_DEPTH
static void get_pixels_16_c(int16_t *av_restrict block, const uint8_t *pixels,
ptrdiff_t line_size)
{
AV_COPY128U(block + 0 * 8, pixels + 0 * line_size);
AV_COPY128U(block + 1 * 8, pixels + 1 * line_size);
AV_COPY128U(block + 2 * 8, pixels + 2 * line_size);
AV_COPY128U(block + 3 * 8, pixels + 3 * line_size);
AV_COPY128U(block + 4 * 8, pixels + 4 * line_size);
AV_COPY128U(block + 5 * 8, pixels + 5 * line_size);
AV_COPY128U(block + 6 * 8, pixels + 6 * line_size);
AV_COPY128U(block + 7 * 8, pixels + 7 * line_size);
}
static void get_pixels_8_c(int16_t *av_restrict block, const uint8_t *pixels,
ptrdiff_t line_size)
{
int i;
#define BIT_DEPTH 8
#include "pixblockdsp_template.c"
/* read the pixels */
for (i = 0; i < 8; i++) {
block[0] = pixels[0];
block[1] = pixels[1];
block[2] = pixels[2];
block[3] = pixels[3];
block[4] = pixels[4];
block[5] = pixels[5];
block[6] = pixels[6];
block[7] = pixels[7];
pixels += line_size;
block += 8;
}
}
static void diff_pixels_c(int16_t *av_restrict block, const uint8_t *s1,
const uint8_t *s2, int stride)
......
/*
* This file is part of FFmpeg.
*
* FFmpeg is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 2.1 of the License, or (at your option) any later version.
*
* FFmpeg is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with FFmpeg; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*/
#include "bit_depth_template.c"
static void FUNCC(get_pixels)(int16_t *av_restrict block, const uint8_t *_pixels,
ptrdiff_t line_size)
{
const pixel *pixels = (const pixel *) _pixels;
int i;
/* read the pixels */
for (i = 0; i < 8; i++) {
block[0] = pixels[0];
block[1] = pixels[1];
block[2] = pixels[2];
block[3] = pixels[3];
block[4] = pixels[4];
block[5] = pixels[5];
block[6] = pixels[6];
block[7] = pixels[7];
pixels += line_size / sizeof(pixel);
block += 8;
}
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment