[PATCH] [LLD] Add a new PE/COFF port

Rafael EspĂ­ndola rafael.espindola at gmail.com
Thu May 28 11:28:24 PDT 2015


>> bool hasData() const { return HasData; }
>
>
> By doing that we can get rid of "virtual" from the function, but I think it
> slightly decreases readability of the thing. Now you have to define a new
> constructor to initialize HasData right?

Yes, the various derived constructors would set it. It is a somewhat
common pattern in llvm, but usually in cases with far more fields than
this.

Cheers,
Rafael



More information about the llvm-commits mailing list