From 462a31f5a5abb905869ea93cc49b096079b11aa4 Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Fri, 23 Dec 2022 10:24:35 -0800 Subject: [PATCH] [APFloat] Remove workaround for old clang. The comment says this is for clang 3.3. Our build requirements are clang 5.0 or newer so I think we can remove this. Reviewed By: nikic, RKSimon Differential Revision: https://reviews.llvm.org/D140613 --- llvm/include/llvm/ADT/APFloat.h | 7 ------- 1 file changed, 7 deletions(-) diff --git a/llvm/include/llvm/ADT/APFloat.h b/llvm/include/llvm/ADT/APFloat.h index a3961442b93f7b..c0e2d13c293920 100644 --- a/llvm/include/llvm/ADT/APFloat.h +++ b/llvm/include/llvm/ADT/APFloat.h @@ -859,13 +859,6 @@ class APFloat : public APFloatBase { 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) {}