qiucf added a comment. In D117181#3262288 <https://reviews.llvm.org/D117181#3262288>, @jsji wrote:
>> 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. > > Good point. Yes, so should be something like: > > IsDistroWithNewToolchain = ( Distro.IsRedhat() && Distro >= Distro::RHEL9 > || (Distro.IsUbuntu() && Distro >= Distro::xxx) > bool IEEELongDouble = T.isOSLinux() && IsDistroWithNewToolchain; Thanks for reminding about checking Linux distro version! Besides, would it be acceptable that we add a variable to cmake to determine default long double semantics (like current GCC)? 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
