mesa: Drop index buffer bounds check
authorIan Romanick <ian.d.romanick@intel.com>
Tue, 11 Nov 2014 09:21:40 +0000 (09:21 +0000)
committerKenneth Graunke <kenneth@whitecape.org>
Thu, 15 Jan 2015 01:09:50 +0000 (17:09 -0800)
commitd5f936367ffbec5833e5eb7cf22666e38c8e3821
treece9833ac3a792f8838772aeca1cb983633623fe5
parenta4aeb534ea8a21770b28cdf1f66cff4afe123009
mesa: Drop index buffer bounds check

The previous check was insufficient (as it did not take 'indices' into
consideration), and DX10 hardware does not need this check anyway.

Since index_bytes is no longer used, remove it.

On Bay Trail-D using Fedora 20 compile flags (-m64 -O2 -mtune=generic
for 64-bit and -m32 -march=i686 -mtune=atom for 32-bit), affects
Gl32Batch7:

32-bit: Difference at 95.0% confidence 1.66929% +/- 0.230107% (n=40)
64-bit: Difference at 95.0% confidence -1.40848% +/- 0.288038% (n=40)

The regression on 64-bit is odd.  Callgrind says the caller,
validate_DrawElements_common is faster.  Before it says 10,321,920
cycles, and after it says 8,945,664.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/main/api_validate.c