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

Add NEWS and docs for the new inlining algorithm #22775

Merged
merged 1 commit into from
Jul 13, 2017
Merged

Conversation

timholy
Copy link
Member

@timholy timholy commented Jul 12, 2017

I also included a number of useful tips I've accumulated while hacking on inference.jl. Hopefully this will save head-scratching for future generations. (Others are encouraged to add to this!)

[ci skip]

cases. For example, an `:invoke` expression (a call for which all
input and output types were inferred in advance) is assigned a fixed
cost (currently 20 cycles). In contrast, a `:call` expression, for
other than intrinsics/builtins, indicates that the call will require
Copy link
Contributor

Choose a reason for hiding this comment

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

for functions other than ?

@@ -0,0 +1,98 @@

# Inference
Copy link
Member

Choose a reason for hiding this comment

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

Perhaps we should add links to https://juliacomputing.com/blog/2016/04/04/inference-convergence.html and https://juliacomputing.com/blog/2017/05/15/inference-converage2.html to describe the overall structure?

I should also probably add a description of #21933 here too, later, when I get a chance.

@timholy timholy force-pushed the teh/docs_inlining branch from 9d6e5bf to 70dfb6d Compare July 13, 2017 12:02
@timholy timholy merged commit 27e6f17 into master Jul 13, 2017
@timholy timholy deleted the teh/docs_inlining branch July 13, 2017 12:04
@timholy
Copy link
Member Author

timholy commented Jul 13, 2017

Hmm, link is broken. Any idea what I missed?

@timholy
Copy link
Member Author

timholy commented Jul 13, 2017

I see there's a blank line at the top of inference.md. Could that be it? (I am not aware of a way to test without merging to master, so I thought it would be worth asking if anyone knows.)

@timholy
Copy link
Member Author

timholy commented Jul 13, 2017

Fixed in 9c1d1b8

@tkelman
Copy link
Contributor

tkelman commented Jul 13, 2017

make docs ?

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.

3 participants