[all-commits] [llvm/llvm-project] f74184: [MemProf] Add NodeId field to ContextNode for debu...

[Teresa Johnson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f74184ccc9053268a5baba129fcfde554cf0d255
      https://github.com/llvm/llvm-project/commit/f74184ccc9053268a5baba129fcfde554cf0d255
  Author: Teresa Johnson <tejohnson at google.com>
  Date:   2025-09-15 (Mon, 15 Sep 2025)

  Changed paths:
    M llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp
    M llvm/test/ThinLTO/X86/memprof-basic.ll
    M llvm/test/ThinLTO/X86/memprof-indirectcall.ll
    M llvm/test/ThinLTO/X86/memprof-inlined.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/basic.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/dot.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/duplicate-context-ids.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/indirectcall.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/inlined.ll
    M llvm/test/Transforms/MemProfContextDisambiguation/tailcall-nonunique.ll

  Log Message:
  -----------
  [MemProf] Add NodeId field to ContextNode for debugging (#158736)

This has been handy locally for debugging cloning issues. The NodeIds
are assigned sequentially on creation and included in the dumps and the
dot graphs.

No measurable memory increase was found for a large thin link.

I only changed one test
(Transforms/MemProfContextDisambiguation/basic.ll)
to actually check the emitted NodeIds, most ignore them.



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