Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(862)

Unified Diff: media/media.gyp

Issue 7003082: Implements RGB to YV12 conversion in YASM. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« media/base/yuv_convert.cc ('K') | « media/base/yuv_convert_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/media.gyp
===================================================================
--- media/media.gyp (revision 91457)
+++ media/media.gyp (working copy)
@@ -6,6 +6,7 @@
'variables': {
'chromium_code': 1,
'player_x11_renderer%': 'x11',
+ 'use_system_yasm%': 0,
Alpha Left Google 2011/08/22 14:16:59 We can drop this and assume we always use our own
},
'targets': [
{
@@ -343,10 +344,94 @@
},
},
}],
+ [ 'target_arch=="ia32" or target_arch=="x64"', {
+ 'sources': [
+ 'base/simd/yuv_row.asm',
+ ],
+ }],
+ [ 'OS=="win"', {
+ 'variables': {
+ 'object_suffix': 'obj',
+ 'yasm_path': '../third_party/yasm/binaries/win/yasm<(EXECUTABLE_SUFFIX)',
Alpha Left Google 2011/08/22 14:16:59 We can always depend on yasm#host now on all platf
+ 'yasm_format': '-fwin32',
+ 'yasm_flags': [
+ '-DWIN32',
+ '-DMSVC',
+ ],
+ },
+ }],
+ [ 'OS=="mac"', {
+ 'dependencies': [
+ '../third_party/yasm/yasm.gyp:yasm#host',
+ ],
+ 'variables': {
+ 'object_suffix': 'o',
+ 'yasm_path': '<(PRODUCT_DIR)/yasm',
+ 'yasm_format': '-fmacho',
+ 'yasm_flags': [
+ '-DMACHO',
+ ],
+ },
+ }],
+ [ 'OS=="linux"', {
+ 'conditions': [
+ [ 'use_system_yasm==0', {
+ 'dependencies': [
+ '../third_party/yasm/yasm.gyp:yasm#host',
+ ],
+ }],
+ ],
+ 'variables': {
+ 'object_suffix': 'o',
+ 'conditions': [
+ [ 'use_system_yasm==1', {
+ 'yasm_path': '<!(which yasm)',
+ }, {
+ 'yasm_path': '<(PRODUCT_DIR)/yasm',
+ }],
+ [ 'target_arch=="ia32"', {
+ 'yasm_format': '-felf',
+ 'yasm_flag': '-D__X86__',
+ 'yasm_flags': [
+ '-D__x86__',
+ '-DELF',
+ ],
+ }, {
+ 'yasm_format': '-felf64',
+ 'yasm_flag': '-D__x86_64__',
+ 'yasm_flags': [
+ '-D__x86_64__',
+ '-DELF',
+ ],
+ }],
+ ],
+ },
+ }],
],
'sources': [
'base/yuv_convert_sse2.cc',
+ 'base/simd/yuv_convert.cc',
],
+ 'rules': [
+ {
+ 'rule_name': 'assemble',
+ 'extension': 'asm',
+ 'inputs': [ '<(yasm_path)', ],
+ 'outputs': [
+ '<(SHARED_INTERMEDIATE_DIR)/media/<(RULE_INPUT_ROOT).<(object_suffix)',
+ ],
+ 'action': [
+ '<(yasm_path)',
+ '<(yasm_format)',
+ '<@(yasm_flags)',
+ '-Isimd/',
+ '-o', '<(SHARED_INTERMEDIATE_DIR)/media/<(RULE_INPUT_ROOT).<(object_suffix)',
+ '<(RULE_INPUT_PATH)',
+ ],
+ 'process_outputs_as_sources': 1,
+ 'message': 'Building <(RULE_INPUT_ROOT).<(object_suffix)',
+ },
+ ],
},
{
'target_name': 'ffmpeg_unittests',
« media/base/yuv_convert.cc ('K') | « media/base/yuv_convert_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698