<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>