Single toolchain for Android platform about polly HOT 7 CLOSED

How

> ANDROID_NDK_VERSION=r10e ANDROID_NATIVE_API_LEVEL=21 ANDROID_ABI=mips build.py --toolchain android

is better than just

> build.py --toolchain android-ndk-r10e-api-21-mips

?

from polly.

When you are targeting API 24 or using NDK 13 or any other combination (there are 100s - maybe 1000s).

Then I want to target clang as well. Do we produce 1000 more toolchain files instead of setting variable?

from polly.

Do we produce 1000 more toolchain files instead of setting variable?

You're using all 1000 variants? Combinatorial explosion discussed here and solution is out of scope of Polly functional (though I'm not 100% sure what it will looks like). For now workaround is to add only toolchains that you're really using.

from polly.

So every time I update my NDK, I should create new toolchains for every ABI I target (eg. armeabi-v7a, armeabi-v7a with NEON, arm64-v8a, x86_64)? Should I also commit it?
I have a machine with r13 and another with r13b.

What about local changes I need to make for toolchain such as exceptions/rtti? I edit every toolchain for this or create new one?

from polly.

So every time I update my NDK, I should create new toolchains for every ABI I target (eg. armeabi-v7a, armeabi-v7a with NEON, arm64-v8a, x86_64)?

Just like every time iOS version changed (ios-*), Visual Studio generator released (vs-*), OSX upgrade (osx-*), etc.

Should I also commit it?

You mean send pull request? Yes, if you want :)

What about local changes I need to make for toolchain such as exceptions/rtti?

It's a topic for separate discussion.

from polly.

Since r13, the NDK version can be detected by reading the $ANDROID_NDK/source.properties file. This is how the Android NDK toolchain does it too.
This should help remove one combination in future at least.
They could be named like so: android-ndk-r13+-api-21-mips

from polly.

Since r13, the NDK version can be detected by reading the $ANDROID_NDK/source.properties file

I guess this will not work for the cases when we need to have both r13x and r13y on the same machine.

from polly.