<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 26, 2016 at 11:14 AM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Easwaran Raman <<a href="mailto:eraman@google.com">eraman@google.com</a>> writes:<br>
> Justin, independent of the fact that this is really not a blocker for<br>
> the profile summary change (as you point out in another thread), this<br>
> seems like a desirable change to separate coverage from the rest of<br>
> the ProfileData. Does that make sense?<br>
<br>
</span>I'm not really convinced that this is a desirable change, personally. It<br>
seems like useless busywork to separate these two things, and if we are<br>
going to make the claim that Coverage is independent/just a user of<br>
ProfileData, why would it be nested inside the ProfileData subdirectory?<br></blockquote><div><br></div><div>I am fine moving Coverage directory out too. However subdir approach is also used in, for instance, CodeGen/SelectionDAG, so it is not  unprecedented.  We can model this also like BitCode. For instance, under ProfileData dir, create two subdirs: Core and Coverage -- but I find that more work than needed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I won't block you from doing this, as having more smaller libraries<br>
doesn't really hurt anything much and I guess it does solve your<br>
immediate problem, but I do consider this just a hack to work around<br>
whatever problems the current layout is causing you guys.<br></blockquote><div><br></div><div>Making ProfileData more independent can help us longer term -- so let's go ahead and commit this change.  Having said that,  Easwaran will also work independently to improve our internal build structure so that similar issues may not be triggered again elsewhere in the future.</div><div><br></div><div>thanks,</div><div><br></div><div>David</div><div><br></div><div> </div></div><br></div></div>