[llvm] [SHT_LLVM_BB_ADDR_MAP][AsmPrinter] Implements PGOAnalysisMap emitting in AsmPrinter with tests. (PR #75202)

Micah Weston via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 12 07:43:45 PST 2023


================
@@ -1377,7 +1399,8 @@ void AsmPrinter::emitBBAddrMapSection(const MachineFunction &MF) {
   uint8_t BBAddrMapVersion = OutStreamer->getContext().getBBAddrMapVersion();
----------------
red1bluelost wrote:

Linking to @MatzeB comment on the draft PR: https://github.com/red1bluelost/llvm-project/pull/1#discussion_r1388492736.

> FWIW as far as I can tell `OutStreamer->getContext().getBBAddrMapVersion()` is never set anywhere in the whole LLVM codebase so is always 2 in practice. @rlavaee would it make sense to remove this field and accessor and just put a constant somewhere instead? (out of scope for this diff of course, should do that separately)

https://github.com/llvm/llvm-project/pull/75202


More information about the llvm-commits mailing list