[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 ?


http://reviews.llvm.org/D18047





More information about the llvm-commits mailing list