VP8 Build Fail for Android at Mac

[GoogleCodeExporter]

What is the expected behavior? What do you see instead?
- Build Succeed

What version are you using? On what operating system?
Mac (Mavericks) / NDK-r10b / libvpx Latest

Can you reproduce using the vpxdec or vpxenc tools? What command line are
you using?

Please provide any additional information below.
1. Clone VPx branch in jni forder
2. set config as below
    ./libvpx/configure --target=armv7-android-gcc --disable-examples --sdk-path=/Users/minhohihi/DevTool/android-ndk-r10b --disable-runtime-cpu-detect --disable-vp9
3. run 'ndk-build'

[armeabi-v7a] Compile arm    : vpx <= variance_neon.c
[armeabi-v7a] Compile arm    : vpx <= vp8_subpixelvariance_neon.c
[armeabi-v7a] Compile arm    : vpx <= denoising_neon.c
[armeabi-v7a] Compile arm    : vpx <= shortfdct_neon.c
[armeabi-v7a] Compile arm    : vpx <= subtract_neon.c
[armeabi-v7a] Compile arm    : vpx <= vp8_mse16x16_neon.c
[armeabi-v7a] Compile arm    : vpx <= vp8_shortwalsh4x4_neon.c
jni/libvpx/vp8/encoder/arm/neon/vp8_shortwalsh4x4_neon.c: In function 
'vp8_short_walsh4x4_neon':
jni/libvpx/vp8/encoder/arm/neon/vp8_shortwalsh4x4_neon.c:118:1: internal 
compiler error: in trunc_int_for_mode, at explow.c:56
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://source.android.com/source/report-bugs.html> for instructions.
make: *** 
[obj/local/armeabi-v7a/objs/vpx/libvpx/vp8/encoder/arm/neon/vp8_shortwalsh4x4_ne
on.o] Error 1

That file 'vp8/encoder/arm/neon/vp8_shortwalsh4x4_neon.c' is added at Aug, 18.
I did rollback the branch before Aug. 18.
Then Build succeed with latest ndk (r10b).

Related issue: 
https://code.google.com/p/webrtc/issues/detail?can=2&start=0&num=100&q=&colspec=
ID%20Pri%20Mstone%20ReleaseBlock%20Area%20Status%20Owner%20Summary&groupby=&sort
=&id=3838

Original issue reported on code.google.com by minhohihi@gmail.com on 22 Sep 2014 at 2:15

[GoogleCodeExporter]

Original comment by fgalli...@google.com on 22 Sep 2014 at 5:05

[GoogleCodeExporter]

Issue webrtc:3838 has been merged into this issue.

Original comment by braveyao@webrtc.org on 22 Sep 2014 at 7:04

[GoogleCodeExporter]

While I'm looking into this, would you please try adding:
NDK_TOOLCHAIN_VERSION=4.8
to your ndk-build command?

Original comment by johannko...@google.com on 23 Sep 2014 at 6:30

• Changed state: Started

[GoogleCodeExporter]

Your comment is working for NDK r10b.
Thank you.
Could you explain the reason?

Original comment by minhohihi@gmail.com on 24 Sep 2014 at 2:09

[GoogleCodeExporter]

The gcc4.6 compiler has an issue with the intrinsics. We're not going back to 
assembly because intrinsics get us 64bit support for the neon code while 
maintaining a single code path. The setting above forces the NDK to use the 
gcc4.8 compiler which does not crash when compiling the function.

Original comment by johannko...@google.com on 24 Sep 2014 at 2:54

[GoogleCodeExporter]

https://gerrit.chromium.org/gerrit/#/c/71663/

Original comment by johannko...@google.com on 25 Sep 2014 at 8:39

• Changed state: Fixed