Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • This project
    • Loading...
  • Sign in / Register
A
angle
  • Project
    • Overview
    • Details
    • Activity
    • Cycle Analytics
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
  • Issues 0
    • Issues 0
    • List
    • Board
    • Labels
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Chen Yisong
  • angle
  • Repository

Switch branch/tag
  • angle
  • src
  • compiler
  • translator
  • VersionGLSL.cpp
Find file
BlameHistoryPermalink
  • Olli Etuaho's avatar
    Don't use TIntermSymbol nodes for function parameters · d4bd963f
    Olli Etuaho authored Mar 08, 2018
    Parameter nodes are not needed - it's simpler to just create a
    TVariable object for each parameter when the TFunction is initialized.
    
    With this change we also store only one object per each parameter type
    used in built-in functions, instead of one array of TConstParameter
    entries for each unique parameter sequence.
    
    This simplifies code and reduces binary size and compiler memory use.
    Compiler perf does not seem to be significantly affected.
    
    BUG=angleproject:2267
    TEST=angle_unittests
    
    Change-Id: I2b82400dd594731074309f92a705e75135a4c82c
    Reviewed-on: https://chromium-review.googlesource.com/955589
    Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
    Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
    d4bd963f
VersionGLSL.cpp 4.49 KB
EditWeb IDE
×

Replace VersionGLSL.cpp

Attach a file by drag & drop or click to upload


Cancel
A new branch will be created in your fork and a new merge request will be started.