[all-commits] [llvm/llvm-project] 36357c: Remove llvm::is_trivially_copyable (NFC)

kazutakahirata via All-commits all-commits at lists.llvm.org
Sun Aug 21 10:39:40 PDT 2022

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 36357c967cdb9006dd7de9be0dd52068da2a4f96
  Author: Kazu Hirata <kazu at google.com>
  Date:   2022-08-21 (Sun, 21 Aug 2022)

  Changed paths:
    M llvm/cmake/config-ix.cmake
    M llvm/include/llvm/ADT/PointerIntPair.h
    M llvm/include/llvm/Config/config.h.cmake
    M llvm/include/llvm/Support/type_traits.h
    M llvm/utils/gn/secondary/llvm/include/llvm/Config/BUILD.gn
    M utils/bazel/llvm-project-overlay/llvm/include/llvm/Config/config.h
    M utils/bazel/llvm_configs/config.h.cmake

  Log Message:
  Remove llvm::is_trivially_copyable (NFC)

This patch removes llvm::is_trivially_copyable as it seems to be dead.
Once I remove it, HAVE_STD_IS_TRIVIALLY_COPYABLE has no users, so this
patch removes the macro also.

The comment on llvm::is_trivially_copyable mentions GCC 4.9, but note
that we now require GCC 7.1 or higher.

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

  Commit: be35870dc8eed02113f730acfcf7da7e50418ae6
  Author: Kazu Hirata <kazu at google.com>
  Date:   2022-08-21 (Sun, 21 Aug 2022)

  Changed paths:
    M llvm/include/llvm/ADT/bit.h

  Log Message:
  [ADT] Simplify llvm::bit_cast (NFC)

This patch removes macro tricks to check GCC versions.

The commit message from 19262fc5966ab569f21f3d440f8b001bca666f17
states that "is_trivially_copyable is only in GCC 5.1 and later".
Note that we now require GCC 7.1 or higher.

Since both std::is_trivially_constructible and
std::is_trivially_copyable are C++11 features, and we now require
C++17, we probably don't need to worry about the availability of the
C++11 features.

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

Compare: https://github.com/llvm/llvm-project/compare/d9ff670330a8...be35870dc8ee

More information about the All-commits mailing list