[PATCH] D150326: [WPD] Update llvm.public.type.test after importing functions

Teresa Johnson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 10 16:12:37 PDT 2023


tejohnson created this revision.
tejohnson added a reviewer: aeubanks.
Herald added subscribers: ormris, steven_wu, hiraditya, emaste.
Herald added a project: All.
tejohnson requested review of this revision.
Herald added subscribers: cfe-commits, MaskRay.
Herald added projects: clang, LLVM.

I noticed that we are converting llvm.public.type.test to regular
llvm.type.test too early, and thus not updating those in imported
functions. This would result in losing out on WPD opportunities. Move
the update to after function importing, and improve test to cover this
case.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D150326

Files:
  clang/test/CodeGenCXX/thinlto_public_type_test_distributed.ll
  lld/test/ELF/lto/update_public_type_test.ll
  llvm/lib/LTO/LTOBackend.cpp
  llvm/lib/LTO/ThinLTOCodeGenerator.cpp
  llvm/test/ThinLTO/X86/devirt_vcall_vis_public.ll
  llvm/test/ThinLTO/X86/public-type-test.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D150326.521139.patch
Type: text/x-patch
Size: 10066 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230510/731c6468/attachment.bin>


More information about the cfe-commits mailing list