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

[stdlib docs] Information on Dependent types #3999

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

owenhilyard
Copy link
Contributor

Chris gave some information I thought should be documented for stdlib developers.

  • always_inline("builtin") vs always_inline("nodebug")
  • Some information on MLIR interpreter limitations.

@ConnorGray
Copy link
Collaborator

Hey @owenhilyard, thanks for contributing this! :)

This looks basically good to me, though I wonder if you might consider including footnotes to the places where Chris' mentions the things added here? Even if the best sources at the moment are things Chris has said on Discord or forum posts, I think any "breadcrumbs" we can give future readers are better than none.

* `always_inline("builtin")` vs `always_inline("nodebug")`
* Some information on MLIR interpreter limitations.

Signed-off-by: Owen Hilyard <[email protected]>
@owenhilyard owenhilyard force-pushed the always_inline_builtin branch from 17de17b to cb23cc3 Compare February 14, 2025 23:29
@owenhilyard
Copy link
Contributor Author

Hey @owenhilyard, thanks for contributing this! :)

This looks basically good to me, though I wonder if you might consider including footnotes to the places where Chris' mentions the things added here? Even if the best sources at the moment are things Chris has said on Discord or forum posts, I think any "breadcrumbs" we can give future readers are better than none.

I've included links to the Discord thread where Chris gave the explanation. I have no idea how stable those are long-term, but we shall see.

@ConnorGray
Copy link
Collaborator

Thanks Owen! Understood, I don't think it'll be great to rely on Discord as a primary source long-term ;)

But perhaps that will tide us over until the Mojo compiler is open source🙂

@ConnorGray
Copy link
Collaborator

!sync

@modularbot modularbot added the imported-internally Signals that a given pull request has been imported internally. label Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
imported-internally Signals that a given pull request has been imported internally.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants