[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