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

code_native: "Unable to find function pointer" #14592

Closed
eschnett opened this issue Jan 7, 2016 · 2 comments · Fixed by #14904
Closed

code_native: "Unable to find function pointer" #14592

eschnett opened this issue Jan 7, 2016 · 2 comments · Fixed by #14904

Comments

@eschnett
Copy link
Contributor

eschnett commented Jan 7, 2016

I see this (Julia master):

julia> code_native(muladd, (Float64,Float64,Float64))
WARNING: Unable to find function pointer

However, this works:

julia> code_native(muladd, (Float32,Float32,Float32))
    .section    __TEXT,__text,regular,pure_instructions
    pushq   %rbp
    movq    %rsp, %rbp
    vfmadd213ss %xmm2, %xmm1, %xmm0
    popq    %rbp
    retq
    nopl    (%rax,%rax)

This works as well:

julia> f(x,y,z) = muladd(x,y,z)
f (generic function with 1 method)

julia> code_native(f, (Float64,Float64,Float64))
    .section    __TEXT,__text,regular,pure_instructions
    pushq   %rbp
    movq    %rsp, %rbp
    vfmadd213sd %xmm2, %xmm1, %xmm0
    popq    %rbp
    retq
    nopl    (%rax,%rax)
@vtjnash
Copy link
Member

vtjnash commented Jan 9, 2016

what is versioninfo() for this build?

@eschnett
Copy link
Contributor Author

eschnett commented Jan 9, 2016

This is

julia> versioninfo()
Julia Version 0.5.0-dev+2054
Commit 9464e6f* (2016-01-08 22:06 UTC)
Platform Info:
  System: Darwin (x86_64-apple-darwin15.2.0)
  CPU: Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas64_
  LIBM: libopenlibm
  LLVM: libLLVM-3.7.1

vtjnash added a commit that referenced this issue Feb 2, 2016
the logic was not equivalent to the logic for the old JIT,
and thus was not going to work.
this make both better by looking only at whether there is known to be debug info for this llvmf

fixes #14592
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 a pull request may close this issue.

2 participants