[all-commits] [llvm/llvm-project] dc9047: [MemProf] Ensure node merging happens for newly cr...

[Teresa Johnson via All-commits]

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

  Changed paths:
    M llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp
    A llvm/test/Transforms/MemProfContextDisambiguation/iterative_merge.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/mergenodes.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/mergenodes2.ll

  Log Message:
  -----------
  [MemProf] Ensure node merging happens for newly created nodes (#151593)

We weren't performing node merging on newly created nodes in some cases.
Use a simple iteration over the node and its callers until no more
opportunities are found. I confirmed that for several large codes the
max iterations is 3 (meaning we only needed to do any work on the first
2, as expected). This can potentially be made more elegant in the
future, but it is a simple and effective solution.

Also fix a bug exposed by the test case, getting the function for a call
instruction in the FullLTO handling, using an existing method to look
through aliases if needed.



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