Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore memory management object for native alloc funcitons #3300

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

umar456
Copy link
Member

@umar456 umar456 commented Sep 29, 2022

This PR adds a new memory manager example and fixes the native alloc function so that it ignores the af_memory_manager object.

Description

  • Creates a new memory manager example
  • The native alloc and native free functions now ignore the af_memory_manager object because they now allocate on the current device.
  • Empty allocations on the CPU backend do not call the alloc function

Changes to Users

N/A

Checklist

  • Rebased on latest master
  • Code compiles
  • Tests pass
  • Functions added to unified API
  • Functions documented

The native alloc and free functions are used to allocate and free
functions. These functions do not need the memory manager implementation
to work because they are just allocating directly with the underlying
backend. This change will now ignore the memory manager object
when allocating the memory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants