<div dir="ltr">On Wed, Mar 20, 2013 at 10:53 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Mar 20, 2013 at 8:39 AM, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:<br>

> On Tue, Mar 19, 2013 at 5:59 PM, Nadav Rotem <<a href="mailto:nrotem@apple.com">nrotem@apple.com</a>> wrote:<br>
>> Hi Eli,<br>
>><br>
>>  The timer code LGTM.   Why do we need an additional command line flag ?<br>
>><br>
>> Nadav<br>
>><br>
><br>
> time-passes is defined in PassManager so we'd have to make IRReader<br>
> aware of that, which is a layering violation. In addition, many LLVM<br>
> tools use IRReader but don't link PassManager in.<br>
<br>
</div></div>Could we put it somewhere more common?<br></blockquote><div><br></div><div style>I'm not sure what you mean. Suggestions welcome.</div><div style><br></div><div style>Eli</div><div style><br></div><div> </div>
</div><br></div></div>