Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • mesa mesa
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3,058
    • Issues 3,058
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1,002
    • Merge requests 1,002
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • MesaMesa
  • mesamesa
  • Merge requests
  • !1873

gallium & radeonsi NIR: Disable GLSL const initializer lowering

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Connor Abbott requested to merge cwabbott0/mesa:review/gallium-disable-glsl-const into master Sep 04, 2019
  • Overview 3
  • Commits 4
  • Pipelines 2
  • Changes 9

We now have a better alternative with nir_opt_large_constants. Also, NIR optimizations may mean that indirect accesses to the variable turn into direct ones, e.g. because NIR unrolled a loop that GLSL didn't, and the old pass left no way for NIR to recover the original constant value and remove the variable.

This has to be a separate PIPE_CAP because some NIR-using gallium drivers don't call nir_opt_large_constants yet. We enable the cap for radeonsi when NIR is enabled, but other drivers like iris or i965 that use nir_opt_large_constants could also benefit.

This is based on !1844 (merged).

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: review/gallium-disable-glsl-const