[PATCH] D52025: [TargetLowering] Android has sincos functions

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 added inline comments.


================
Comment at: lib/CodeGen/TargetLoweringBase.cpp:165
-  if (TT.isGNUEnvironment() || TT.isOSFuchsia()) {
+  if (TT.isGNUEnvironment() || TT.isOSFuchsia() ||
+      (TT.isAndroid() && !TT.isAndroidVersionLT(9))) {
     setLibcallName(RTLIB::SINCOS_F32, "sincosf");
     setLibcallName(RTLIB::SINCOS_F64, "sincos");
----------------
I think there should be SDK version.. sdk 28.


Repository:
  rL LLVM

https://reviews.llvm.org/D52025