Skip to content

Added Tracy Profiler#221

Open
MrDrElliot wants to merge 1 commit intoPanosK92:masterfrom
MrDrElliot:Tracy
Open

Added Tracy Profiler#221
MrDrElliot wants to merge 1 commit intoPanosK92:masterfrom
MrDrElliot:Tracy

Conversation

@MrDrElliot
Copy link
Copy Markdown
Collaborator

This PR adds the amazing Tracy profiler.

https://github.com/wolfpld/tracy

image image

Insanely efficient profiling with a very nice UI. For convenience I also added a button to open Tracy.

image

Panos, since you control the binaries separately, you will need to download and add the binaries to the correct locations and setup premake to be able to handle the new binaries. This includes adding the executable for Tracy so it can be launched from the editor if desired.

@kristiker
Copy link
Copy Markdown

Can it not be a dll why add 50k loc

@MrDrElliot
Copy link
Copy Markdown
Collaborator Author

Doesn't really matter either way, these translation units won't affect compile-times in any reasonable measure. Adding DLLs just adds complexity that isn't needed (they must be maintained through an external dropbox), however Spartan compiles monolithically, and does not produce its own libraries for things. You're either going to have to stick the DLL everywhere you want to use Spartan, or you build monolithically.

The other option is to refactor Spartan's premake files to build all third party dependencies into their own libraries and link them separately.

However, no matter what's done, it's introducing complexity. Which is why I haven't merged this PR myself (and probably will not).

This will likely be closed, since it depends on Panos introducing new dependencies to the project's already large Dropbox.

Thanks!

@kristiker
Copy link
Copy Markdown

Thank you!

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