-
Notifications
You must be signed in to change notification settings - Fork 428
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
feat: non-opaque UInt64.toUSize
#6202
Conversation
It is not entirely clear to me whether these definitions belong in |
Mathlib CI status (docs):
|
41f2122
to
bd444f2
Compare
As far as I am aware, the only reason |
bd444f2
to
edcaf9b
Compare
tweak `UInt64.toUSize` def
edcaf9b
to
5619fac
Compare
Correct. The import cycle here is due to the fact that some |
This PR makes
USize.toUInt64
a regular non-opaque definition.It also moves it to
Init.Data.UInt.Basic
, as it is not actually used inInit.Prelude
anymore.