Skip to content

Conversation

@t6s
Copy link
Member

@t6s t6s commented Feb 18, 2025

Motivation for this change

In reals_stdlib/Rstruct.v, GRing.inv has been instantiated not by Rinv, but Rinvx,
which is separately defined in Rstruct but is extensionally equal to Rinv.
This PR removes this redundancy and deprecates Rinvx.

Infotheo will benefit from this change when extracting real-number algorithms.
(affeldt-aist/infotheo#142)

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Reminder to reviewers

@t6s t6s requested a review from affeldt-aist February 18, 2025 18:35
@affeldt-aist affeldt-aist merged commit aec3e47 into math-comp:master Feb 19, 2025
45 checks passed
@t6s t6s deleted the deprecate_Rinvx branch February 19, 2025 13:33
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.

2 participants