[PATCH] D117809: [clang] Add an extract-api driver option
Juergen Ributzka via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 21 10:12:07 PST 2022
ributzka accepted this revision.
ributzka added a comment.
This revision is now accepted and ready to land.
LGTM. Just a few minor nits.
================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:7082
CmdArgs.push_back(Args.MakeArgString(OutputFilename));
+ } else if (Output.getType() == types::TY_API_INFO) {
+ SmallString<128> OutputFilename(Output.getFilename());
----------------
Is this change required for extract-api? I don't think the output file needs to be renamed, because there is only one output in this case.
================
Comment at: clang/lib/Frontend/CompilerInvocation.cpp:2431
{frontend::PrintDependencyDirectivesSourceMinimizerOutput,
- OPT_print_dependency_directives_minimized_source},
};
----------------
This looks like an unrelated change.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117809/new/
https://reviews.llvm.org/D117809
More information about the cfe-commits
mailing list