KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

nir/idiv_const: improve idiv(n, INT_MIN) 

This lowering is smaller and -INT64_MIN is probably UB (signed overflow).

No fossil-db changes.

Signed-off-by: Rhys Perry <pendingchaos02@gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12039>
This commit is contained in:
Rhys Perry 2021-07-21 17:15:51 +01:00 committed by Marge Bot
parent 4e2b94331b
commit 96168301f9
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/compiler/nir/nir_opt_idiv_const.c
View File

@ -65,6 +65,10 @@ build_umod(nir_builder *b, nir_ssa_def *n, uint64_t d)
static nir_ssa_def *
build_idiv(nir_builder *b, nir_ssa_def *n, int64_t d)
{
int64_t int_min = u_intN_min(n->bit_size);
if (d == int_min)
return nir_b2i(b, nir_ieq_imm(b, n, int_min), n->bit_size);
uint64_t abs_d = d < 0 ? -d : d;
if (d == 0) {