<div dir="ltr"><div dir="ltr">On Fri, May 3, 2019 at 1:30 PM James Y Knight via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">IMO, a limit of at most 20 nested response files would make a lot more sense than 20 total response files. I don't think the total should really have a limit at all.</div><div dir="ltr"><br></div><div dir="ltr">Since we expand the files in place while iterating over the arglist, we'd need to keep a separate array listing the end-offset of each file we're currently nested within, and update the offsets with every expansion of arguments. That's a bit more complex than just an integer count of number of files seen so far, but it should be implementable completely within the ExpandResponseFiles function, and I don't think it'd be _that_ tricky.</div><div dir="ltr"><div><br></div></div></div></blockquote><div><br></div><div>Seems reasonable to me. And would absolutely solve the issue here (I think we only nest 2 or 3 levels deep).</div><div><br></div><div> If this is an agreed upon solution, I'd be happy to attempt to implement it.</div><div><br></div><div>-- chris</div></div></div>