Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • A aml
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 33
    • Issues 33
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 7
    • Merge requests 7
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • argo
  • aml
  • Merge requests
  • !38

Remove arenas, move operations.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Swann Perarnau requested to merge noarena into master Mar 25, 2019
  • Overview 7
  • Commits 67
  • Pipelines 1
  • Changes 223+

New, simpler area implementation. Arena abstraction is removed but can be re-insterted later as an area allocator. The main reason for that change is the simplification of the abstraction that most users will see first: area. The linux implementation in particular was too complex, with too much inner details leaking out.

This merge request also remove dma move operations from the library, to allow us to focus on what works: copies & transforms, while simplifying the overall API. The reality is that move can never be really performant, and including it makes the rest of the library more complex.

Major side-effects of this merge request:

  • jemalloc not needed at all, and might be reintroduced as an external dependency later if needed.
  • dma/scratch loose some generic API calls.
  • most of area API redesigned.

Fixes #23 (closed), #20 (closed).

Closes #15 (closed), #5 (closed), #7 (closed), #12 (closed) (not longer valid).

Edited Mar 25, 2019 by Swann Perarnau
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: noarena