<div dir="ltr">This is one of series of patches, and I'll do this for other drivers as well.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 9, 2015 at 3:53 PM, Shankar Easwaran <span dir="ltr"><<a href="mailto:shankare@codeaurora.org" target="_blank">shankare@codeaurora.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>I already LGTM the patch for PECOFF.<div><div class="h5"><br>
      <br>
      On 2/9/2015 5:41 PM, Rui Ueyama wrote:<br>
    </div></div></div>
    <blockquote type="cite"><div><div class="h5">
      <pre>That's what I wanted to ask you. We didn't have this gigantic switch and
lots of break's there but a series of small ifs. You did converted them as
a part of unrelated change (which is the large InputGraph patch) without
any explanation. Yes, the original code looks better to me, and I guess
that's at least part of the reason why the original code was written as it
was. I'm basically trying to restore the original code.

On Fri, Feb 6, 2015 at 7:40 PM, Shankar Kalpathi Easwaran <
<a href="mailto:shankarke@gmail.com" target="_blank">shankarke@gmail.com</a>> wrote:

</pre>
      <blockquote type="cite">
        <pre>I really dont understand, Its a matter of style, every developer may
choose to follow different styles. Nothing is written in the coding
convention document about what style to choose over another.


<a href="http://reviews.llvm.org/D7478" target="_blank">http://reviews.llvm.org/D7478</a>

EMAIL PREFERENCES
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a>



</pre>
      </blockquote>
      <pre></pre>
      <br>
      <fieldset></fieldset>
      <br>
      </div></div><pre>_______________________________________________
llvm-commits mailing list
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><span class="HOEnZb"><font color="#888888">
</font></span></pre><span class="HOEnZb"><font color="#888888">
    </font></span></blockquote><span class="HOEnZb"><font color="#888888">
    <br>
    <br>
    <pre cols="72">-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation</pre>
  </font></span></div>

</blockquote></div><br></div>