[clang] ab1722a - [Clang] Remove NetBSD/i386 workaround for FP eval method with older versions (#74025)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 22 16:31:04 PDT 2024


Author: Brad Smith
Date: 2024-07-22T19:31:01-04:00
New Revision: ab1722aaabb454111d65cce60cdb8735631d5114

URL: https://github.com/llvm/llvm-project/commit/ab1722aaabb454111d65cce60cdb8735631d5114
DIFF: https://github.com/llvm/llvm-project/commit/ab1722aaabb454111d65cce60cdb8735631d5114.diff

LOG: [Clang] Remove NetBSD/i386 workaround for FP eval method with older versions (#74025)

NetBSD 6.x is long EoL. Make 7.x the minimum and even that is EoL.

Added: 
    

Modified: 
    clang/docs/ReleaseNotes.rst
    clang/lib/Basic/Targets/X86.h

Removed: 
    


################################################################################
diff  --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index 9a07ad42b8f6f..d03459f7cc42c 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -1258,6 +1258,11 @@ AIX Support
   This access sequence is not used for TLS variables larger than 32KB, and is
   currently only supported on 64-bit mode.
 
+NetBSD Support
+^^^^^^^^^^^^^^
+
+- Removed support for building NetBSD/i386 6.x or older binaries.
+
 WebAssembly Support
 ^^^^^^^^^^^^^^^^^^^
 

diff  --git a/clang/lib/Basic/Targets/X86.h b/clang/lib/Basic/Targets/X86.h
index cdec41afd1a4b..ba34ab2c7f336 100644
--- a/clang/lib/Basic/Targets/X86.h
+++ b/clang/lib/Basic/Targets/X86.h
@@ -513,15 +513,6 @@ class LLVM_LIBRARY_VISIBILITY NetBSDI386TargetInfo
 public:
   NetBSDI386TargetInfo(const llvm::Triple &Triple, const TargetOptions &Opts)
       : NetBSDTargetInfo<X86_32TargetInfo>(Triple, Opts) {}
-
-  LangOptions::FPEvalMethodKind getFPEvalMethod() const override {
-    VersionTuple OsVersion = getTriple().getOSVersion();
-    // New NetBSD uses the default rounding mode.
-    if (OsVersion >= VersionTuple(6, 99, 26) || OsVersion.getMajor() == 0)
-      return X86_32TargetInfo::getFPEvalMethod();
-    // NetBSD before 6.99.26 defaults to "double" rounding.
-    return LangOptions::FPEvalMethodKind::FEM_Double;
-  }
 };
 
 class LLVM_LIBRARY_VISIBILITY OpenBSDI386TargetInfo


        


More information about the cfe-commits mailing list