Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • VTK-m VTK-m
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 191
    • Issues 191
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 35
    • Merge requests 35
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • VTK
  • VTK-mVTK-m
  • Merge requests
  • !119

Change StorageBasic to use an aligned allocator.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Will Usher requested to merge Twinklebear/vtk-m:add-aligned-storage into master Aug 05, 2015
  • Overview 48
  • Commits 1
  • Changes

The storage used will now be aligned to 64 bytes, resulting in slightly better cache usage and load/store performance.

The allocator used should (I think) also be STL compatible so if user's want an aligned vector then they can use the StorageBasic allocator. This will also help in the case that they provide memory to the storage since we don't do anything in the case that the user provides unaligned memory (things will just be slower).

I also switched benchmarks that would use unaligned user memory from a std::vector to just do the allocation in an aligned StorageBasic, since the vectors weren't really needed anyways.

I'm a bit unsure on the portability of the aligned allocation functions used but it sounds like they should be available in most places, let me know if there's a better alternative or other platform to support.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: add-aligned-storage