[all-commits] [llvm/llvm-project] 883dbb: Revert "[MemProf] Context disambiguation cloning p...

Nikita Popov via All-commits all-commits at lists.llvm.org
Wed Mar 22 07:44:58 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 883dbb9c86be87593a58ef10b070b3a0564c7fee
      https://github.com/llvm/llvm-project/commit/883dbb9c86be87593a58ef10b070b3a0564c7fee
  Author: Nikita Popov <npopov at redhat.com>
  Date:   2023-03-22 (Wed, 22 Mar 2023)

  Changed paths:
    R llvm/include/llvm/Transforms/IPO/MemProfContextDisambiguation.h
    M llvm/lib/Passes/PassBuilder.cpp
    M llvm/lib/Passes/PassBuilderPipelines.cpp
    M llvm/lib/Passes/PassRegistry.def
    M llvm/lib/Transforms/IPO/CMakeLists.txt
    R llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp
    A llvm/test/ThinLTO/X86/memprof-summary.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/basic.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/duplicate-context-ids.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/duplicate-context-ids2.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/indirectcall.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/inlined.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/inlined2.ll
    R llvm/test/Transforms/MemProfContextDisambiguation/pass-pipeline.ll

  Log Message:
  -----------
  Revert "[MemProf] Context disambiguation cloning pass [patch 1a/3]"

This reverts commit d6ad4f01c3dafcab335bca66dac6e36d9eac8421.

Fails to build on at least gcc 12.2:

/home/npopov/repos/llvm-project/llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp:482:1: error: no declaration matches ‘ContextNode<DerivedCCG, FuncTy, CallTy>* CallsiteContextGraph<DerivedCCG, FuncTy, CallTy>::getNodeForInst(const CallInfo&)’
  482 | CallsiteContextGraph<DerivedCCG, FuncTy, CallTy>::getNodeForInst(
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/home/npopov/repos/llvm-project/llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp:393:16: note: candidate is: ‘CallsiteContextGraph<DerivedCCG, FuncTy, CallTy>::ContextNode* CallsiteContextGraph<DerivedCCG, FuncTy, CallTy>::getNodeForInst(const CallInfo&)’
  393 |   ContextNode *getNodeForInst(const CallInfo &C);
      |                ^~~~~~~~~~~~~~
/home/npopov/repos/llvm-project/llvm/lib/Transforms/IPO/MemProfContextDisambiguation.cpp:99:7: note: ‘class CallsiteContextGraph<DerivedCCG, FuncTy, CallTy>’ defined here
   99 | class CallsiteContextGraph {
      |       ^~~~~~~~~~~~~~~~~~~~




More information about the All-commits mailing list