[PATCH] D62798: Explicitly detect recursive response files

[Chris Glover via Phabricator via llvm-commits]

chrisglover created this revision.
chrisglover added reviewers: smeenai, hans, jyknight.
Herald added subscribers: llvm-commits, kristina, hiraditya.
Herald added a project: LLVM.

Previous detection relied upon an arbitrary hard coded limit of 21 response files, which some code bases were running up against.

The new detection maintains a stack of processing response files and explicitly checks if a newly encountered file is in the current stack. Some bookkeeping data is necessary in order to detect when to pop the stack.


Repository:
  rL LLVM

https://reviews.llvm.org/D62798

Files:
  llvm/lib/Support/CommandLine.cpp
  llvm/unittests/Support/CommandLineTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62798.202635.patch
Type: text/x-patch
Size: 4946 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190603/1ecddbec/attachment.bin>