aganea marked an inline comment as done. aganea added inline comments.
================ Comment at: clang/lib/Driver/Job.cpp:390-392 Argv.push_back(nullptr); + Argv.pop_back(); // The terminating null element shall not be part of the + // slice (main() behavior). ---------------- aheejin wrote: > If we push `nullptr` and then pop it right after that, why do we push it in > the first place? Simply to guarantee that we will have a `nullptr` element after the end of the container. `.pop_back()` only decrements the size, the `nullptr` value will remain there. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116011/new/ https://reviews.llvm.org/D116011 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
