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 | adc91e6597562c9472e25a81636725e95ef94f86 (patch) | |
tree | 8478b032ac15df61510b7f4510580a17e42fe3df /clang/lib/Driver/Driver.cpp | |
parent | ae0e55eb7f21e654d6f4af9810354432b0c978da (diff) | |
download | bcm5719-llvm-adc91e6597562c9472e25a81636725e95ef94f86.tar.gz bcm5719-llvm-adc91e6597562c9472e25a81636725e95ef94f86.zip |
Driver: Start warning about unused arguments.
- This has a number of current flaws, enabling now to flush out
problems while bringing up other parts.
llvm-svn: 67015
Diffstat (limited to 'clang/lib/Driver/Driver.cpp')
-rw-r--r-- | clang/lib/Driver/Driver.cpp | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp index 2aa50180bd2..d3993589c3c 100644 --- a/clang/lib/Driver/Driver.cpp +++ b/clang/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; } |