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

NDK 19 and 20 support #8524

Closed
wants to merge 5 commits into from
Closed

NDK 19 and 20 support #8524

wants to merge 5 commits into from

Conversation

jin
Copy link
Member

@jin jin commented May 30, 2019

Successfully tested to build Tensorflow's Android app on 4f9ae09cff5f357eb8260c468fbf22828f58e4b6 (HEAD today) with bazel-dev build //tensorflow/examples/android:tensorflow_demo --cxxopt='--std=c++11' -c opt --fat_apk_cpu=x86,armeabi-v7a

This change includes:

  • Support for building with NDK 19 and 20
  • Simplifying crosstool building by centralizing common configuration in AndroidNdkCrosstoolsR19.
  • Applying recommended flags documented in the NDK Build System Maintainer's Guide

This change doesn't contain support for lld or use the new standalone GCC tools yet. These will come in a follow up PR.

RELNOTES: Added support for Android NDK 19 and 20.

jin added 5 commits May 30, 2019 19:13
Change-Id: Ia52bd1352b2c2b95628adbbc324f18dc4be367d9
Change-Id: Ib4611868b6fe747b2db5472bc8150e6284762f8c
Change-Id: I7ee2674e2252473e1cc6b0edc596976f72e4e241
Change-Id: I306ce443ac7ee93319efb9ae53c9c7e59a9ae006
Change-Id: I800b3661b24057316cbb537d51e6ba346cc30f45
@jin jin requested a review from ahumesky as a code owner May 30, 2019 23:43
@jin jin mentioned this pull request May 30, 2019
@steeve
Copy link
Contributor

steeve commented May 31, 2019

That was fast! Thank you !

@bazel-io bazel-io closed this in 00e29b7 May 31, 2019
@jin jin mentioned this pull request May 31, 2019
irengrig pushed a commit to irengrig/bazel that referenced this pull request Jun 18, 2019
Successfully tested to build Tensorflow's Android app on `4f9ae09cff5f357eb8260c468fbf22828f58e4b6` (HEAD today) with `bazel-dev build //tensorflow/examples/android:tensorflow_demo --cxxopt='--std=c++11' -c opt --fat_apk_cpu=x86,armeabi-v7a`

This change includes:

* Support for building with NDK 19 and 20
* Simplifying crosstool building by centralizing common configuration in `AndroidNdkCrosstoolsR19`.
* Applying recommended flags documented in the [NDK Build System Maintainer's Guide](https://android.googlesource.com/platform/ndk/+/ndk-release-r19/docs/BuildSystemMaintainers.md#build-system-maintainers-guide)

This change doesn't contain support for `lld` or use the new standalone GCC tools yet. These will come in a follow up PR.

RELNOTES: Added support for Android NDK 19 and 20.

Closes bazelbuild#8524.

Change-Id: I800b3661b24057316cbb537d51e6ba346cc30f45
PiperOrigin-RevId: 250911997
irengrig pushed a commit to irengrig/bazel that referenced this pull request Jul 15, 2019
Successfully tested to build Tensorflow's Android app on `4f9ae09cff5f357eb8260c468fbf22828f58e4b6` (HEAD today) with `bazel-dev build //tensorflow/examples/android:tensorflow_demo --cxxopt='--std=c++11' -c opt --fat_apk_cpu=x86,armeabi-v7a`

This change includes:

* Support for building with NDK 19 and 20
* Simplifying crosstool building by centralizing common configuration in `AndroidNdkCrosstoolsR19`.
* Applying recommended flags documented in the [NDK Build System Maintainer's Guide](https://android.googlesource.com/platform/ndk/+/ndk-release-r19/docs/BuildSystemMaintainers.md#build-system-maintainers-guide)

This change doesn't contain support for `lld` or use the new standalone GCC tools yet. These will come in a follow up PR.

RELNOTES: Added support for Android NDK 19 and 20.

Closes bazelbuild#8524.

Change-Id: I800b3661b24057316cbb537d51e6ba346cc30f45
PiperOrigin-RevId: 250911997
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants