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

fix alignment of inlining cost with verbose dinfo #198

Merged
merged 1 commit into from
Aug 4, 2021

Conversation

simeonschaub
Copy link
Collaborator

Before:
image

After:
image

@vchuravy
Copy link
Member

vchuravy commented Aug 4, 2021

Can you add a test as well if that makes sense?

@simeonschaub
Copy link
Collaborator Author

I tried out a more systematic approach to testing in #199, since I otherwise couldn't come up with a good way of testing something like this. Please let me know what you think.

@vchuravy
Copy link
Member

vchuravy commented Aug 4, 2021

Great thanks!

@vchuravy vchuravy merged commit e3cd7b1 into master Aug 4, 2021
@vchuravy vchuravy deleted the sds/inlining_verbose_dinfo branch August 4, 2021 18:35
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.

None yet

2 participants