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

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo 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
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D47002