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

[stateHandling] Remove swap semantics optimization support. #1710

Merged

Conversation

schweitzpgi
Copy link
Collaborator

Follows the latest design change for ownership management of state objects. Remove the support for "move semantics" to allow the logical lifetime of a state object to escape into called functions and the swapping of any resources it may contain.

  • remove the move constructor from qvector
  • remove move attribute from init_state op
  • remove explicit test for move semantics
  • simplify the handling of init_state data: it has to be a pointer to a supported data type, floating-point, complex, or state.

Description

Follows the latest design change for ownership management of state objects.
Remove the support for "move semantics" to allow the logical lifetime of a
state object to escape into called functions and the swapping of any
resources it may contain.
  - remove the move constructor from qvector
  - remove move attribute from init_state op
  - remove explicit test for move semantics
  - simplify the handling of init_state data: it has to be a pointer
    to a supported data type, floating-point, complex, or state.
github-actions bot pushed a commit that referenced this pull request May 24, 2024
Copy link
Collaborator

@1tnguyen 1tnguyen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@schweitzpgi schweitzpgi merged commit 24accba into NVIDIA:experimental/stateHandling May 24, 2024
125 checks passed
@schweitzpgi schweitzpgi deleted the sh-remove.dol branch May 24, 2024 16:02
@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants