<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 9, 2015 at 4:30 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"><div dir="ltr">This probably makes sense for consistency with clang, but FYI with current libOption each of the "getLastArg" is linear in the command line length, so the previous switch version was actually much more efficient.<div><br></div><div>This might be more problematic for LLD than for clang since linker command lines can sometimes end up being much longer than compiler command lines.</div><div><br></div><div> The right fix of course is for libOption to cache or otherwise avoid O(cmdline length) work per getLastArg.</div></div></blockquote></div><br>Indeed. libOption should provide a fast lookup structure here.</div></div>