On Wednesday, August 12, 2015, Dan Gohman via llvm-commits <<a href="javascript:_e(%7B%7D,'cvml','llvm-commits@lists.llvm.org');" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Aug 12, 2015 at 8:01 AM, Rafael Espíndola<br>
<<a>rafael.espindola@gmail.com</a>> wrote:<br>
> On 12 August 2015 at 10:55, Dan Gohman <<a>dan433584@gmail.com</a>> wrote:<br>
>> I don't understand the vision for triples well enough to properly<br>
>> review this. Is there an eventual boundary for the information that<br>
>> should be sourced from a triple? I read the recent "trouble with<br>
>> triples" thread but am still not fully clear on where things are<br>
>> headed.<br>
><br>
> I am not sure, but note that this patch doesn't add any new<br>
> information to the triple. I just uses what is already there.<br>
<br>
It does remove information from MCObjectFileInfo though. Is it really<br>
desirable to pick the Triple to be the one that gets to keep this<br>
information, rather than the other way around? I don't have a clear<br>
view of where this is headed.</blockquote><div><br></div><div>I think the decision to keep it in the triple is sound.</div><div><br></div><div>Although from an abstraction POV, the MOFI should know about the format (see my comment about not thinking that the enumeration belongs in the Triple), it is more reasonable to simply store it in the triple.  The triple needs to understand the target object format since it may be part of the triple (e.g. windows-elf).  The triple is almost universally available, making it easier to query at various places.  Finally, although the MOFI had this information, it was derived from the triple - the source of truth.<span></span></div><div><br></div><div>The information in MOFI was duplicating the information in the triple.  Why not pass it along?</div><div><br></div><div>Now, I could be convinced that keeping the interface in MFOI and making it a means of querying the triple is just as valid.  But, that gives you two ways to accomplish the same thing.  I think that this would also solve the issue, but is it really better to have multiple ways to query this?</div><div><br></div><div>From the "trouble with triples" thread, it seemed that the idea was to have some generic collection of key/value pairs (converting the triple into a tuple).  This information could easily map into that view, if this information is kept in the triple.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Dan<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a>llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote>
<br><br>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org<br>