<div dir="ltr">No, I'm not proposing to kill the layout references because components other than LayoutPass are using them.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 22, 2015 at 7:12 AM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>Does this mean you are going to kill
      the Layout specific references ? <br>
      <br>
      The kindLayoutAfter, kindInGroup references are used to layout
      atoms by both Mach-O (atom order file) and ELF.<br>
      <br>
      Shankar Easwaran<div><div class="h5"><br>
      <br>
      On 1/21/2015 10:07 PM, Rui Ueyama wrote:<br>
    </div></div></div>
    <blockquote type="cite"><div><div class="h5">
      <pre>In r226336 I shove off 1.2 seconds out of 9.8 seconds for lld to link lld.
That's done by parallelizing archive member parsing. But I realized that
was not the slowest pass.

The single slowest pass in LLD is LayoutPass. Only sort() at the last of
Layoutpass::perform takes about 3 seconds (one third of total execution
time). It is because the comparison function passed to sort, compareAtoms,
does too much stuff.

It looks to me that the entire pass is overkill. We don't really need that
complexity there. I think nobody is actually depends on the details of the
pass's behavior.

I'd like to simplify LayoutPass, so that the pass sorts atoms only by file
position and position in file (that means atoms are sorted in the same
order as the command line.) I believe that should be enough and we can
still keep output the same.

Any comments?

</pre>
      <br>
      <fieldset></fieldset>
      <br>
      </div></div><pre>_______________________________________________
LLVM Developers mailing list
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><span class="HOEnZb"><font color="#888888">
</font></span></pre><span class="HOEnZb"><font color="#888888">
    </font></span></blockquote><span class="HOEnZb"><font color="#888888">
    <br>
    <br>
    <pre cols="72">-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation</pre>
  </font></span></div>

</blockquote></div><br></div>