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

pan/bi: Simplify bi_compose_clamp 

Realized this trick when reversing Valhall.

Signed-off-by: Alyssa Rosenzweig <alyssa@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12205>
This commit is contained in:
Alyssa Rosenzweig 2021-08-03 13:47:59 -04:00
parent cfd73b4542
commit c27af4ad36
1 changed files with 2 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/panfrost/bifrost/bi_opt_mod_props.c
View File

@ -157,20 +157,6 @@ bi_takes_clamp(bi_instr *I)
}
}
/* Treating clamps as functions, compute the composition f circ g. For {NONE,
* SAT, SAT_SIGNED, CLAMP_POS}, anything left- or right-composed with NONE is
* unchanged, anything composed with itself is unchanged, and any two
* nontrivial distinct clamps compose to SAT (left as an exercise) */
static enum bi_clamp
bi_compose_clamp(enum bi_clamp f, enum bi_clamp g)
{
return (f == BI_CLAMP_NONE) ? g :
(g == BI_CLAMP_NONE) ? f :
(f == g) ? f :
BI_CLAMP_CLAMP_0_1;
}
static bool
bi_is_fclamp(bi_instr *I)
{
@ -187,7 +173,8 @@ bi_optimizer_clamp(bi_instr *I, bi_instr *use)
if (!bi_is_fclamp(use)) return false;
if (!bi_takes_clamp(I)) return false;
I->clamp = bi_compose_clamp(I->clamp, use->clamp);
/* Clamps are bitfields (clamp_m1_1/clamp_0_inf) so composition is OR */
I->clamp |= use->clamp;
I->dest[0] = use->dest[0];
return true;
}