stephan.dollberg accepted this revision.
stephan.dollberg added a comment.
This revision is now accepted and ready to land.

LGTM but probably want to apply clang-format suggestion.



================
Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:2055
   if (SysRoot.empty() && TargetTriple.getOS() == llvm::Triple::Linux) {
+    // CentOS 8
+    Prefixes.push_back("/opt/rh/gcc-toolset-10/root/usr");
----------------
Comment only says CentOS while title also says RHEL. I assume it applies to 
both?

Might also want to merge with the comment below as they are kind of the same 
thing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108908

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

Reply via email to