sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang-tools-extra/clangd/CompileCommands.cpp:250
+ ArgList.getLastArgNoClaim(driver::options::OPT__DASH_DASH)) {
+ for (auto I = 0U; I <= DashDash->getNumValues(); ++I)
+ IndicesToDrop.push_back(DashDash->getIndex() + I);
----------------
iterate from dashdash->getIndex() + 1 to Cmd.size() instead?
Or just Cmd.resize(dashdash->getIndex() + 2)?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107637/new/
https://reviews.llvm.org/D107637
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
