[all-commits] [llvm/llvm-project] 938cdb: [flang] Migrate away from std::nullopt (NFC) (#145...

[Kazu Hirata via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 938cdb30f16f4fefc4c7177d7a47fb571a297c43
      https://github.com/llvm/llvm-project/commit/938cdb30f16f4fefc4c7177d7a47fb571a297c43
  Author: Kazu Hirata <kazu at google.com>
  Date:   2025-06-26 (Thu, 26 Jun 2025)

  Changed paths:
    M flang/include/flang/Lower/AbstractConverter.h
    M flang/include/flang/Optimizer/Builder/FIRBuilder.h
    M flang/include/flang/Optimizer/Dialect/Support/KindMapping.h
    M flang/lib/Lower/Allocatable.cpp
    M flang/lib/Lower/Bridge.cpp
    M flang/lib/Lower/CallInterface.cpp
    M flang/lib/Lower/ConvertConstant.cpp
    M flang/lib/Lower/ConvertExprToHLFIR.cpp
    M flang/lib/Lower/ConvertVariable.cpp
    M flang/lib/Lower/HostAssociations.cpp
    M flang/lib/Lower/Mangler.cpp
    M flang/lib/Optimizer/Builder/Character.cpp
    M flang/lib/Optimizer/Builder/MutableBox.cpp
    M flang/lib/Optimizer/CodeGen/TypeConverter.cpp

  Log Message:
  -----------
  [flang] Migrate away from std::nullopt (NFC) (#145928)

ArrayRef has a constructor that accepts std::nullopt.  This
constructor dates back to the days when we still had llvm::Optional.

Since the use of std::nullopt outside the context of std::optional is
kind of abuse and not intuitive to new comers, I would like to move
away from the constructor and eventually remove it.

This patch replaces std::nullopt with {}.  There are a couple of
places where std::nullopt is replaced with TypeRange() to accommodate
perfect forwarding.



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