[all-commits] [llvm/llvm-project] c8c66e: [C++20] [Modules] Don't insert class not in named ...

[Chuanqi Xu via All-commits]

  Branch: refs/heads/release/19.x
  Home:   https://github.com/llvm/llvm-project
  Commit: c8c66e01d83323a3db57fade24befb26b0e6fe84
      https://github.com/llvm/llvm-project/commit/c8c66e01d83323a3db57fade24befb26b0e6fe84
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2024-09-01 (Sun, 01 Sep 2024)

  Changed paths:
    M clang/include/clang/Serialization/ASTWriter.h
    M clang/lib/Serialization/ASTWriter.cpp
    A clang/test/Modules/pr106483.cppm

  Log Message:
  -----------
  [C++20] [Modules] Don't insert class not in named modules to PendingEmittingVTables (#106501)

Close https://github.com/llvm/llvm-project/issues/102933

The root cause of the issue is an oversight in
https://github.com/llvm/llvm-project/pull/102287 that I didn't notice
that PendingEmittingVTables should only accept classes in named modules.

(cherry picked from commit 47615ff2347a8be429404285de3b1c03b411e7af)



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