[all-commits] [llvm/llvm-project] fde55a: [LTO] Fix cloning of llvm*.used when splitting module

Teresa Johnson via All-commits all-commits at lists.llvm.org
Sat Feb 20 09:47:13 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fde55a9c9bf147241249ae478760ae69dd1925ed
      https://github.com/llvm/llvm-project/commit/fde55a9c9bf147241249ae478760ae69dd1925ed
  Author: Teresa Johnson <tejohnson at google.com>
  Date:   2021-02-20 (Sat, 20 Feb 2021)

  Changed paths:
    M llvm/lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp
    M llvm/lib/Transforms/Utils/ModuleUtils.cpp
    A llvm/test/Transforms/ThinLTOBitcodeWriter/split-used.ll

  Log Message:
  -----------
  [LTO] Fix cloning of llvm*.used when splitting module

Refines the fix in 3c4c205060c9398da705eb71b63ddd8a04999de9 to only
put globals whose defs were cloned into the split regular LTO module
on the cloned llvm*.used globals. This avoids an issue where one of the
attached values was a local that was promoted in the original module
after the module was cloned. We only need to have the values defined in
the new module on those globals.

Fixes PR49251.

Differential Revision: https://reviews.llvm.org/D97013




More information about the All-commits mailing list