<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 31, 2014 at 1:51 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">> By "use case" I meant "feeding a directory of files to llvm-link", not the<br>
> overall script. Feeding a directory of files to llvm-link sounds like a<br>
> perfectly normal thing to do.<br>
<br>
</div>For what?<br></blockquote><div><br></div><div>For feeding a large number of bitcode files to llvm-link (especially without worrying about command-line length and/or response file escaping).</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class=""><br>
> Besides, making the tool easier to use is not a concession that it should be<br>
> used for production purposes or is supported for production purposes; it's<br>
> just that: making the tool easier to use. Think about it this way: if<br>
> llvm-link were originally written with this behavior, would you be dying to<br>
> rip that functionality out?<br>
<br>
</div>Would not be the top of my priorities. But it is not in trunk, so the<br>
burden is on the other side to show that this is a useful thing to<br>
have in a development environment.<br></blockquote><div><br></div><div>Agreed. I think the experience last Summer fulfills that burden of proof and demonstrates that this feature would be handy to have.</div><div><br></div>
<div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class=""><br>
> Khilan, sorry for all this crossfire on your first patch. I'd like to see<br>
> another iteration of this patch with the behavior I described:<br>
> "Even better: there is no -dir argument. Positional arguments can be either<br>
> bitcode files, or directories. If a positional argument happens to be a<br>
> directory, it is scanned for bitcode files, which are added to the list of<br>
> inputs. It is not an error to specify a directory that contains no bitcode<br>
> files."<br>
> Actually, I'd prefer an incremental step of just detecting directory<br>
> arguments and adding all files in the directory (a future patch can filter<br>
> bitcode files if that feature seems useful).<br>
><br>
<br>
</div>Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>