Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • mesa mesa
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 3.2k
    • Issues 3.2k
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1k
    • Merge requests 1k
  • 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
  • !6591

WIP: nir: split ffma

  • Review changes

  • Download
  • Patches
  • Plain diff
Open Karol Herbst requested to merge karolherbst/mesa:nir_ffma_split into main Sep 03, 2020
  • Overview 34
  • Commits 2
  • Pipelines 6
  • Changes 25

This will allow us to track better the intent of the original source and enable more optimizations depending on hardware support.

Also this will be useful for OpenCL where the spec requires ffma to be fused and a * b + c to be unfused except -cl-mad-enable was added to the compiler options.

TODOS:

  • in glsl_to_nir we should emit nir_op_fmad for hardware without nir_op_ffma support for fma.
  • wire up nir_op_fmad in more drivers supporting it.
  • Find solution for drivers wanting to support neither, but keep nir_op_ffma around
  • Find solution for drivers having both, but has a strong preference on one. E.g. nir_op_fmad being faster on some AMD hardware.
  • Splitting up nir_op_ffma first doing optimizations and merging it back can result in better codegen.
Edited Sep 03, 2020 by Karol Herbst
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: nir_ffma_split