mgorny added inline comments.
================ Comment at: llvm/lib/Support/CommandLine.cpp:1215-1222 + } else if (ArgStr == "--config") { + if (I + 1 == E) + return createStringError(std::errc::invalid_argument, + "Option '--config' requires an argument"); + I = NewArgv.erase(I); + E = NewArgv.end(); + FileName = *I; ---------------- Not sure if we want to add extra complexity for the legacy spelling here. @MaskRay, what do you think? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136354/new/ https://reviews.llvm.org/D136354 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits