diff options
author | Daniel Dunbar <daniel@zuster.org> | 2009-03-15 01:38:15 +0000 |
---|---|---|
committer | Daniel Dunbar <daniel@zuster.org> | 2009-03-15 01:38:15 +0000 |
commit | 8d2554a2c3201aa664cbf2108cf9d57aa0aa4b0a (patch) | |
tree | af1fc2ae701d972bc8034c22cf28e0ac4d73236a /lib/Driver/Driver.cpp | |
parent | 8022fd46d45005de63306a7513aece20a1be16ed (diff) |
Driver: Start warning about unused arguments.
- This has a number of current flaws, enabling now to flush out
problems while bringing up other parts.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@67015 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Driver/Driver.cpp')
-rw-r--r-- | lib/Driver/Driver.cpp | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/lib/Driver/Driver.cpp b/lib/Driver/Driver.cpp index 2aa50180bd..d3993589c3 100644 --- a/lib/Driver/Driver.cpp +++ b/lib/Driver/Driver.cpp @@ -171,7 +171,21 @@ Compilation *Driver::BuildCompilation(int argc, const char **argv) { return 0; } - return BuildJobs(*Args, Actions); + Compilation *C = BuildJobs(*Args, Actions); + + // If there were no errors, warn about any unused arguments. + for (ArgList::iterator it = Args->begin(), ie = Args->end(); it != ie; ++it) { + Arg *A = *it; + + // FIXME: It would be nice to be able to send the argument to the + // Diagnostic, so that extra values, position, and so on could be + // printed. + if (!A->isClaimed()) + Diag(clang::diag::warn_drv_unused_argument) + << A->getOption().getName(); + } + + return C; } void Driver::PrintOptions(const ArgList &Args) const { @@ -563,7 +577,6 @@ Action *Driver::ConstructPhaseAction(const ArgList &Args, phases::ID Phase, Compilation *Driver::BuildJobs(const ArgList &Args, const ActionList &Actions) const { - assert(0 && "FIXME: Implement"); return 0; } |