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

add includepath for atomic on ios #221

Closed
wants to merge 6 commits into from
Closed

Conversation

johanvos
Copy link
Contributor

Make sure can be found when compiling for ios (at least x86_64)

@saudet
Copy link
Member

saudet commented Dec 14, 2017

Does that work with the version of clang from Xcode?? And what about other architectures?

@saudet
Copy link
Member

saudet commented Dec 27, 2017

If this is to get libnd4j to build, we should fix this over there...

@johanvos
Copy link
Contributor Author

johanvos commented Jan 2, 2018

That is with

  • Apple LLVM version 9.0.0 (clang-900.0.39.2)
  • Xcode 9.2

This is indeed to build nd4j (libnd4j//blas/Environment.h includes )

@saudet
Copy link
Member

saudet commented Jan 2, 2018

Xcode doesn't officially support that. BTW, it also doesn't support OpenMP and that's why we use GCC on other platforms. Would there be a way to use GCC with iOS too?

@saudet
Copy link
Member

saudet commented Jan 16, 2018

Let's make this modification in ND4J, see pull https://github.com/deeplearning4j/nd4j/pull/2538.

@saudet saudet closed this Jan 16, 2018
saudet added a commit that referenced this pull request Feb 13, 2018
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.

2 participants