<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 31, 2014 at 12:23 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"><div class="">On 3/31/2014 2:11 PM, Rui Ueyama wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not sure if you get my suggestion. This suggestion is basically to keep your code as is, except that replacing conditions inside if's with atom->isComdatGroup(). That would look better than writing the same conditions in many places. We already have a precedence of a utility method that is to be used inside a if condition, which is DefinedAtom::<u></u>occupiesDiskSpace().<br>


</blockquote></div>
Ah!!! Again, .gnu.linkonce doesnot call for itself a comdatGroup, developers might get a wrong impression of calling .gnu.linkonce for a COMDAT group.</blockquote><div><br></div><div>Then, how about isComdatGroupSematics or something like that?</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
Thanks<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>
</div></div></blockquote></div><br></div></div>