[PATCH] D132379: [Support] Class for response file expansion
Serge Pavlov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 23 09:37:49 PDT 2022
sepavloff updated this revision to Diff 462528.
sepavloff added a comment.
Address reviewer's notes
- Pass BumpPtrAllocator instead of StringSaver,
- Use setter to specify file system object.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132379/new/
https://reviews.llvm.org/D132379
Files:
clang/lib/Driver/Driver.cpp
clang/lib/Tooling/ExpandResponseFilesCompilationDatabase.cpp
clang/tools/driver/driver.cpp
llvm/include/llvm/Support/CommandLine.h
llvm/include/llvm/Support/StringSaver.h
llvm/lib/Support/CommandLine.cpp
llvm/unittests/Support/CommandLineTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132379.462528.patch
Type: text/x-patch
Size: 21016 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220923/f7863755/attachment-0001.bin>
More information about the cfe-commits
mailing list