rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land.
lgtm ================ Comment at: tools/driver/driver.cpp:464-466 @@ +463,5 @@ + PrependedOpts); + // Insert right after the program name to prepend to the argument list. + for (const char *PrependedOpt : PrependedOpts) + argv.insert(argv.begin() + 1, AdjustOpt(PrependedOpt)); + } ---------------- How about adjusting all the options in place and then inserting them all at once to avoid the N^2 complexity? http://reviews.llvm.org/D11896 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits