[PATCH] D29104: Add !associated metadata.

Evgeniy Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 17 17:10:20 PST 2017


eugenis added inline comments.


================
Comment at: include/llvm/MC/MCSectionELF.h:95
+
+  const MCSymbol *getAssociatedSymbol() const { return AssociatedSymbol; }
 
----------------
pcc wrote:
> eugenis wrote:
> > pcc wrote:
> > > I don't quite follow why you need to change this from storing a section to a symbol.
> > Because we need the symbol name when printing textual assembly.
> > 
> I see. Could you separate the printer fix out into a separate change please?
See D30129


Repository:
  rL LLVM

https://reviews.llvm.org/D29104





More information about the llvm-commits mailing list