ro 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";
----------------
glaubitz wrote:
> Can we do it "IsLinux()" instead of "IsDebian()"?
>
> I think Gentoo should profit from this change as well.
No, for the reason I mentioned: there are Linux distributions with SPARC
support that **don't support SPARC v9**, like [[
https://www.gaisler.com/index.php/products/operating-systems/linux | LEON Linux
]]. Solaris is easy here because it's v9-only.
LLVM explicitly has extensive LEON support, so we shouldn't break it there!
I don't really known which Linux distributions which are considered supported
by current LLVM, have SPARC support at all and require v9 CPUs. That's for the
community to tell.
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
