[PATCH] Debug Info on Windows

David Blaikie dblaikie at gmail.com
Fri Jul 26 12:58:41 PDT 2013


>> b) Could we sink this logic down into DIELabel (or
>> DIEString)::EmitValue rather than having the special case up here in
>> emitDIE?
>
>
> DIELabel then. DIEString calls DIELabel.

Great... and sorry to keep iterating like this, but looking now at the
only current (committed) use of "needsDwarfSectionOffsetDirective"
that's down in AsmPrinter::EmitSectionOffset, so what would happen if
we pushed this one layer further down, into
AsmPrinter::EmitLabelReference?

- Dave



More information about the llvm-commits mailing list