[PATCH] D140613: [APFloat] Remove workaround for old clang.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 23 10:47:44 PST 2022


This revision was automatically updated to reflect the committed changes.
Closed by commit rG462a31f5a5ab: [APFloat] Remove workaround for old clang. (authored by craig.topper).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140613

Files:
  llvm/include/llvm/ADT/APFloat.h


Index: llvm/include/llvm/ADT/APFloat.h
===================================================================
--- llvm/include/llvm/ADT/APFloat.h
+++ llvm/include/llvm/ADT/APFloat.h
@@ -859,13 +859,6 @@
     APFLOAT_DISPATCH_ON_SEMANTICS(makeSmallestNormalized(Neg));
   }
 
-  // FIXME: This is due to clang 3.3 (or older version) always checks for the
-  // default constructor in an array aggregate initialization, even if no
-  // elements in the array is default initialized.
-  APFloat() : U(IEEEdouble()) {
-    llvm_unreachable("This is a workaround for old clang.");
-  }
-
   explicit APFloat(IEEEFloat F, const fltSemantics &S) : U(std::move(F), S) {}
   explicit APFloat(DoubleAPFloat F, const fltSemantics &S)
       : U(std::move(F), S) {}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140613.485144.patch
Type: text/x-patch
Size: 751 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221223/1d6a19fb/attachment.bin>


More information about the llvm-commits mailing list