diff options
| author | Vinson Lee <vlee@freedesktop.org> | 2014-02-19 22:49:40 -0800 |
|---|---|---|
| committer | Vinson Lee <vlee@freedesktop.org> | 2014-02-19 22:51:51 -0800 |
| commit | cae3de343eaccc840373f75c2b236f4543b28d8c (patch) | |
| tree | c667f54dde5abd22c9497252ed977a9cefd5f6aa | |
| parent | 3107ff1d25545e4fe6cf8f86ce4e3e1f36dcc962 (diff) | |
ext_transform_feedback-alignment: Fix undefined multiplication.
This patch fixes this GCC aggressive-loop-optimizations warning.
alignment.c: In function 'piglit_display':
alignment.c:184:28: warning: iteration 32u invokes undefined behavior [-Waggressive-loop-optimizations]
0x00010203 + 0x04040404 * i;
^
alignment.c:182:2: note: containing loop
for (i = 0; i < EXPECTED_NUM_OUTPUTS; ++i) {
^
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
| -rw-r--r-- | tests/spec/ext_transform_feedback/alignment.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/spec/ext_transform_feedback/alignment.c b/tests/spec/ext_transform_feedback/alignment.c index dda8823c1..aedc0f2dc 100644 --- a/tests/spec/ext_transform_feedback/alignment.c +++ b/tests/spec/ext_transform_feedback/alignment.c @@ -151,7 +151,7 @@ enum piglit_result piglit_display(void) GLuint *readback; GLuint buffer[BUFFER_SIZE]; GLuint expected[BUFFER_SIZE]; - int i; + unsigned int i; GLboolean pass = GL_TRUE; glUseProgram(prog); @@ -181,7 +181,7 @@ enum piglit_result piglit_display(void) memset(expected, 0xffffffff, sizeof(expected)); for (i = 0; i < EXPECTED_NUM_OUTPUTS; ++i) { expected[i + additional_offset / 4] = - 0x00010203 + 0x04040404 * i; + 0x00010203u + 0x04040404u * i; } /* Check output */ |