[all-commits] [llvm/llvm-project] 7846d5: Extend the C++03 definition of POD to include defa...

[David Blaikie via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7846d590033e8d661198f4c00f56f46a4993c526
      https://github.com/llvm/llvm-project/commit/7846d590033e8d661198f4c00f56f46a4993c526
  Author: David Blaikie <dblaikie at gmail.com>
  Date:   2022-10-26 (Wed, 26 Oct 2022)

  Changed paths:
    M clang/include/clang/Basic/LangOptions.h
    M clang/include/clang/Basic/TargetInfo.h
    M clang/lib/AST/DeclCXX.cpp
    M clang/lib/Basic/TargetInfo.cpp
    M clang/lib/Basic/Targets/OSTargets.h
    M clang/test/AST/conditionally-trivial-smfs.cpp
    M clang/test/SemaCXX/class-layout.cpp

  Log Message:
  -----------
  Extend the C++03 definition of POD to include defaulted functions

The AST/conditionally-trivial-smfs tests look a bit questionable, but
are consistent with GCC's POD-ness, at least as far as packing is
concerned: https://godbolt.org/z/36nqPMbKM
(questionable because it looks like the type would be non-copyable, so
how could it be pod? But the calling convention/pass by value seems to
work correctly (local testing verifies that this behavior is preserved
even with this patch: https://godbolt.org/z/3Pa89zsv6 ))

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