[all-commits] [llvm/llvm-project] 964888: [llvm][CFI] Ensure COFF comdat renaming applies fo...

[PiJoules via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 964888d01f0b0f81540f8548370f00c315952042
      https://github.com/llvm/llvm-project/commit/964888d01f0b0f81540f8548370f00c315952042
  Author: PiJoules <6019989+PiJoules at users.noreply.github.com>
  Date:   2025-06-16 (Mon, 16 Jun 2025)

  Changed paths:
    M llvm/lib/Transforms/IPO/LowerTypeTests.cpp
    A llvm/test/Transforms/LowerTypeTests/Inputs/import-thinlto-funcs.yaml
    M llvm/test/Transforms/LowerTypeTests/cfi-coff-comdat-rename.ll

  Log Message:
  -----------
  [llvm][CFI] Ensure COFF comdat renaming applies for imported functions (#143421)

I ran into the same issue as
https://github.com/llvm/llvm-project/pull/139962 regarding the comdat
corresponding to a renamed key function but for thinlto. My last patch
had not considered the thinlto case, so this applies the same fix for
imported functions.



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