kadircet added inline comments.
================
Comment at: llvm/lib/Support/CommandLine.cpp:1183
ExpandResponseFile(FName, Saver, Tokenizer, ExpandedArgv, MarkEOLs,
- RelativeNames, FS)) {
+ RelativeNames, FS, CurrentDir)) {
// We couldn't read this file, so we leave it in the argument stream and
----------------
sammccall wrote:
> this is subtle, consider a comment: CurrentDir is only relevant for
> "top-level" expansions || !RelativeNames, but nested ones always have
> absolute paths if RelativeNames so CurrentDir is ignored.
did better, changed ExpandResponseFile to take FName as an absolute path, PTAL
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70857/new/
https://reviews.llvm.org/D70857
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
