[all-commits] [llvm/llvm-project] 0ab3d6: Reapply "[MemProf] Reduce cloning overhead by shar...

[Teresa Johnson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0ab3d6e14305ce8a97bfe3af7ddc52c416e698a6
      https://github.com/llvm/llvm-project/commit/0ab3d6e14305ce8a97bfe3af7ddc52c416e698a6
  Author: Teresa Johnson <tejohnson at google.com>
  Date:   2024-08-30 (Fri, 30 Aug 2024)

  Changed paths:
    M llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp
    A llvm/test/ThinLTO/X86/memprof-aliased-location1.ll
    A llvm/test/ThinLTO/X86/memprof-aliased-location2.ll
    A llvm/test/ThinLTO/X86/memprof-tailcall-aliased-location1.ll
    A llvm/test/ThinLTO/X86/memprof-tailcall-aliased-location2.ll
    A llvm/test/Transforms/MemProfContextDisambiguation/aliased-location1.ll
    A llvm/test/Transforms/MemProfContextDisambiguation/aliased-location2.ll
    A llvm/test/Transforms/MemProfContextDisambiguation/tailcall-aliased-location1.ll
    A llvm/test/Transforms/MemProfContextDisambiguation/tailcall-aliased-location2.ll

  Log Message:
  -----------
  Reapply "[MemProf] Reduce cloning overhead by sharing nodes when possible" (#102932) with fixes (#106623)

This reverts commit 11aa31f595325d6b2dede3364e4b86d78fffe635, restoring
commit 055e4319112282354327af9908091fdb25149e9b, with added fixes for
linker unsats.

In some cases multiple calls to different targets may end up with the
same debug information, and therefore callsite id. We will end up
sharing the node between these calls. We don't know which one matches
the callees until all nodes are matched with calls, at which point any
non-matching calls should be removed from the node. The fix extends the
handling in handleCallsitesWithMultipleTargets to do this, and adds
tests for various permutations of this situation.



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