<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 14, 2015 at 9:46 AM, Ahmed Bougacha <span dir="ltr"><<a href="mailto:ahmed.bougacha@gmail.com" target="_blank">ahmed.bougacha@gmail.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="">On Wed, Jan 14, 2015 at 12:03 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:<br>
> While this is target information, it isn't actually part of the target today<br>
> and doesn't use any part of the target code. Maybe some day we can use a<br>
> target hook to initialize the data structures, but even then I feel like<br>
> this really belongs in the core analysis library.<br>
<br>
</span>So I'm curious: do you have an idea of where better to put this<br>
hypothetical hook? Since it's mostly OS-dependent, it doesn't seem<br>
very fit for the way we do targets.<br></blockquote><div><br></div><div>No idea at all, and I completely agree with you.</div><div><br></div><div>Ultimately, I think the word "target" here is misleading. It means target in the since of a cross compilation target, but it has little to no relevance to LLVM targets. As you mention, it is an OS and runtime library availability issue.</div><div><br></div><div>Anyways, we don't need a hook today, and if we do at some point in the future we can figure out how to add one.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> Would there be any objections to just moving it into Analysis?<br>
><br>
> Doing this would remove a seriously ugly layering violation we have today. I<br>
> believe this would remove the last dependency on lib/Target by lib/Analysis<br>
> and the largest set of dependencies on lib/Target by lib/Transforms.<br>
><br>
> If there is general agreement, I will make it so.<br>
<br>
</span>FWIW I agree, on all counts.</blockquote></div><br>Awesome. Will make it so. (Especially now that the release is branched.)</div></div>