diff --git a/flang/include/flang/Lower/AbstractConverter.h b/flang/include/flang/Lower/AbstractConverter.h --- a/flang/include/flang/Lower/AbstractConverter.h +++ b/flang/include/flang/Lower/AbstractConverter.h @@ -51,7 +51,7 @@ class AbstractConverter { public: //===--------------------------------------------------------------------===// - // Symbols + // Symbols //===--------------------------------------------------------------------===// /// Get the mlir instance of a symbol. diff --git a/flang/include/flang/Parser/characters.h b/flang/include/flang/Parser/characters.h --- a/flang/include/flang/Parser/characters.h +++ b/flang/include/flang/Parser/characters.h @@ -96,9 +96,9 @@ inline constexpr char DecimalDigitValue(char ch) { return ch - '0'; } inline constexpr char HexadecimalDigitValue(char ch) { - return IsUpperCaseLetter(ch) - ? ch - 'A' + 10 - : IsLowerCaseLetter(ch) ? ch - 'a' + 10 : DecimalDigitValue(ch); + return IsUpperCaseLetter(ch) ? ch - 'A' + 10 + : IsLowerCaseLetter(ch) ? ch - 'a' + 10 + : DecimalDigitValue(ch); } inline constexpr std::optional BackslashEscapeValue(char ch) { diff --git a/flang/include/flang/Semantics/scope.h b/flang/include/flang/Semantics/scope.h --- a/flang/include/flang/Semantics/scope.h +++ b/flang/include/flang/Semantics/scope.h @@ -265,8 +265,9 @@ // Inline so that it can be called from Evaluate without a link-time dependency. inline const Symbol *Scope::GetSymbol() const { - return symbol_ ? symbol_ - : derivedTypeSpec_ ? &derivedTypeSpec_->typeSymbol() : nullptr; + return symbol_ ? symbol_ + : derivedTypeSpec_ ? &derivedTypeSpec_->typeSymbol() + : nullptr; } } // namespace Fortran::semantics diff --git a/flang/lib/Evaluate/fold-integer.cpp b/flang/lib/Evaluate/fold-integer.cpp --- a/flang/lib/Evaluate/fold-integer.cpp +++ b/flang/lib/Evaluate/fold-integer.cpp @@ -157,9 +157,8 @@ } else if (name == "ceiling" || name == "floor" || name == "nint") { if (const auto *cx{UnwrapExpr>(args[0])}) { // NINT rounds ties away from zero, not to even - common::RoundingMode mode{name == "ceiling" - ? common::RoundingMode::Up - : name == "floor" ? common::RoundingMode::Down + common::RoundingMode mode{name == "ceiling" ? common::RoundingMode::Up + : name == "floor" ? common::RoundingMode::Down : common::RoundingMode::TiesAwayFromZero}; return std::visit( [&](const auto &kx) { @@ -322,8 +321,8 @@ return name == "index" ? CharacterUtils::INDEX(str, other) : name == "scan" - ? CharacterUtils::SCAN(str, other) - : CharacterUtils::VERIFY(str, other); + ? CharacterUtils::SCAN(str, other) + : CharacterUtils::VERIFY(str, other); }}); } }, diff --git a/flang/lib/Lower/ConvertType.cpp b/flang/lib/Lower/ConvertType.cpp --- a/flang/lib/Lower/ConvertType.cpp +++ b/flang/lib/Lower/ConvertType.cpp @@ -177,18 +177,16 @@ /// mlir::Type. The type returned may be an MLIR standard or FIR type. class TypeBuilder { public: - /// Constructor. explicit TypeBuilder( mlir::MLIRContext *context, const Fortran::common::IntrinsicTypeDefaultKinds &defaults) : context{context}, defaults{defaults} {} - //===--------------------------------------------------------------------===// // Generate type entry points //===--------------------------------------------------------------------===// - + template