[PATCH] D55916: [clang] Replace getOS() == llvm::Triple::*BSD with isOS*BSD() [NFCI]

[Michał Górny via Phabricator via cfe-commits]

mgorny added inline comments.


================
Comment at: lib/Basic/Targets/ARM.cpp:285
     default:
-      if (Triple.getOS() == llvm::Triple::NetBSD)
+      if (Triple.isOSNetBSD())
         setABI("apcs-gnu");
----------------
krytarowski wrote:
> Actually we could use IsNetBSD and IsOpenBSD here.
Good catch. Thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55916/new/

https://reviews.llvm.org/D55916



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits