[PATCH] D136090: Handle errors in expansion of response files
Serge Pavlov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 17 09:06:40 PDT 2022
sepavloff created this revision.
sepavloff added reviewers: kadircet, mgorny, MaskRay, smeenai.
Herald added subscribers: StephenFan, hiraditya.
Herald added a reviewer: awarzynski.
Herald added projects: Flang, All.
sepavloff requested review of this revision.
Herald added subscribers: llvm-commits, jdoerfert.
Herald added projects: clang, LLVM.
Previously an error raised during an expansion of response files (including
configuration files) was ignored and only the fact of its presence was
reported to the user with generic error messages. This made it difficult to
analyze problems. For example, if a configuration file tried to read an
inexistent file, the error message said that 'configuration file cannot
be found', which is wrong and misleading.
This change enhances handling errors in the expansion so that users
could get more informative error messages.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D136090
Files:
clang/include/clang/Basic/DiagnosticDriverKinds.td
clang/lib/Driver/Driver.cpp
clang/lib/Tooling/ExpandResponseFilesCompilationDatabase.cpp
clang/test/Driver/config-file-errs.c
clang/tools/driver/driver.cpp
clang/unittests/Driver/ToolChainTest.cpp
flang/tools/flang-driver/driver.cpp
llvm/include/llvm/Support/CommandLine.h
llvm/lib/Support/CommandLine.cpp
llvm/unittests/Support/CommandLineTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D136090.468229.patch
Type: text/x-patch
Size: 29899 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20221017/02245051/attachment-0001.bin>
More information about the cfe-commits
mailing list