[all-commits] [llvm/llvm-project] 2a4748: [ADT] Implement Optional::transform

kazutakahirata via All-commits all-commits at lists.llvm.org
Sat Aug 13 11:48:43 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2a4748576e4f734a15c9b0553eec602dd6f03633
      https://github.com/llvm/llvm-project/commit/2a4748576e4f734a15c9b0553eec602dd6f03633
  Author: Kazu Hirata <kazu at google.com>
  Date:   2022-08-13 (Sat, 13 Aug 2022)

  Changed paths:
    M llvm/include/llvm/ADT/Optional.h
    M llvm/unittests/ADT/OptionalTest.cpp

  Log Message:
  -----------
  [ADT] Implement Optional::transform

This patch implements Optional::transform for consistency with
std::optional::transform in C++23.

Note that the new function is identical to Optional::map.  My plan is
to deprecate Optional::map after migrating all of its uses to
Optional::transform.

Differential Revision: https://reviews.llvm.org/D131829


  Commit: 2117fcb1c06207b5eea09b66fb8e0ff58314afa8
      https://github.com/llvm/llvm-project/commit/2117fcb1c06207b5eea09b66fb8e0ff58314afa8
  Author: Kazu Hirata <kazu at google.com>
  Date:   2022-08-13 (Sat, 13 Aug 2022)

  Changed paths:
    M clang-tools-extra/clang-tidy/bugprone/ReservedIdentifierCheck.cpp
    M clang/include/clang/AST/PropertiesBase.td
    M clang/include/clang/Analysis/FlowSensitive/DataflowAnalysis.h
    M llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp

  Log Message:
  -----------
  Use Optional::transform instead of Optional::map (NFC)

I'm planning to deprecate map in favor of transform for consistency
with std::optional::transform in C++23.


Compare: https://github.com/llvm/llvm-project/compare/83fa97567949...2117fcb1c062


More information about the All-commits mailing list