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

Library doesn't build w/ ICC on OSX #2

Closed
amanoel opened this issue Sep 4, 2015 · 6 comments
Closed

Library doesn't build w/ ICC on OSX #2

amanoel opened this issue Sep 4, 2015 · 6 comments

Comments

@amanoel
Copy link

amanoel commented Sep 4, 2015

Following the discussion at JuliaLang/julia#9145, the .s extension must be capitalized (and the Makefile changed accordingly) in order for ICC to be able to build it on OSX.
I'm not sure if doing this wouldn't break the build on other OSes though...

@tkelman
Copy link

tkelman commented Sep 4, 2015

That would surprise me. We only use this library on OSX anyway.

@amanoel
Copy link
Author

amanoel commented Sep 4, 2015

Hmm, if it's only used in OSX, wouldn't it be a good idea to always use the system-provided library?
I've tried using the USE_SYSTEM_LIBUNWIND=1 flag and apparently it's also a good way to circumvent this.

@tkelman
Copy link

tkelman commented Sep 4, 2015

You'd have to ask @Keno exactly what purpose this serves. I dunno the difference USE_SYSTEM_LIBUNWIND would make in practice.

@Keno
Copy link
Member

Keno commented Sep 4, 2015

You'd get less accurate profiles. The modifications in this library have some heuristics that allow it to unwind from non-callsite locations.

@nolta
Copy link
Member

nolta commented Sep 7, 2015

Another option would be to add -x assembler-with-cpp to SFLAGS.

Tried to fix this, but i don't have permission.

@amanoel
Copy link
Author

amanoel commented Sep 8, 2015

Can confirm make CC="icc" SFLAGS="$SFLAGS -x assembler-with-cpp" works!

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

No branches or pull requests

4 participants