[PATCH] Remove log2 dependency when building against Android API-9 SDK
Pierre-Andre Saulais
pierre-andre at codeplay.com
Thu May 21 03:17:35 PDT 2015
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:
1. Use 'log(n) / log(2)' instead of log2l on Android, don't use HAVE_LOG2 at all.
2. 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.
REPOSITORY
rL LLVM
http://reviews.llvm.org/D9884
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list