diff options
| author | Roland Scheidegger <[email protected]> | 2014-12-05 23:08:34 +0100 |
|---|---|---|
| committer | Roland Scheidegger <[email protected]> | 2014-12-06 18:03:25 +0100 |
| commit | d8da6deceadf5e48201d848b7061dad17a5b7cac (patch) | |
| tree | 7b7af19b59b8b2ead4ccbe2cdb7dc08be8cab551 /src/loader/loader.c | |
| parent | 6f2cf5f3d0764e096b6b099ef9dc7bc92047c3cb (diff) | |
mesa/st: don't use CMP / I2F for conditional assignments with native integers
The original idea was to optimize away the condition by integrating it directly
into the CMP instruction. However, with native integers this requires an extra
I2F instruction. It is also fishy because the negation used didn't really honor
ieee754 float comparison rules, not to mention the CMP instruction itself
(being pretty much a legacy instruction) doesn't really have defined special
float value behavior in any case.
So, use UCMP and adjust the code trying to optimize the condition away
accordingly (I have absolutely no idea if such conditions are actually hit
or would be translated away somewhere else already).
v2: cosmetic changes
No piglit regressions on llvmpipe.
Reviewed-by: Jose Fonseca <[email protected]>
Reviewed-by: Brian Paul <[email protected]>
Diffstat (limited to 'src/loader/loader.c')
0 files changed, 0 insertions, 0 deletions