Keep identity of FortranDecimal

Author: Meinersbur

flang/include/flang/Common/binary-floating-point.h renamed to flang/include/flang/Decimal/binary-floating-point.h

@@ -1,13 +1,13 @@
-//===-- include/flang/Common/binary-floating-point.h ------------*- C++ -*-===//
+//===-- include/flang/Decimal/binary-floating-point.h -----------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
-#ifndef FORTRAN_COMMON_BINARY_FLOATING_POINT_H_
-#define FORTRAN_COMMON_BINARY_FLOATING_POINT_H_
+#ifndef FORTRAN_DECIMAL_BINARY_FLOATING_POINT_H_
+#define FORTRAN_DECIMAL_BINARY_FLOATING_POINT_H_
 
 // Access and manipulate the fields of an IEEE-754 binary
 // floating-point value via a generalized template.
@@ -208,4 +208,4 @@ template <int BINARY_PRECISION> class BinaryFloatingPointNumber {
   RawType raw_{0};
 };
 } // namespace Fortran::decimal
-#endif /* FORTRAN_COMMON_BINARY_FLOATING_POINT_H_ */
+#endif

flang/include/flang/Common/decimal.h renamed to flang/include/flang/Decimal/decimal.h

@@ -1,15 +1,16 @@
-/*===-- include/flang/Common/decimal.h ------------------------------*- C -*-===
+/*===-- include/flang/Decimal/decimal.h ---------------------------*- C++ -*-===
  *
  * Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
  * See https://llvm.org/LICENSE.txt for license information.
  * SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
  *
- *===----------------------------------------------------------------------===*/
+ * ===-----------------------------------------------------------------------===
+ */
 
 /* C and C++ API for binary-to/from-decimal conversion package. */
 
-#ifndef FORTRAN_COMMON_DECIMAL_H_
-#define FORTRAN_COMMON_DECIMAL_H_
+#ifndef FORTRAN_DECIMAL_DECIMAL_H_
+#define FORTRAN_DECIMAL_DECIMAL_H_
 
 #include "flang/Common/api-attrs.h"
 #include <stddef.h>
@@ -136,4 +137,4 @@ RT_API_ATTRS enum NS(ConversionResultFlags) ConvertDecimalToLongDouble(
 #ifdef __cplusplus
 } // extern "C"
 #endif
-#endif /* FORTRAN_COMMON_DECIMAL_H_ */
+#endif

flang/lib/CMakeLists.txt

@@ -1,5 +1,6 @@
 add_subdirectory(Common)
 add_subdirectory(Evaluate)
+add_subdirectory(Decimal)
 add_subdirectory(Lower)
 add_subdirectory(Parser)
 add_subdirectory(Semantics)

flang/lib/Common/CMakeLists.txt

@@ -35,8 +35,6 @@ if(FLANG_VENDOR)
 endif()
 
 add_flang_library(FortranCommon
-  binary-to-decimal.cpp
-  decimal-to-binary.cpp
   Fortran.cpp
   Fortran-features.cpp
   default-kinds.cpp

flang/lib/Decimal/CMakeLists.txt

@@ -0,0 +1,12 @@
+#===-- lib/Decimal/CMakeLists.txt ------------------------------------------===#
+#
+# Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+# See https://llvm.org/LICENSE.txt for license information.
+# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+#
+#===------------------------------------------------------------------------===#
+
+add_flang_library(FortranDecimal
+  binary-to-decimal.cpp
+  decimal-to-binary.cpp
+)

flang/lib/Common/big-radix-floating-point.h renamed to flang/lib/Decimal/big-radix-floating-point.h

@@ -1,13 +1,13 @@
-//===-- lib/Common/big-radix-floating-point.h -------------------*- C++ -*-===//
+//===-- lib/Decimal/big-radix-floating-point.h ------------------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
-#ifndef FORTRAN_COMMON_BIG_RADIX_FLOATING_POINT_H_
-#define FORTRAN_COMMON_BIG_RADIX_FLOATING_POINT_H_
+#ifndef FORTRAN_DECIMAL_BIG_RADIX_FLOATING_POINT_H_
+#define FORTRAN_DECIMAL_BIG_RADIX_FLOATING_POINT_H_
 
 // This is a helper class for use in floating-point conversions between
 // binary and decimal representations.  It holds a multiple-precision
@@ -21,11 +21,11 @@
 // for conversions between binary and decimal representations; it is not
 // a general-purpose facility.
 
-#include "flang/Common/binary-floating-point.h"
 #include "flang/Common/bit-population-count.h"
-#include "flang/Common/decimal.h"
 #include "flang/Common/leading-zero-bit-count.h"
 #include "flang/Common/uint128.h"
+#include "flang/Decimal/binary-floating-point.h"
+#include "flang/Decimal/decimal.h"
 #include <cinttypes>
 #include <limits>
 #include <type_traits>
@@ -393,4 +393,4 @@ template <int PREC, int LOG10RADIX = 16> class BigRadixFloatingPointNumber {
   enum FortranRounding rounding_ { RoundNearest };
 };
 } // namespace Fortran::decimal
-#endif /* FORTRAN_COMMON_BIG_RADIX_FLOATING_POINT_H_ */
+#endif

flang/lib/Common/binary-to-decimal.cpp renamed to flang/lib/Decimal/binary-to-decimal.cpp

@@ -1,4 +1,4 @@
-//===-- lib/Common/binary-to-decimal.cpp ------------------------*- C++ -*-===//
+//===-- lib/Decimal/binary-to-decimal.cpp ---------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -7,7 +7,7 @@
 //===----------------------------------------------------------------------===//
 
 #include "big-radix-floating-point.h"
-#include "flang/Common/decimal.h"
+#include "flang/Decimal/decimal.h"
 #include <cassert>
 #include <cfloat>
 #include <string>

flang/lib/Common/decimal-to-binary.cpp renamed to flang/lib/Decimal/decimal-to-binary.cpp

@@ -1,4 +1,4 @@
-//===-- lib/Common/decimal-to-binary.cpp ------------------------*- C++ -*-===//
+//===-- lib/Decimal/decimal-to-binary.cpp ---------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -7,10 +7,10 @@
 //===----------------------------------------------------------------------===//
 
 #include "big-radix-floating-point.h"
-#include "flang/Common/binary-floating-point.h"
 #include "flang/Common/bit-population-count.h"
-#include "flang/Common/decimal.h"
 #include "flang/Common/leading-zero-bit-count.h"
+#include "flang/Decimal/binary-floating-point.h"
+#include "flang/Decimal/decimal.h"
 #include "flang/Runtime/freestanding-tools.h"
 #include <cinttypes>
 #include <cstring>

flang/lib/Evaluate/CMakeLists.txt

@@ -61,6 +61,7 @@ add_flang_library(FortranEvaluate
 
   LINK_LIBS
   FortranCommon
+  FortranDecimal
   FortranParser
   ${LIBPGMATH}
   ${QUADMATHLIB}

flang/lib/Evaluate/real.cpp

@@ -8,8 +8,8 @@
 
 #include "flang/Evaluate/real.h"
 #include "int-power.h"
-#include "flang/Common/decimal.h"
 #include "flang/Common/idioms.h"
+#include "flang/Decimal/decimal.h"
 #include "flang/Parser/characters.h"
 #include "llvm/Support/raw_ostream.h"
 #include <limits>

flang/runtime/CMakeLists.txt

@@ -111,8 +111,8 @@ add_definitions(-U_LIBCPP_ENABLE_ASSERTIONS)
 add_subdirectory(Float128Math)
 
 set(sources
-  ${FLANG_SOURCE_DIR}/lib/Common/binary-to-decimal.cpp
-  ${FLANG_SOURCE_DIR}/lib/Common/decimal-to-binary.cpp
+  ${FLANG_SOURCE_DIR}/lib/Decimal/binary-to-decimal.cpp
+  ${FLANG_SOURCE_DIR}/lib/Decimal/decimal-to-binary.cpp
   ISO_Fortran_binding.cpp
   allocator-registry.cpp
   allocatable.cpp

flang/runtime/edit-input.h

@@ -11,7 +11,7 @@
 
 #include "format.h"
 #include "io-stmt.h"
-#include "flang/Common/decimal.h"
+#include "flang/Decimal/decimal.h"
 
 namespace Fortran::runtime::io {
 

flang/runtime/edit-output.h

@@ -20,8 +20,8 @@
 
 #include "format.h"
 #include "io-stmt.h"
-#include "flang/Common/decimal.h"
 #include "flang/Common/uint128.h"
+#include "flang/Decimal/decimal.h"
 
 namespace Fortran::runtime::io {
 

flang/runtime/environment.h

@@ -9,8 +9,8 @@
 #ifndef FORTRAN_RUNTIME_ENVIRONMENT_H_
 #define FORTRAN_RUNTIME_ENVIRONMENT_H_
 
-#include "flang/Common/decimal.h"
 #include "flang/Common/optional.h"
+#include "flang/Decimal/decimal.h"
 
 struct EnvironmentDefaultList;
 

flang/runtime/format-implementation.h

@@ -15,8 +15,8 @@
 #include "format.h"
 #include "io-stmt.h"
 #include "memory.h"
-#include "flang/Common/decimal.h"
 #include "flang/Common/format.h"
+#include "flang/Decimal/decimal.h"
 #include "flang/Runtime/main.h"
 #include <algorithm>
 #include <cstring>

flang/runtime/format.h

@@ -14,8 +14,8 @@
 #include "environment.h"
 #include "io-error.h"
 #include "flang/Common/Fortran-consts.h"
-#include "flang/Common/decimal.h"
 #include "flang/Common/optional.h"
+#include "flang/Decimal/decimal.h"
 #include "flang/Runtime/freestanding-tools.h"
 #include <cinttypes>
 

flang/test/CMakeLists.txt

@@ -72,6 +72,7 @@ set(FLANG_TEST_DEPENDS
   llvm-readobj
   split-file
   FortranRuntime
+  FortranDecimal
 )
 if (LLVM_ENABLE_PLUGINS AND NOT WIN32)
   list(APPEND FLANG_TEST_DEPENDS Bye)

flang/unittests/Decimal/CMakeLists.txt

@@ -1,10 +1,10 @@
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR})
 add_flang_nongtest_unittest(quick-sanity-test
-  FortranCommon
+  FortranDecimal
 )
 
 # This test is not run by default as it takes a long time to execute.
 add_flang_nongtest_unittest(thorough-test
   SLOW_TEST
-  FortranCommon
+  FortranDecimal
 )

flang/unittests/Decimal/quick-sanity-test.cpp

@@ -1,4 +1,4 @@
-#include "flang/Common/decimal.h"
+#include "flang/Decimal/decimal.h"
 #include "llvm/Support/raw_ostream.h"
 #include <cinttypes>
 #include <cstdio>

flang/unittests/Decimal/thorough-test.cpp

@@ -1,4 +1,4 @@
-#include "flang/Common/decimal.h"
+#include "flang/Decimal/decimal.h"
 #include "llvm/Support/raw_ostream.h"
 #include <cinttypes>
 #include <cstdio>

flang/unittests/Evaluate/CMakeLists.txt

@@ -43,6 +43,7 @@ add_flang_nongtest_unittest(intrinsics
   FortranCommon
   FortranEvaluateTesting
   FortranEvaluate
+  FortranDecimal
   FortranSemantics
   FortranParser
   FortranRuntime
@@ -63,7 +64,7 @@ set(LLVM_REQUIRES_RTTI ON)
 add_flang_nongtest_unittest(real
   FortranEvaluateTesting
   FortranEvaluate
-  FortranCommon
+  FortranDecimal
   FortranSemantics
 )
 llvm_update_compile_flags(real.test)