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