[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