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 c_profile rake tasks for MacOS, using the Xcode instruments command #62

Merged
merged 1 commit into from
Oct 1, 2020

Conversation

dylanahsmith
Copy link
Contributor

Problem

As we move more code into C, we will have less visibility into where time is being spent from the ruby profile.

Solution

Add c_profile rake tasks that uses the instruments Xcode command. I added separate tasks to this c_profile namespace for compile, render and run (compile and render). These leverage the same ThemeRunner that the profile:run task uses, which is defined in the liquid gem.

@dylanahsmith dylanahsmith merged commit d0012d2 into master Oct 1, 2020
@dylanahsmith dylanahsmith deleted the c-profile branch October 1, 2020 15:00
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.

2 participants