-
Notifications
You must be signed in to change notification settings - Fork 125
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
Clad crashes when trying to initialise a class type declaration in the reverse mode #1082
Milestone
Comments
I guess what needs to be done is that we need some support of the |
PetroZarytskyi
added a commit
to PetroZarytskyi/clad
that referenced
this issue
Jan 7, 2025
Previously, we replaced ``std::initializer_list`` variables with ``clad::array`` in the reverse mode so that they become modifiable. This PR moves the logic for type replacement from ``RMV::DifferentiateVarDecl`` to a``RMV::CloneType`` to use it to handle constructors with ``std::initializer_list`` parameters. Fixes vgvassilev#1082.
PetroZarytskyi
added a commit
to PetroZarytskyi/clad
that referenced
this issue
Jan 7, 2025
Previously, we replaced ``std::initializer_list`` variables with ``clad::array`` in the reverse mode so that they become modifiable. This PR moves the logic for type replacement from ``RMV::DifferentiateVarDecl`` to a``RMV::CloneType`` to use it to handle constructors with ``std::initializer_list`` parameters. Fixes vgvassilev#1082.
PetroZarytskyi
added a commit
to PetroZarytskyi/clad
that referenced
this issue
Jan 7, 2025
Previously, we replaced ``std::initializer_list`` variables with ``clad::array`` in the reverse mode so that they become modifiable. This PR moves the logic for type replacement from ``RMV::DifferentiateVarDecl`` to a``RMV::CloneType`` to use it to handle constructors with ``std::initializer_list`` parameters. Fixes vgvassilev#1082.
PetroZarytskyi
added a commit
to PetroZarytskyi/clad
that referenced
this issue
Jan 7, 2025
Previously, we replaced ``std::initializer_list`` variables with ``clad::array`` in the reverse mode so that they become modifiable. This PR moves the logic for type replacement from ``RMV::DifferentiateVarDecl`` to a``RMV::CloneType`` to use it to handle constructors with ``std::initializer_list`` parameters. Fixes vgvassilev#1082.
PetroZarytskyi
added a commit
to PetroZarytskyi/clad
that referenced
this issue
Jan 10, 2025
Previously, we replaced ``std::initializer_list`` variables with ``clad::array`` in the reverse mode so that they become modifiable. This PR moves the logic for type replacement from ``RMV::DifferentiateVarDecl`` to a``RMV::CloneType`` to use it to handle constructors with ``std::initializer_list`` parameters. Fixes vgvassilev#1082.
vgvassilev
pushed a commit
that referenced
this issue
Jan 11, 2025
* Support std::initializer_list parameters in the reverse mode. Previously, we replaced ``std::initializer_list`` variables with ``clad::array`` in the reverse mode so that they become modifiable. This PR moves the logic for type replacement from ``RMV::DifferentiateVarDecl`` to a``RMV::CloneType`` to use it to handle constructors with ``std::initializer_list`` parameters. Fixes #1082.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
reproducer:
same thing is happening with other types
The text was updated successfully, but these errors were encountered: