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 linking #7

Open
mratsim opened this issue Jan 10, 2021 · 0 comments
Open

Fix linking #7

mratsim opened this issue Jan 10, 2021 · 0 comments
Labels

Comments

@mratsim
Copy link
Collaborator

mratsim commented Jan 10, 2021

Currently we use a mix of dynamic linking with passL: -lc10 -ltorch_cpu and static linking with {.link.}

We want to cleanly use dynamic linking for starter as the linker step is very slow and c10/torch have no static lib provided, at least on Linux.

So we want either dynlib or the passL strategy but without any {.link.}.
Furthermore we need to check where we need the library o minimize path issues, is it nimble bin or does it have a lib folder?

@mratsim mratsim added the devops label Jan 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant