[all-commits] [llvm/llvm-project] 99b547: [Modules] Add test for merging of template member ...

[Jonas Hahnfeld via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 99b54743106bae89217c3591ec7461266dbd00a4
      https://github.com/llvm/llvm-project/commit/99b54743106bae89217c3591ec7461266dbd00a4
  Author: Jonas Hahnfeld <jonas.hahnfeld at cern.ch>
  Date:   2023-08-02 (Wed, 02 Aug 2023)

  Changed paths:
    A clang/test/Modules/merge-template-members-parent.cpp

  Log Message:
  -----------
  [Modules] Add test for merging of template member parent

This is a reduced test case originally meant to be addressed by
https://reviews.llvm.org/D137787. It was recently fixed by commit
61c7a9140b ("Commit to a primary definition for a class when we load
its first member."), noting the difficulty to come up with a reduced
test case. This setup with four modules seems to fail consistently
before the fix mentioned above with an assertion in CGExprCXX.cpp,
CodeGenFunction::EmitCXXDestructorCall():
Assertion `ThisTy->getAsCXXRecordDecl() == DtorDecl->getParent() &&
"Pointer/Object mixup"' failed.

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