[PATCH] D149215: [MemProf] Control availability of hot/cold operator new from LTO link

Tue Apr 25 16:32:57 PDT 2023

tejohnson added a comment.

This was refactored out of patch 3 (the applyImport stuff that I refactored into patch 4 in D149117 <https://reviews.llvm.org/D149117>). Previously I was controlling this behavior via the MemProfContextDisambiguation pass itself, but in refactoring I realized that it is much cleaner and more sensible to be controlling it separately via LTO. I also realized it wasn't being tested, so added some new tests here.

================
Comment at: llvm/test/ThinLTO/X86/memprof-basic.ll:80

-;; Try again but with distributed ThinLTO
-; RUN: llvm-lto2 run %t.o -enable-memprof-context-disambiguation \
----------------
Ignore this and similar differences, need to rebase this on top of my more recent version of D149117 that removed these duplicate invocations of test RUN lines.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149215/new/

https://reviews.llvm.org/D149215