Skip to content
  • Ian Romanick's avatar
    nir: Simplify min and max of b2f · 68420d83
    Ian Romanick authored
    
    
    v2: Rebase on almost 2 years.  Require that one of the arguments to fmin
    or fmax be used only once.  This prevents some regressions.
    
    shader-db results:
    
    Skylake and Broadwell had similar results.  Skylake shown.
    total instructions in shared programs: 14526021 -> 14525913 (<.01%)
    instructions in affected programs: 4613 -> 4505 (-2.34%)
    helped: 31
    HURT: 0
    helped stats (abs) min: 1 max: 4 x̄: 3.48 x̃: 4
    helped stats (rel) min: 0.62% max: 6.67% x̄: 3.31% x̃: 2.42%
    
    total cycles in shared programs: 533118710 -> 533118403 (<.01%)
    cycles in affected programs: 34334 -> 34027 (-0.89%)
    helped: 24
    HURT: 0
    helped stats (abs) min: 4 max: 24 x̄: 12.79 x̃: 14
    helped stats (rel) min: 0.25% max: 2.40% x̄: 1.08% x̃: 1.03%
    
    No changes on GM45, Iron Lake, Sandy Bridge, Ivy Bridge, or Haswell.
    
    Signed-off-by: default avatarIan Romanick <ian.d.romanick@intel.com>
    Reviewed-by: default avatarSamuel Iglesias Gonsálvez <siglesias@igalia.com>
    Reviewed-by: default avatarElie Tournier <elie.tournier@collabora.com>
    68420d83