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.

Sign up for GitHub

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

Jump to bottom

Investigate inlining macros #17

Open
t3kt opened this issue Jul 12, 2020 · 1 comment
Open

Investigate inlining macros #17

t3kt opened this issue Jul 12, 2020 · 1 comment
Labels
compiler performance

Comments

@t3kt
Copy link
Owner

t3kt commented Jul 12, 2020

See if it improves compile times
@t3kt
Copy link
Owner Author

t3kt commented Jul 12, 2020

Could potentially use a python library that can do preprocessor evaluation.

t3kt added a commit that referenced this issue Jul 14, 2020
@t3kt
partially completed (but disabled) parameter alias inlining
2675213 
There's a bug related to the order of the replacements that it does, so it's switched off for now.
See #17
t3kt added a commit that referenced this issue Jul 14, 2020
@t3kt
fix inlining bug
9557158 
it's not yet clear if it improves performance.

see #17
t3kt added a commit that referenced this issue Aug 20, 2020
@t3kt
initial implementation of embedding some preprocessing in definitionCore
69d1211 
see #16, #17
t3kt added a commit that referenced this issue Aug 20, 2020
@t3kt
more work on preprocessor
911dea5 
 it isn't really working yet though.

see #16, #17
t3kt added a commit that referenced this issue Aug 28, 2020
@t3kt
fix coord type replacement bug
46ee423 
related to #17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
compiler performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@t3kt