[all-commits] [llvm/llvm-project] a89df7: WholeProgramDevirt: Fix importing in llvm.type.che...

[Peter Collingbourne via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a89df72ec0864301f102296dcf7b3bd22844adf5
      https://github.com/llvm/llvm-project/commit/a89df72ec0864301f102296dcf7b3bd22844adf5
  Author: Peter Collingbourne <peter at pcc.me.uk>
  Date:   2025-06-13 (Fri, 13 Jun 2025)

  Changed paths:
    M llvm/lib/Transforms/IPO/WholeProgramDevirt.cpp
    A llvm/test/ThinLTO/X86/Inputs/devirt_single_hybrid_foo_tcl.ll
    M llvm/test/ThinLTO/X86/devirt_single_hybrid.ll

  Log Message:
  -----------
  WholeProgramDevirt: Fix importing in llvm.type.checked.load case.

We were clearing SummaryTypeCheckedLoadUsers to prevent devirtualized
llvm.type.checked.load calls from being converted to llvm.type.test,
which meant that AddCalls would not see them in the list of
callsites and they would not get imported. Fix that by not clearing
SummaryTypeCheckedLoadUsers so that the list survives to AddCalls and
using AllCallSitesDevirted to control whether to convert them instead.

Reviewers: teresajohnson

Reviewed By: teresajohnson

Pull Request: https://github.com/llvm/llvm-project/pull/144019



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