[all-commits] [llvm/llvm-project] 53d5ff: [clang] Check inline defs when emitting speculativ...

[Fabian Parzefall via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 53d5ffea6be7216589599b6415c021f8bd13cd37
      https://github.com/llvm/llvm-project/commit/53d5ffea6be7216589599b6415c021f8bd13cd37
  Author: Fabian Parzefall <parzefall at meta.com>
  Date:   2024-09-05 (Thu, 05 Sep 2024)

  Changed paths:
    M clang/lib/CodeGen/ItaniumCXXABI.cpp
    M clang/test/CodeGenCXX/vtable-available-externally.cpp

  Log Message:
  -----------
  [clang] Check inline defs when emitting speculative vtable (#100785)

Clang should only emit an available_externally vtable when there are no
unused virtual inline functions. Currently, if such such a function is
declared without inline inside the class, but is defined inline outside
the class, Clang might emit the vtable as available_externally. This
happens because Clang only considers the declarations of vtable entries,
but not the definitions. This patch addresses this by inspecting the
definitions in addition to the declarations.



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