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.

Sign up for GitHub

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

Jump to bottom

Symbol lookup error: undefined symbol: jl_fptr_sparam #56701

Closed
serenity4 opened this issue Nov 27, 2024 · 2 comments · Fixed by #57083
Closed

Symbol lookup error: undefined symbol: jl_fptr_sparam #56701

serenity4 opened this issue Nov 27, 2024 · 2 comments · Fixed by #57083

Comments

@serenity4
Copy link
Contributor

serenity4 commented Nov 27, 2024 •
edited by giordano
Loading

Here is a bug apparently related to precompilation:

(@v1.12) pkg> activate --temp
  Activating new project at `/tmp/jl_Q2CEQp`

(jl_Q2CEQp) pkg> add SymbolicGA#lookup-error
... [truncated]

julia> using SymbolicGA: @ga

julia> @ga 2 (1, (1, 2))::(0, 1)
/home/serenity4/.julia/juliaup/julia-nightly/bin/julia: symbol lookup error: /home/serenity4/.julia/compiled/v1.12/SymbolicGA/krY9u_Z0AFz.so: undefined symbol: jl_fptr_sparam

git bisect indicates that the first bad commit is cd99cfc (#56179).

If commenting the line as done here: serenity4/SymbolicGA.jl@3fe2ccc, with the same reproducer (you may set it up with pkg> add SymbolicGA#segfault-reproducer - don't mind the branch name, it was from #56690), the error no longer appears and the code executes fine.
@serenity4 serenity4 mentioned this issue Nov 27, 2024
Closed
@giordano
Copy link
Contributor

git bisect indicates that the first bad commit is cd99cfc (#56179).

CC: @vtjnash

@vtjnash
Copy link
Member

vtjnash commented Jan 5, 2025

IIUC, this is supposed to be exported (along with jl_fptr_args), and we aren't doing that, so codegen currently can instead generate code which can't be loaded

vtjnash added a commit that referenced this issue Jan 17, 2025
@vtjnash
codegen: export all ABIs that we refer to in codegen
045b28c 
Since codegen may refer to these, it helps to have the linker able to
resolve them, if that case ever happens.

Fixes #56701
@vtjnash vtjnash mentioned this issue Jan 17, 2025
Merged
vtjnash added a commit that referenced this issue Jan 21, 2025
@vtjnash
codegen: export all ABIs that we refer to in codegen (#57083)
ff5f66c 
Since codegen may refer to these, it helps to have the linker able to
resolve them, if that case ever happens.

Fixes #56701
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

codegen: export all ABIs that we refer to in codegen JuliaLang/julia
3 participants
@vtjnash @giordano @serenity4