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

jl_setjmp is not exported on Windows by the LTS version #43340

Closed
Taaitaaiger opened this issue Dec 5, 2021 · 2 comments
Closed

jl_setjmp is not exported on Windows by the LTS version #43340

Taaitaaiger opened this issue Dec 5, 2021 · 2 comments
Labels
system:windows Affects only Windows
Milestone

Comments

@Taaitaaiger
Copy link
Contributor

Taaitaaiger commented Dec 5, 2021

Sorry for not noticing this sooner despite opening an issue about it before, but on Windows jl_setjmp is not exported by the LTS version. As a result the JL_TRY and JL_CATCH macros can't be used when embedding Julia.

#42245

@ViralBShah ViralBShah added the system:windows Affects only Windows label Dec 5, 2021
@IanButterworth
Copy link
Member

IanButterworth commented Dec 5, 2021

There are also some jl_setjmp redefinition warnings on master, at least on MacOS

    CC src/llvm-pass-helpers.o
In file included from /Users/ian/Documents/GitHub/julia/src/llvm-pass-helpers.cpp:18:
./jl_internal_funcs.inc:513:9: warning: 'jl_setjmp' macro redefined [-Wmacro-redefined]
#define jl_setjmp ijl_setjmp
        ^
./julia.h:1908:9: note: previous definition is here
#define jl_setjmp(a,b) sigsetjmp(a,b)
        ^
1 warning generated.
    CC src/llvm-late-gc-lowering.o

@JeffBezanson
Copy link
Member

#42267 should be backported

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
system:windows Affects only Windows
Projects
None yet
Development

No branches or pull requests

5 participants