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
  • tests
  • compiler_tests
  • ImmutableString_test_autogen.cpp
Find file
BlameHistoryPermalink
  • Olli Etuaho's avatar
    Rely on hash to check for some mangled name matches · e79d0f86
    Olli Etuaho authored Mar 19, 2018
    When we are looking up a function with only a few parameters, we can
    optimize the lookup by relying on the information encoded in the hash
    value. There's often only one list of parameters with the same
    function name and mangled name length that results in a matching hash,
    so we don't actually need to compare the full mangled name. We can
    just compare
    
    1) the hash value of the mangled name
    2) the mangled name length
    3) the function name
    
    to make sure that the mangled name matches the mangled name of the
    function.
    
    This decreases the binary size since we don't need store as many
    mangled names of built-in functions. Effect on symbol lookup speed is
    marginal.
    
    BUG=angleproject:2267
    BUG=chromium:823856
    TEST=angle_unittests
    
    Change-Id: I3ef41d943209509d4e8e6ece14ebad7e2677abc6
    Reviewed-on: https://chromium-review.googlesource.com/973242Reviewed-by: 's avatarCorentin Wallez <cwallez@chromium.org>
    Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org>
    Commit-Queue: Olli Etuaho <oetuaho@nvidia.com>
    e79d0f86
ImmutableString_test_autogen.cpp 76 KB
EditWeb IDE
×

Replace ImmutableString_test_autogen.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.