[PATCH] D71149: [PGO][PGSO] Instrument the code gen / target passes.

Hiroshi Yamauchi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 14:15:59 PST 2019


yamauchi added a comment.

The diff from D71072 <https://reviews.llvm.org/D71072> is MachineOptimizationRemarkEmitter.h and AsmPrinter.cpp around line 1694.

  --- a/llvm/include/llvm/CodeGen/MachineOptimizationRemarkEmitter.h
  +++ b/llvm/include/llvm/CodeGen/MachineOptimizationRemarkEmitter.h
  @@ -182,6 +182,10 @@ public:
       }
     }
   
  +  MachineBlockFrequencyInfo *getBFI() {
  +    return MBFI;
  +  }
  +
   private:
     MachineFunction &MF;
   
  --- a/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
  +++ b/llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
  @@ -1692,7 +1692,10 @@ void AsmPrinter::SetupMachineFunction(MachineFunction &MF) {
     ORE = &getAnalysis<MachineOptimizationRemarkEmitterPass>().getORE();
     PSI = &getAnalysis<ProfileSummaryInfoWrapperPass>().getPSI();
     MBFI = (PSI && PSI->hasProfileSummary()) ?
  -         &getAnalysis<LazyMachineBlockFrequencyInfoPass>().getBFI() :
  +         // ORE conditionally computes MBFI. If available, use it, otherwise
  +         // request it.
  +         (ORE->getBFI() ? ORE->getBFI() :
  +          &getAnalysis<LazyMachineBlockFrequencyInfoPass>().getBFI()) :
            nullptr;
   }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71149





More information about the llvm-commits mailing list