[PATCH] D45736: [SimplifyLibcalls] Replace locked IO with unlocked IO

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 1 00:19:08 PDT 2018


xbolva00 added inline comments.


================
Comment at: lib/Analysis/TargetLibraryInfo.cpp:486
+  if (T.isOSLinux()) {
+    // available IO unlocked variants on Linux
+    TLI.setAvailable(LibFunc_getc_unlocked);
----------------
efriedma wrote:
> You probably also need to check T.isAndroid(), like the code above this.  (I haven't actually checked what bionic provides, but I'm guessing it doesn't have all of these.)
They will be available since Android P, so I will enable this for Android P and higher

https://android.googlesource.com/platform/bionic/+/master/docs/status.md


https://reviews.llvm.org/D45736





More information about the llvm-commits mailing list