<div dir="ltr"><br><br><div class="gmail_quote">On Mon, Mar 2, 2015 at 10:25 AM Mehdi Amini <<a href="mailto:mehdi.amini@apple.com">mehdi.amini@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On Mar 2, 2015, at 9:40 AM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
><br>
> Few small comments:<br>
><br>
> a) You've got Function::getDataLayout - I think I'd prefer people to generally have to ask the Function parent to get the data layout. (It's also a little inconsistent in the code, but should be fairly sed'able).<br>
<br>
Not sure I understand what you mean. Are you saying that getDataLayout() should be available on the module only?<br>
Note: I haven’t added this, there is a getDataLayout() in Instruction, BasicBlock, and GlobalValue.<br>
It saves some space when the only thing you have is an instruction:<br>
   I->getParent()->getParent()-><u></u>getParent()->getDataLayout();<br>
<br>
<br></blockquote><div><br></div><div>Oh, I don't particularly care (though Instruction and BasicBlock seem weird), just that it wasn't consistent in the patch, but I can't say anything about that either.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> b) Routines to construct a data layout (Totally a follow up patch) in llvm rather than requiring each front end to handle construct them up themselves.<br>
><br>
> c) Relatedly you don't do anything about the code in the backends that constructs the data layouts?<br>
<br>
I haven’t touch the backend yet, I plan on reworking this when I’m done with the IR-level.<br>
<br></blockquote><div><br></div><div>OK.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
—<br>
Mehdi<br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>