<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 10, 2014 at 3:30 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</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 Mon, Feb 10, 2014 at 3:26 PM, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:<br>


> On Mon, Feb 10, 2014 at 2:00 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Any reason not to go ahead and just put it into lib/IR (alongside the<br>
>> PassManager infrastructure) or some such?<br>
><br>
><br>
><br>
> I guess just the principle of least impact :-) What I mean by that is - this<br>
> seems to be only useful within the context of the opt tool now. My<br>
> medium-range goal is being able to create opt-like tools easily without<br>
> copy-pasting most of opt.cpp; so for this reason, and just because this<br>
> functionality is self contained and thus living in a separate module is good<br>
> engineering practice, moving it into a separate file in tools/opt made<br>
> sense. I agree that if there's a need to use this tool outside tools/opt,<br>
> putting it in lib/IR or some such is a better idea. But we don't know in<br>
> advance, so taking this in smaller steps made sense to me. What do you<br>
> think?<br>
><br>
<br>
</div>Mostly just that the default factoring for things in tools/ doesn't<br>
include "auxiliary files" and that we normally would pull those out<br>
into their own library. We should probably have done that for a few<br>
things already - there's some pretty good boilerplate cut and paste<br>
already :)<br>
<br>
But for now splitting it into a separate module inside tools/opt seems<br>
reasonable. (i.e. LGTM).<br></blockquote><div><br></div><div>Thanks. r201116.</div><div><br></div><div>Eli </div></div></div></div>