[PATCH] D117181: [PowerPC] Use IEEE long double in proper toolchain

Jinsong Ji via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 21 12:40:57 PST 2022


jsji added a comment.

> 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;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117181



More information about the cfe-commits mailing list