Commit ccfdfce1 authored by Ralph Giles's avatar Ralph Giles Committed by Ralph Giles
Browse files

aom_dsp: regularize EXT_PARTITION_TYPES handling. compares most CONFIG_* keys to "yes"
to see if they're set. The script was checking just

  if (aom_config("CONFIG_EXT_PARTITION_TYPES"))

in some cases. The build system doesn't add disabled
configuration options to so this is effectively
the same, however it means that setting the config
key explicitly to 0 or "no" in the config headers
was treated the same as setting it to 1 or "yes",
and aom_dsp_rtcd.h would have opposite expections
from aom_config.h or aom_config.asm.

Treat this key similarly to others for consistency.

Change-Id: I27bd7a5532ba4afc2bb289b43b57a1b1971c0348
parent 9c5b1088
......@@ -40,7 +40,7 @@ foreach $w (@block_widths) {
push @block_sizes, [$w, $h] if ($w <= 2*$h && $h <= 2*$w) ;
if (aom_config("CONFIG_EXT_PARTITION_TYPES")) {
if (aom_config("CONFIG_EXT_PARTITION_TYPES") eq "yes") {
push @block_sizes, [4, 16];
push @block_sizes, [16, 4];
push @block_sizes, [8, 32];
......@@ -947,7 +947,7 @@ specialize qw/aom_sub_pixel_avg_variance8x4 msa sse2 ssse3/;
specialize qw/aom_sub_pixel_avg_variance4x8 msa sse2 ssse3/;
specialize qw/aom_sub_pixel_avg_variance4x4 msa sse2 ssse3/;
if (aom_config("CONFIG_EXT_PARTITION_TYPES")) {
if (aom_config("CONFIG_EXT_PARTITION_TYPES") eq "yes") {
specialize qw/aom_variance4x16 sse2/;
specialize qw/aom_variance16x4 sse2/;
specialize qw/aom_variance8x32 sse2/;
Supports Markdown
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