[all-commits] [llvm/llvm-project] 9eac38: [MemProf] Remove context id set from nodes and rec...

[Teresa Johnson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9eac38a000d510e9b162d6996816ef298019b553
      https://github.com/llvm/llvm-project/commit/9eac38a000d510e9b162d6996816ef298019b553
  Author: Teresa Johnson <tejohnson at google.com>
  Date:   2024-06-06 (Thu, 06 Jun 2024)

  Changed paths:
    M llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp

  Log Message:
  -----------
  [MemProf] Remove context id set from nodes and recompute on demand (#94415)

The ContextIds set on the ContextNode struct is not technically needed
as we can compute it from either the callee or caller edge context ids.
Remove it and add a helper to recompute from the edges on demand. Also
add helpers to compute the node allocation type and whether the context
ids are empty from the edges without needing to first compute the node's
context id set, to minimize the runtime cost increase.

This yielded a 20% reduction in peak memory for a large thin link, for
about a 2% time increase (which is more than offset by some other recent
time efficiency improvements).



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