grimar added a comment. 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 ? http://reviews.llvm.org/D18047