[PATCH] D107637: [clangd] Canonicalize inputs provided with `--`

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 6 05:45:55 PDT 2021


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



More information about the cfe-commits mailing list