[llvm-commits] [PATCH 4/5] Add ELF ObjectWriter and Streamer support.
Eli Friedman
eli.friedman at gmail.com
Sun Aug 1 16:43:26 PDT 2010
On Sun, Aug 1, 2010 at 3:56 PM, Matt Fleming <matt at console-pimps.org> wrote:
>> > +
>> > + virtual void EmitELFSize(MCSymbol *Symbol, const MCExpr *Value) {
>>
>> Push definition outside of class defn.
>
> I fixed a bug in this method and now it's only 2 lines long. Does the
> definition still need to be pushed outside of the class definition?
No, that should be fine.
-Eli
More information about the llvm-commits
mailing list