Android's API-9 SDK is missing log2 builtins. A previous commit added support for building against this API revision but this requires log2l to be present. This patch removes the dependency on log2l when HAVE_LOG2 is zero.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
This commit doesn't really make sense to me as at no point does it define HAVE_LOG2....
This comment was removed by pasaulais.
Comment Actions
That's true, this commit would require manually setting HAVE_LOG2 to zero on Android. Since HAVE_LOG2 and other macros have been dropped (r231679) I was trying not to pull the autoconf/CMake changes back. I see two solutions:
- Use 'log(n) / log(2)' instead of log2l on Android, don't use HAVE_LOG2 at all.
- Reintroduce HAVE_LOG2 through CMake and autoconf. Remove the Android-specific changes added by Vince.
I am uploading a new patch that does 1) since that's the least intrusive change.