diff options
author | Zhang Qing Shan <zhangqingshan.zll@bytedance.com> | 2022-09-14 11:32:41 +0800 |
---|---|---|
committer | Zhang Qing Shan <zhangqingshan.zll@bytedance.com> | 2022-09-14 11:32:41 +0800 |
commit | 734843ebc7c348a154182da00d4f0e215932d64e (patch) | |
tree | dd2461981db2154694578ea0fd5ab0eb55bb8565 /llvm/tools/llvm-dwp/llvm-dwp.cpp | |
parent | ecf327f1548e5933ff032bde488e45210cba9863 (diff) | |
download | llvm-734843ebc7c348a154182da00d4f0e215932d64e.zip llvm-734843ebc7c348a154182da00d4f0e215932d64e.tar.gz llvm-734843ebc7c348a154182da00d4f0e215932d64e.tar.bz2 |
[llvm-dwp] Report the filename if it cannot be found
For now, we report nothing if the execution/dwo file is missing, which is confusing.
Reviewed By: dblaikie
Differential Revision: https://reviews.llvm.org/D133549
Diffstat (limited to 'llvm/tools/llvm-dwp/llvm-dwp.cpp')
-rw-r--r-- | llvm/tools/llvm-dwp/llvm-dwp.cpp | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/llvm/tools/llvm-dwp/llvm-dwp.cpp b/llvm/tools/llvm-dwp/llvm-dwp.cpp index bc170a1..f9d1dc4 100644 --- a/llvm/tools/llvm-dwp/llvm-dwp.cpp +++ b/llvm/tools/llvm-dwp/llvm-dwp.cpp @@ -111,7 +111,13 @@ int main(int argc, char **argv) { for (const auto &ExecFilename : ExecFilenames) { auto DWOs = getDWOFilenames(ExecFilename); if (!DWOs) { - logAllUnhandledErrors(DWOs.takeError(), WithColor::error()); + logAllUnhandledErrors( + handleErrors(DWOs.takeError(), + [&](std::unique_ptr<ECError> EC) -> Error { + return createFileError(ExecFilename, + Error(std::move(EC))); + }), + WithColor::error()); return 1; } DWOFilenames.insert(DWOFilenames.end(), @@ -127,7 +133,13 @@ int main(int argc, char **argv) { auto ErrOrTriple = readTargetTriple(DWOFilenames.front()); if (!ErrOrTriple) { - logAllUnhandledErrors(ErrOrTriple.takeError(), WithColor::error()); + logAllUnhandledErrors( + handleErrors(ErrOrTriple.takeError(), + [&](std::unique_ptr<ECError> EC) -> Error { + return createFileError(DWOFilenames.front(), + Error(std::move(EC))); + }), + WithColor::error()); return 1; } |