glaubitz added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/Arch/Sparc.cpp:139
+  if (Triple.getArch() == llvm::Triple::sparc &&
+      (Triple.isOSSolaris() || Distro(D.getVFS(), Triple).IsDebian()))
     return "v9";
----------------
Can we do it "IsLinux()" instead of "IsDebian()"?

I think Gentoo should profit from this change as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130688

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D130688:... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D13... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D13... Rainer Orth via Phabricator via cfe-commits
    • [PATCH] D13... Fangrui Song via Phabricator via cfe-commits
    • [PATCH] D13... Rainer Orth via Phabricator via cfe-commits

Reply via email to