diff options
author | Hans Wennborg <hans@chromium.org> | 2023-03-17 10:59:03 +0100 |
---|---|---|
committer | Hans Wennborg <hans@chromium.org> | 2023-03-17 13:47:02 +0100 |
commit | e495eabd326801c1e00508898dbcb04073a31014 (patch) | |
tree | 47e4081a966c6c85c0ce4c517106319b885ede5d /clang/lib/Frontend/CompilerInstance.cpp | |
parent | 2d91b0dae0a902317d384a44f588107d04c9ad3c (diff) | |
download | llvm-e495eabd326801c1e00508898dbcb04073a31014.zip llvm-e495eabd326801c1e00508898dbcb04073a31014.tar.gz llvm-e495eabd326801c1e00508898dbcb04073a31014.tar.bz2 |
[clang] Include the error message in file reading error diagnostic
in order to provide as much information as possible to the user. The
diagnostic will now look like for example:
error: error reading '/tmp/foo.c': Permission denied
(This addresses a FIXME from 2019, 9ef6c49baf45)
Differential revision: https://reviews.llvm.org/D146280
Diffstat (limited to 'clang/lib/Frontend/CompilerInstance.cpp')
-rw-r--r-- | clang/lib/Frontend/CompilerInstance.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/Frontend/CompilerInstance.cpp b/clang/lib/Frontend/CompilerInstance.cpp index ecf9394..691f779 100644 --- a/clang/lib/Frontend/CompilerInstance.cpp +++ b/clang/lib/Frontend/CompilerInstance.cpp @@ -981,10 +981,9 @@ bool CompilerInstance::InitializeSourceManager(const FrontendInputFile &Input, ? FileMgr.getSTDIN() : FileMgr.getFileRef(InputFile, /*OpenFile=*/true); if (!FileOrErr) { - // FIXME: include the error in the diagnostic even when it's not stdin. auto EC = llvm::errorToErrorCode(FileOrErr.takeError()); if (InputFile != "-") - Diags.Report(diag::err_fe_error_reading) << InputFile; + Diags.Report(diag::err_fe_error_reading) << InputFile << EC.message(); else Diags.Report(diag::err_fe_error_reading_stdin) << EC.message(); return false; |