<div dir="ltr"><div class="gmail_extra">On Sat, Feb 28, 2015 at 8:52 AM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankar.kalpathi.easwaran@gmail.com" target="_blank">shankar.kalpathi.easwaran@gmail.com</a>></span> wrote:<br></div><div class="gmail_extra"><div class="gmail_quote"><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">in my opinion this is a shared attribute for all atoms that belong to a section. Can we store this information for the atom in the context? The other way is to add a Reference to store the size. If the idea is to add all format specific attributes in the Atom I am fine but we need to discuss on llvmdev before adding those features.<br></blockquote><div><br></div><div>It's not feasible to make this an attribute of a section because as you know we don't have a class representing a section. We basically copy section properties to atoms in that section. Examples are permission (rwx permissions), section name and merge. They are in all platforms properties of sections, not of symbols, but we store them to atoms. So this is not new at all.</div><div><br></div><div>As I wrote before, using references as a "generic key-value store" is a bad idea. That way is basically equivalent to using a string-key associative list as a replacement for C++ member variables. Sure we would be able to store anything to that without defining new members. But that's apparently what we don't want. It's too generic and unstructured. (If that's a good idea, we should probably remove all member variables from Atom and make the atom just a map. All members currently defined for Atom, such as ordinal or size field, would then be a value in the map. That's obviously wrong design.)</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">
<br>
Anyways this patch is not complete as it's missing YAML Support and native file format support.<br></blockquote><div><br></div><div>Will add.</div><div> </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">
<br>
<br>
<a href="http://reviews.llvm.org/D7966" target="_blank">http://reviews.llvm.org/D7966</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div><br></div></div>