<div dir="ltr"><div class="gmail_extra">On Fri, Dec 7, 2012 at 6:35 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com" target="_blank" class="cremed">chandlerc@google.com</a>></span> wrote:<br>
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
     Some "style" is the combination of different reviewer's tastes. 
    Some reviewers' taste are just opposite to you personal opinions.<br></div></blockquote><div><br></div></div><div>Within the LLVM project, style is part of code review. My comments below about the style issues with your code are code review that needs to be addressed, even if they don't seem like important style issues to you.</div>
</blockquote></div><br></div><div class="gmail_extra" style>Shuxin, I hate to dig this back up, but the style of this code really does need to be addressed. I'm having to make changes to this code as I refactor other parts of LLVM, and it is substantially harder to read, modify, and generally maintain. That is why we have code review even for style and design, and why it is so important that these review comments are addressed.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>Please prioritize resolving these issues with this code. It has been in the tree in a form that is well below the expected code quality of LLVM for a month now. I think that is too long.</div>
</div>