<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 2, 2014 at 8:45 PM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Fix a bunch of mislayered clang/Lex includes from Sema</blockquote></div><br></div><div class="gmail_extra">The direction of this change makes lots of sense, but this commit log isn't really helpful for reviewers because you're actually making non-trivial changes in order to fix the layering issues. It would be really helpful for me at least to both split each change into its own commit unless they are interdependent in some way, and to describe the nature of the layering fix when it isn't just a "include the right header instead of the wrong header".</div>
</div>