<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 31, 2014 at 10:18 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 3/31/2014 11:47 AM, Rui Ueyama wrote:<br>


</div><div class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On Mon, Mar 31, 2014 at 9:27 AM, Shankar Easwaran<br>
<<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>><u></u>wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On 3/30/2014 11:11 PM, Rui Ueyama wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
This is not an obvious change to Core, not suitable for post-commit<br>
review.<br>
Please roll it back until we reach a conclusion that this is the right<br>
design.<br>
<br>
  I am not sure why its not a change, that has to be present in core. See<br>
</blockquote>
my notes below.<br>
</blockquote>
<br>
I don't get it. What do you mean?<br>
<br>
Anyways, 1) this change is to a common component of LLD shared by all<br>
ports, 2) made everybody writing code for COMDAT to check gnu.linkonce too,<br>
and 3) was not discussed if that is the right design. If this repository is<br>
owned by you that's fine but this is a shared repository. Please follow the<br>
rule.<br>
</blockquote></div>
I dont have any issues. with reverting this patch for now, but there is no need to send an email like this.<br>
<br>
I raised my opinion on why I chose this design, and I see contributing inputs to a better design might have been much more useful.<br>
This is what I see mostly in open source development.</blockquote><div><br></div><div>I did not simply push back this patch but also pointed out some issues that were in your patch with a suggestion.</div><div><br></div>

<div>My suggestion was to merge it with COMDAT group section if they share the same semantics. If COMDAT groups and .gnu.linkonce sections need to be distinguished only to detect collisions of different types, i'd be great if we can compartment that detail in a small piece of code in Resolver, rather than spreading <span style="font-family:arial,sans-serif;font-size:13px">typeGroupComdat||typeGnuLinkOnce at every if's. Can't you use name prefix ".gnu.linkonce" to detect one section is a .gnu.linkonce section? If not, can't you add a new attribute for it?</span></div>

<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class=""><font color="#888888"><br>
<br>
Shankar Easwaran<br>
<br>
-- <br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation<br>
<br>
</font></span></blockquote></div><br></div></div>