[PATCH] D69452: [ThinLTO/WPD] Fix index-based WPD for available_externally vtables

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson created this revision.
tejohnson added reviewers: pcc, davidxl, evgeny777.
Herald added subscribers: arphaman, dexonsmith, steven_wu, hiraditya, inglorion, mehdi_amini.
Herald added a project: LLVM.

Clang does not add type metadata to available_externally vtables. When
choosing a summary to look at for virtual function definitions, make
sure we skip summaries for any available externally vtables as they will
not describe any virtual function functions, which are only summarized
in the presence of type metadata on the vtable def. Simply look for the
corresponding strong def's summary.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69452

Files:
  llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp
  llvm/test/ThinLTO/X86/Inputs/devirt_available_externally.ll
  llvm/test/ThinLTO/X86/devirt_available_externally.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69452.226504.patch
Type: text/x-patch
Size: 5940 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191025/8cc20b9f/attachment.bin>