<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 21, 2015 at 11:43 PM, Erik Faye-Lund <span dir="ltr"><<a href="mailto:kusmabite@gmail.com" target="_blank">kusmabite@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 Tue, Oct 20, 2015 at 12:44 AM, Nanley Chery <<a href="mailto:nanleychery@gmail.com">nanleychery@gmail.com</a>> wrote:<br>
> From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com">nanley.g.chery@intel.com</a>><br>
><br>
> Since the version numbers being compared are integral and we don't ever<br>
> expect gl_context::Version to be equal to 0, subtract 1 from the rhs of<br>
> the equation and perform the optimization of removing the equality check.<br>
<br>
</span>Is this really an improvement? Changing '>=' to '>' and then making<br>
all versions off-by-one seems like trading less readable and debugable<br>
code for no real upside...<br>
</blockquote></div><br></div><div class="gmail_extra">After looking at the assembly generated, I've determined that this is<br>actually not an improvement. I'll drop this patch. I was  using .text size<br>as my metric earlier. I now realize that it isn't completely reliable. <br></div></div>