<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 25, 2016 at 5:29 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas added inline comments.<br>
<span class=""><br>
================<br>
Comment at: ELF/Driver.cpp:259<br>
@@ +258,3 @@<br>
+  for (unsigned I = 1, E = Args.size(); I < E; ++I)<br>
+    OS << " " << Args[I];<br>
+  OS << "\n";<br>
----------------<br>
</span><span class="">ruiu wrote:<br>
> silvas wrote:<br>
> > An argument might contain a space, so let's separate these by newlines instead.<br>
> > (an argument can technically contain a newline but I think that's sufficiently rare that we don't need to worry about that in a first patch.)<br>
> Pathname with spaces are also rare (except on Windows), no? An upside of writing all options in one line is you can (re-)execute the command just by passing the file to the shell.<br>
> Pathname with spaces are also rare (except on Windows), no?<br>
<br>
</span>Exactly. Windows is one of our supported platforms.<br></blockquote><div><br></div><div>Yeah, right. I would run lld only on Unix but I shouldn't have speculated other people's use cases only from my use case.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> An upside of writing all options in one line is you can (re-)execute the command just by passing the file to the shell.<br>
<br>
</span>For now, let's go ahead with how Davide had it (one line). We can make this more robust in future patches.</blockquote><div><br></div><div>Feel free to create a patch to separate it with newlines.</div></div><br></div></div>