[PATCH] D153189: [MC] Make .pseudo_probe GC-able

[Hongtao Yu via Phabricator via llvm-commits]

hoy added inline comments.


================
Comment at: llvm/lib/MC/MCObjectFileInfo.cpp:1180
+  return Ctx->getELFSection(PseudoProbeSection->getName(), ELF::SHT_PROGBITS,
+                            Flags, 0, GroupName, true, ElfSec.getUniqueID(),
+                            cast<MCSymbolELF>(TextSec.getBeginSymbol()));
----------------
MaskRay wrote:
> hoy wrote:
> > Should the pseudo probe section be a comdat only if the text section is a comdat? 
> The `const MCSymbol *Group = ElfSec.getGroup()` condition ensures that the `.pseudo_probe` section is in a section group only when the text section is in a section group. When `GroupName` is empty, `getELFSection` does not use a section group.
So IIUC, when `GroupName` is empty, `IsComdat=true` doesn't really make a difference right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153189