OSDN Git Service

avcenc: Clarify the ifdefs surrounding inline assembly
authorMartin Storsjo <martin@martin.st>
Fri, 13 Apr 2012 11:23:26 +0000 (14:23 +0300)
committerMartin Storsjo <martin@martin.st>
Tue, 10 Jul 2012 11:16:02 +0000 (14:16 +0300)
commitf5af6314db25ff3bef9bd2eeba201bc6cc60805d
tree85b8a3ecc385390a00e1709092de6873725cc7dc
parentffb829430ff20ccd6c13e6ed894f2373b2d93939
avcenc: Clarify the ifdefs surrounding inline assembly

Previously, the code was structured like this:

#if defined(__GNUC__) && defined(__arm__)
    /* Generic C implementation */
#elif defined(__CC_ARM)
    /* RVCT version */
#elif defined(__GNUC__) && defined(__arm__)
    /* GCC arm assembly version */
#endif

This had two implications - the gcc arm version never was used
in practice, and the code required -D__arm__ to build at all for
other architectures.

The inline assembly constraints are buggy (requires changes so as
to not crash), and if fixed, the optimizations still aren't faster
than the generic C code on modern gcc versions. Therefore, just
ignore them and use the generic C code.

Due to the peculiar code block ordering and preprocessor conditions,
the generic C code is what has been used before anyway, but it
did require -D__arm__ to build, which can cause problems if building
for other architectures, if the system headers have ifdefs for this
define. This change makes it build without that define.

Change-Id: Ib3fcf5651ee720310b7dff10fd04279b5f90759c
media/libstagefright/codecs/avc/enc/Android.mk
media/libstagefright/codecs/avc/enc/src/sad_halfpel_inline.h
media/libstagefright/codecs/avc/enc/src/sad_inline.h
media/libstagefright/codecs/avc/enc/src/sad_mb_offset.h