diff options
author | Dan Gohman <gohman@apple.com> | 2009-08-25 15:34:52 +0000 |
---|---|---|
committer | Dan Gohman <gohman@apple.com> | 2009-08-25 15:34:52 +0000 |
commit | baa26395ccf17fc988bb9cf62d6659ca8415ece9 (patch) | |
tree | af3d61a2611ae0c26bb7d9b8ac261976592ec108 /tools/llvm-link | |
parent | cf48efcf3dc6c825a113448105763fab9de9aa53 (diff) | |
download | llvm-baa26395ccf17fc988bb9cf62d6659ca8415ece9.tar.gz llvm-baa26395ccf17fc988bb9cf62d6659ca8415ece9.tar.bz2 llvm-baa26395ccf17fc988bb9cf62d6659ca8415ece9.tar.xz |
Make LLVM command-line tools overwrite their output files without -f.
This is conventional command-line tool behavior. -f now just means
"enable binary output on terminals".
Add a -f option to llvm-extract and llvm-link, for consistency.
Remove F_Force from raw_fd_ostream and enable overwriting and
truncating by default. Introduce an F_Excl flag to permit users to
enable a failure when the file already exists. This flag is
currently unused.
Update Makefiles and documentation accordingly.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@79990 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'tools/llvm-link')
-rw-r--r-- | tools/llvm-link/llvm-link.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tools/llvm-link/llvm-link.cpp b/tools/llvm-link/llvm-link.cpp index 6dc04e6722..14327af70c 100644 --- a/tools/llvm-link/llvm-link.cpp +++ b/tools/llvm-link/llvm-link.cpp @@ -22,6 +22,7 @@ #include "llvm/Support/MemoryBuffer.h" #include "llvm/Support/PrettyStackTrace.h" #include "llvm/Support/raw_ostream.h" +#include "llvm/Support/SystemUtils.h" #include "llvm/System/Signals.h" #include "llvm/System/Path.h" #include <memory> @@ -35,7 +36,8 @@ static cl::opt<std::string> OutputFilename("o", cl::desc("Override output filename"), cl::init("-"), cl::value_desc("filename")); -static cl::opt<bool> Force("f", cl::desc("Overwrite output files")); +static cl::opt<bool> +Force("f", cl::desc("Enable binary output on terminals")); static cl::opt<bool> Verbose("v", cl::desc("Print information about actions taken")); @@ -122,12 +124,9 @@ int main(int argc, char **argv) { std::string ErrorInfo; std::auto_ptr<raw_ostream> Out(new raw_fd_ostream(OutputFilename.c_str(), ErrorInfo, - raw_fd_ostream::F_Binary | - (Force ? raw_fd_ostream::F_Force : 0))); + raw_fd_ostream::F_Binary)); if (!ErrorInfo.empty()) { errs() << ErrorInfo << '\n'; - if (!Force) - errs() << "Use -f command line argument to force output\n"; return 1; } @@ -142,7 +141,8 @@ int main(int argc, char **argv) { } if (Verbose) errs() << "Writing bitcode...\n"; - WriteBitcodeToFile(Composite.get(), *Out); + if (Force || !CheckBitcodeOutputToConsole(*Out, true)) + WriteBitcodeToFile(Composite.get(), *Out); return 0; } |