Skip to content

Illegal inttoptr in ApproxFun branch at compile time #28379

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

Closed
dlfivefifty opened this issue Jul 31, 2018 · 6 comments
Closed

Illegal inttoptr in ApproxFun branch at compile time #28379

dlfivefifty opened this issue Jul 31, 2018 · 6 comments
Labels
bug Indicates an unexpected problem or unintended behavior

Comments

@dlfivefifty
Copy link
Contributor

This may be related to #28208

The branches are a bit non-standard: I have

ApproxFun on branch "bug-illegalint"
JuliaApproximation/Domains.jl on branch "dl/approxfun"
IntervalSets.jl on PR JuliaMath/IntervalSets.jl#26

julia> versioninfo()
Julia Version 0.7.0-beta2.199
Commit 6026374bce (2018-07-31 13:20 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin17.7.0)
  CPU: Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.0 (ORCJIT, skylake)
Environment:
  JULIA_VERSION = 0.6

julia> using ApproxFun

julia> V, a = view([1.0,2.0],1:1:2),Chebyshev(0..1.0)
([1.0, 2.0], Chebyshev(0.0..1.0))

julia> precompile(ApproxFun.defaultcoefficients, (typeof(V), typeof(a),typeof(a)))
Illegal inttoptr
	  %57 = ptrtoint double addrspace(13)* %56 to i64
Illegal inttoptr
	  %72 = inttoptr i64 %71 to i8 addrspace(13)*

signal (6): Abort trap: 6
in expression starting at no file:0
__pthread_kill at /usr/lib/system/libsystem_kernel.dylib (unknown line)
Allocations: 89369522 (Pool: 89366283; Big: 3239); GC: 60
Abort trap: 6
@dlfivefifty dlfivefifty changed the title Illegal inttoptr in ApproxFun branch Illegal inttoptr in ApproxFun branch at compile time Jul 31, 2018
@Keno
Copy link
Member

Keno commented Jul 31, 2018

Please try again with the latest master and make sure to do make -C deps distclean-llvm first. We just merged an LLVM patch for an issue with these symptoms.

@dlfivefifty
Copy link
Contributor Author

No, that did not fix the problem. I updated to:

julia> versioninfo()
Julia Version 0.7.0-rc1.0
Commit 15fcc7c4e6 (2018-07-31 20:29 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin17.7.0)
  CPU: Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.0 (ORCJIT, skylake)
Environment:
  JULIA_VERSION = 0.6

@Keno
Copy link
Member

Keno commented Aug 1, 2018

Did you make sure to distclean llvm?

@dlfivefifty
Copy link
Contributor Author

Yes. I'll try wiping the whole build and starting from fresh.

@dlfivefifty
Copy link
Contributor Author

Still waiting to try a clean build, but FYI it was working fine in 0.7.0-beta2.126:

julia> precompile(ApproxFun.defaultcoefficients, (typeof(V), typeof(a),typeof(a)))
true

julia> versioninfo()
Julia Version 0.7.0-beta2.126
Commit 04dbf7a3ec (2018-07-26 05:54 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin17.7.0)
  CPU: Intel(R) Core(TM) i5-7600K CPU @ 3.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.0 (ORCJIT, skylake)
Environment:
  JULIA_VERSION = 0.6

@dlfivefifty
Copy link
Contributor Author

Still there on a fresh build:

julia> versioninfo()
Julia Version 0.7.0-rc1.4
Commit 0ef882679f (2018-08-01 00:52 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin17.7.0)
  CPU: Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.0 (ORCJIT, skylake)
Environment:
  JULIA_VERSION = 0.6

@JeffBezanson JeffBezanson added the bug Indicates an unexpected problem or unintended behavior label Aug 1, 2018
Keno added a commit that referenced this issue Aug 2, 2018
Keno added a commit that referenced this issue Aug 2, 2018
Keno added a commit that referenced this issue Aug 2, 2018
* Add LLVM patch to explicitly represent umin in SCEV

Fixes #28379
Fixes #28388

* bump LLVM BB_REL 6.0.0-6
KristofferC pushed a commit that referenced this issue Feb 11, 2019
* Add LLVM patch to explicitly represent umin in SCEV

Fixes #28379
Fixes #28388

* bump LLVM BB_REL 6.0.0-6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

3 participants