[PATCH] D47002: [Analysis] Only use _unlocked stdio functions on linux

[Stephen Hines via Phabricator via llvm-commits]

srhines added inline comments.


================
Comment at: lib/Analysis/TargetLibraryInfo.cpp:485
 
-  if (T.isGNUEnvironment() || (T.isAndroid() && !T.isAndroidVersionLT(28))) {
+  if (T.isOSLinux() || (T.isAndroid() && !T.isAndroidVersionLT(28))) {
     // available IO unlocked variants on GNU/Linux and Android P or later
----------------
mstorsjo wrote:
> xbolva00 wrote:
> > I would leave isLinux && isGNU...
> Yeah, I thought about that as well - but the rest of this function just plainly does isOSLinux everywhere, even if comments talk about glibc specifically. But I can make it Linux && GNU here for extra clarity.
isOSLinux is true for Android, isn't it? Doesn't that break the rest of the case for < API 28 on Android?


Repository:
  rL LLVM

https://reviews.llvm.org/D47002