[all-commits] [llvm/llvm-project] 0cda97: [Flang][NFC] Split common headers to reduce depend...

[Michael Kruse via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0cda970ecc8a885acf7298a61370a1368b0ea39b
      https://github.com/llvm/llvm-project/commit/0cda970ecc8a885acf7298a61370a1368b0ea39b
  Author: Michael Kruse <llvm-project at meinersbur.de>
  Date:   2024-12-05 (Thu, 05 Dec 2024)

  Changed paths:
    A flang/include/flang/Common/Fortran-consts.h
    M flang/include/flang/Common/Fortran.h
    M flang/include/flang/Common/format.h
    A flang/include/flang/Common/target-rounding.h
    M flang/include/flang/Evaluate/common.h
    M flang/include/flang/Evaluate/target.h
    M flang/include/flang/Runtime/cpp-type.h
    M flang/include/flang/Runtime/type-code.h
    M flang/runtime/format.h
    M flang/runtime/non-tbp-dio.h
    M flang/runtime/type-info.h
    M flang/unittests/Evaluate/fp-testing.cpp
    M flang/unittests/Evaluate/fp-testing.h
    M flang/unittests/Runtime/Complex.cpp

  Log Message:
  -----------
  [Flang][NFC] Split common headers to reduce dependencies. (#110244)

Fortran.h and target.h are defining symbols where some are used by both, the Fortran runtime (Flang-RT) and Fortran compiler (Flang), and others are used by Flang only. With the upcoming refactoring of the Fortran runtime into its own subproject (#110217), move the declarations that are used by both into new headers to minimize the amount of code that will need to be shared by Flang-RT and Flang.

Details:

 * `Fortran.h`: Flang-RT  only uses some enum definitions out of this file, but not `AsFortran` which is defined in `Fortran.cpp`. Moving the enums into `Fortran-consts.h` allows keeping `Fortran.cpp` within Flang.

 * `target.h`: Contains some floating-point definitions that is used by the non-GTest unittests in `fp-testing.h`. Flang-RT also uses some non-GTest as well. Moving those definitions avoids the dependence on the entire FortranEvaluate library.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications