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

Side by Side Diff: third_party/ffmpeg/munge_config_optimizations.sh

Issue 300013: Add yasm and ffmpeg into the build tree for linux. (Closed)
Patch Set: Fix mark's comments. Created 11 years, 1 month 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 unified diff | Download patch
OLDNEW
(Empty)
1 # Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file.
4
5 # This script munges the config.h to mark EBP unavailable so that the ffmpeg
6 # library can be compiled as WITHOUT -fomit-frame-pointer allowing for
7 # breakpad to work.
8 #
9 # Without this, building without -fomit-frame-pointer on ia32 will result in
10 # the the inclusion of a number of inline assembly blocks that use too many
11 # registers for its input/output operands. This will cause gcc to barf with:
12 #
13 # error: can't find a register in class ‘GENERAL_REGS’ while reloading ‘asm’
14 #
15 # This modification should only be required on ia32, and not x64.
16 #
17 # Note that HAVE_EBX_AVAILABLE is another flag available in config.h. One would
18 # think that setting this to 0 would allow for ffmpeg to be built with -fPIC.
19 # However, not all the assembly blocks requiring 6 registers are excluded by
20 # this flag.
21
22 set -e
23
24 sed -i.orig -e '
25 /HAVE_EBP_AVAILABLE/ {
26 c\
27 #define HAVE_EBP_AVAILABLE 0
28 }
29 ' \
30 $1
OLDNEW
« no previous file with comments | « third_party/ffmpeg/ffmpeg.gyp ('k') | third_party/ffmpeg/source/config/Chrome/linux/ia32/config.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698