diff options
| author | Dimitry Andric <dimitry@andric.com> | 2017-06-07 12:05:41 +0000 |
|---|---|---|
| committer | Dimitry Andric <dimitry@andric.com> | 2017-06-07 12:05:41 +0000 |
| commit | 0527c32be86c40c4589e7d5402555074d121ffd6 (patch) | |
| tree | 29d873c7407c20de4409b0d6befdfe330b51f8ea | |
| parent | 0196427b037d4830ff38c4bc3c75429998b6a0a0 (diff) | |
| download | bcm5719-llvm-0527c32be86c40c4589e7d5402555074d121ffd6.tar.gz bcm5719-llvm-0527c32be86c40c4589e7d5402555074d121ffd6.zip | |
Only print registered targets for `--version`
Summary:
In D33900, I added printing of the registered targets in clang's
`PrintVersion` function, which is not only used for `--version` output,
but also for `-v` (verbose mode) and `-###`. Especially the latter
seems to trip up some test cases, so it is probably better to only print
the registered targets for `--version`.
Reviewers: nemanjai, mehdi_amini
Reviewed By: nemanjai
Subscribers: cfe-commits
Differential Revision: https://reviews.llvm.org/D33981
llvm-svn: 304899
| -rw-r--r-- | clang/lib/Driver/Driver.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp index ec106fa93be..63496af3f3d 100644 --- a/clang/lib/Driver/Driver.cpp +++ b/clang/lib/Driver/Driver.cpp @@ -1117,10 +1117,6 @@ void Driver::PrintVersion(const Compilation &C, raw_ostream &OS) const { // Print out the install directory. OS << "InstalledDir: " << InstalledDir << '\n'; - - // Print registered targets. - OS << '\n'; - llvm::TargetRegistry::printRegisteredTargetsForVersion(OS); } /// PrintDiagnosticCategories - Implement the --print-diagnostic-categories @@ -1166,6 +1162,10 @@ bool Driver::HandleImmediateArgs(const Compilation &C) { if (C.getArgs().hasArg(options::OPT__version)) { // Follow gcc behavior and use stdout for --version and stderr for -v. PrintVersion(C, llvm::outs()); + + // Print registered targets. + llvm::outs() << '\n'; + llvm::TargetRegistry::printRegisteredTargetsForVersion(llvm::outs()); return false; } |

