Skip to content
This repository has been archived by the owner on Mar 18, 2021. It is now read-only.

Initial hackish implementation of inlining #1

Merged
merged 4 commits into from Feb 1, 2016

Conversation

davidmalcolm
Copy link
Collaborator

I had a go at implementing function call inlining ("inline substitution")

This is nowhere near ready yet (see the various TODO and FIXME notes), but hopefully is interesting.

vstinner added a commit that referenced this pull request Feb 1, 2016
Initial hackish implementation of inlining
@vstinner vstinner merged commit 98596c8 into vstinner:master Feb 1, 2016
@vstinner
Copy link
Owner

vstinner commented Feb 1, 2016

Oh great! FYI my motivation to write FAT Python was your patch https://bugs.python.org/issue10399 (funtion inlining)! IMHO we will get visible speedup on macrobenchmarks when the function inlining will be fully functional.

@vstinner
Copy link
Owner

vstinner commented Feb 1, 2016

I made changes to your code:

  • fix Config constructor: initialize inlining attribute (and document it with a comment)
  • respect the configuration and fix the unit test on config
  • document the optimization and its configuration in doc/ (optim, config, changelog)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants