<div dir="ltr"><div>I know you wanted to bring LayoutPass back, and I think I described about why that's not a good idea many times. The most recent discussion is this. Could you read this and then elaborate why you think we should use the LayoutPass based on that discussion?</div><div><br></div><a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2015-February/082130.html">http://lists.cs.uiuc.edu/pipermail/llvmdev/2015-February/082130.html</a></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 18, 2015 at 6:06 PM, 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">I feel using kindLayoutBefore references would be a preferred way for this solution.<br>
<br>
We could enable the LayoutPass only if Alias symbols exist. Does COFF need alias symbols for default operation ?<span class="HOEnZb"><font color="#888888"><br>
<br>
Shankar Easwaran</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 2/18/2015 5:50 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Feb 18, 2015 at 3:47 PM, Shankar Easwaran <<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 2/18/2015 5:11 PM, Rui Ueyama wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+  alias->setOrdinal(target-><u></u>ordinal() - 1);<br>
<br>
</blockquote>
Wouldn't this cause a wrap around when the ordinal of the first atom is 0 ?<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
   +// getNextOrdinal returns a monotonically increasaing uint64_t number<br>
+// starting from 1. There's a large gap between two numbers returned<br>
+// from this function, so that you can put other atoms between them.<br>
+uint64_t FileCOFF::getNextOrdinal() {<br>
+  return _ordinal++ << 32;<br>
+}<br>
+<br>
<br>
</blockquote>
Do you need it this to be shifted by 32 ? Wouldnt incrementing by 2 be<br>
enough ?<br>
</blockquote>
<br>
You could define two or more aliases to a symbol so we need more room than<br>
for one alias. That doesn't work now, though.<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Shankar Easwaran<br>
<br>
--<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted<br>
by the Linux Foundation<br>
<br>
<br>
</blockquote></blockquote>
<br>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
</div></div></blockquote></div><br></div>