Skip to content

Add INLINE and INLINABLE pragmas - #1252

Open
meooow25 wants to merge 1 commit into
haskell:masterfrom
meooow25:more-inlinable
Open

meooow25 wants to merge 1 commit into
haskell:masterfrom
meooow25:more-inlinable

Conversation

@meooow25

Copy link
Copy Markdown
Contributor
  • Add INLINE where we surely want to inline, e.g. for list fusion or if the body is very small.
  • Add INLINABLE for specialization or to expose the unfolding. See the added Note [INLINABLE to expose unfoldings] for an explanation of the latter.

Closes #1243

@alexfmpe alexfmpe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Was reading out of curiosity and noticed a couple typos

Comment thread containers/src/Data/IntMap/Internal.hs Outdated
Comment thread containers/src/Data/IntMap/Internal.hs Outdated
* Add INLINE where we surely want to inline, e.g. for list fusion or
  if the body is very small.
* Add INLINABLE for specialization or to expose the unfolding. See the
  added Note [INLINABLE to expose unfoldings] for an explanation of the
  latter.
@meooow25

Copy link
Copy Markdown
Contributor Author

Thanks for spotting those, fixed!

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.

Data.IntMap.unionWithKey is slower than mergeWithKey

2 participants