Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable NEON for the arm64 architecture too #5924

Merged
merged 1 commit into from
Jan 11, 2016

Conversation

dulacp
Copy link
Contributor

@dulacp dulacp commented Jan 5, 2016

The compilation for iOS was broken due to this.

Leading to issues like this (for posterity) :

Undefined symbols for architecture arm64:
  "_png_init_filter_functions_neon", referenced from:
      _png_read_filter_row in opencv2(pngrutil.o)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

So now the build_framework.py force the NEON optimizations for the arm64 arch.

Cheers

and fix the compilation for iOS in the same time
@alalek
Copy link
Member

alalek commented Jan 11, 2016

👍

@opencv-pushbot opencv-pushbot merged commit 262a52f into opencv:master Jan 11, 2016
@benjamincarp
Copy link

Is there a downloadable build that includes this fix? I tried compiling locally following the instructions here but wound up with errors and the latest download on http://opencv.org/downloads.html says it is from December.

I found the page for the nightly builds but didn't see a way to download those.

@alalek
Copy link
Member

alalek commented Apr 19, 2016

Current artifacts from nightly build is here (with opencv_contrib)

@benjamincarp
Copy link

Thank you! I was on that page earlier today but it wasn't obvious how to (or if I could) download those builds.

@RishabGargeya
Copy link

Hi! I'm experiencing the same issues as dulaccc mentioned, and am hoping to find a downloadable build with the fix. How specifically can I download the build from the aforementioned website? Sorry if I missed something. Thank you so much!

@benjamincarp
Copy link

@RishabGargeya if you go to one of the iOS builds (http://pullrequest.opencv.org/buildbot/builders/master-contrib_iOS-mac/builds/264 works for me but there is a newer one from last night now too) then you should look at the very bottom left corner of the screen. The very last thing in the list under "Steps and Logfiles" is a link to the zip of that build. In this case it is http://pullrequest.opencv.org/buildbot/export/opencv_releases/master-contrib_iOS-mac/20160418-231924_264/

I hope that makes sense. That page is not very easy to understand at first glance but it does have what we need.

@RishabGargeya
Copy link

@benjamincarp Awesome! Thanks so much for the instructions -- I was able to download the build and successfully compile for iOS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants