<div dir="ltr">On 17 May 2013 18:49, Matt Beaumont-Gay <span dir="ltr"><<a href="mailto:matthewbg@google.com" target="_blank">matthewbg@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<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 class="im">> +  TexVTXClauseSize = (Device->getGeneration() >= AMDGPUDeviceInfo::HD4XXX)?16:8;<br>

<br>
</div>Style nit: Spaces around '?' and ':', please.<br>
<div class=""><div class="h5"></div></div></blockquote></div><br></div><div class="gmail_extra" style>I also get a warning on that line:</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>lib/Target/R600/AMDGPUSubtarget.cpp:40:68: warning: comparison of unsigned expression >= 0 is always true [-Wtype-limits]<br>
</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>cheers,</div><div class="gmail_extra" style>--renato</div></div>