The builtins library name is special on Android:
- There is an "-android" suffix.
- For the compiler-rt i386 architecture, Android targets i686 (in the triple and in the builtins library filename)
With this change, check-builtins works with Android.
Differential D82149
[compiler-rt][Android] Use correct builtins library name for tests rprichard on Jun 18 2020, 11:55 PM. Authored by
Details The builtins library name is special on Android:
With this change, check-builtins works with Android.
Diff Detail
Unit Tests Event TimelineComment Actions FWIW, to run check-builtins, I followed the instructions at https://llvm.org/docs/HowToCrossCompileBuiltinsOnArm.html, but adjusted them to use the NDK to build compiler-rt. Each test is a simple dynamic executable built with something like -nodefaultlibs -lc -lm, and I set -DCOMPILER_RT_EMULATOR to a script I have that runs a program using adb push and adb shell. To run the tests on arm32, I also needed to make this change, D82147. Some of the floating-point tests also failed, and were fixed by my other 80-bit FP revisions. |