diff options
| author | Ben Widawsky <[email protected]> | 2014-11-28 14:28:19 -0800 |
|---|---|---|
| committer | Ben Widawsky <[email protected]> | 2015-03-02 19:55:20 -0800 |
| commit | 3d4d77a5dc7bc9f60d7845ff1d8d5b23f988232a (patch) | |
| tree | 755f570d64dc1dae8f37aed47019fb8e6dd581f0 /src/mesa | |
| parent | 37c2687645bf44515dbd1bc1c066b02889ca46c4 (diff) | |
i965: Fix assertion in brw_reg_type_letters
While using various debugging features (optimization debug, instruction dumping,
etc) this function is called in order to get a readable letter for the type of
unit.
On GEN8, two new units were added, the Qword and the Unsigned Qword (Q, and UQ
respectively). The existing assertion tries to determine that the argument
passed in is within the correct boundary, however, it was using UQ as the upper
limit instead of Q.
To my knowledge you can only hit this case with the branch I am currently
working on, so it doesn't fix any known issues.
Signed-off-by: Ben Widawsky <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/mesa')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_eu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_eu.c b/src/mesa/drivers/dri/i965/brw_eu.c index a1b7fda9336..146202ba24b 100644 --- a/src/mesa/drivers/dri/i965/brw_eu.c +++ b/src/mesa/drivers/dri/i965/brw_eu.c @@ -61,7 +61,7 @@ brw_reg_type_letters(unsigned type) [BRW_REGISTER_TYPE_UQ] = "UQ", [BRW_REGISTER_TYPE_Q] = "Q", }; - assert(type <= BRW_REGISTER_TYPE_UQ); + assert(type <= BRW_REGISTER_TYPE_Q); return names[type]; } |