diff options
author | Zhang Qing Shan <zhangqingshan.zll@bytedance.com> | 2022-09-13 12:56:23 +0800 |
---|---|---|
committer | Zhang Qing Shan <zhangqingshan.zll@bytedance.com> | 2022-09-13 13:00:15 +0800 |
commit | 35028d417bb32bc05294e77c9ddcf50f24f277eb (patch) | |
tree | c3d6db02e3c4e72e5393bea1e0c311d678cf4f34 /llvm/tools/llvm-dwp/llvm-dwp.cpp | |
parent | 12e78d96f2fb46c7f4e2fc69db35f0a74502dd60 (diff) | |
download | llvm-35028d417bb32bc05294e77c9ddcf50f24f277eb.zip llvm-35028d417bb32bc05294e77c9ddcf50f24f277eb.tar.gz llvm-35028d417bb32bc05294e77c9ddcf50f24f277eb.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 d2d162d..7d1dd61 100644 --- a/llvm/tools/llvm-dwp/llvm-dwp.cpp +++ b/llvm/tools/llvm-dwp/llvm-dwp.cpp @@ -108,7 +108,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(), @@ -124,7 +130,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; } |