nemanjai added a comment. In D117181#3261396 <https://reviews.llvm.org/D117181#3261396>, @jsji wrote:
> So, the proposal is we change the default on Linux, so we will not change > the default based on the gcc version, it is based on OS and clang version > only. > but will emit warning if we detect that the GCC toolchain is too old. > > Can we just update bool IEEELongDouble = false; to bool IEEELongDouble = > T.isOSLinux(); in clang/lib/Driver/ToolChains/Clang.cpp? Won't that end up producing a warning on ALL code built on any Linux distro with a GCC toolchain older than 12.1? That would be terrible. Ultimately, what we care about is checking for the default for the distro toolchain. I am not sure what a good proxy for that is, but I don't think it is "all Linux distros". Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117181/new/ https://reviews.llvm.org/D117181 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
