[PATCH] D18047: [ELF] - Do not call doInitSymbols for all ELFTs

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 10 09:06:12 PST 2016

grimar added a comment.

In http://reviews.llvm.org/D18047#371990, @grimar wrote:

> In http://reviews.llvm.org/D18047#371986, @rafael wrote:
> > LGTM. As a followup patch, can you move doInitSymbols to Driver.cpp and make it static?
> Sure. Should this change also be reviewed or can I just commit it after this one ?

I mean "followup" here is the same commit or separate one ?


More information about the llvm-commits mailing list