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

Illegal instruction on Julia v1.2-rc1 (regression) #32386

Closed
oschulz opened this issue Jun 21, 2019 · 10 comments
Closed

Illegal instruction on Julia v1.2-rc1 (regression) #32386

oschulz opened this issue Jun 21, 2019 · 10 comments
Assignees
Labels
bug Indicates an unexpected problem or unintended behavior regression Regression in behavior compared to a previous version

Comments

@oschulz
Copy link
Contributor

oschulz commented Jun 21, 2019

On Julia v1.2-rc1 and v1.3.0-DEV.433 (2019-06-20), we get an Illegal instruction error with SolidStateDetectors:

(v1.2) pkg> add TypedTables#master SolidStateDetectors
(v1.2) pkg> test SolidStateDetectors
   Testing SolidStateDetectors
 Resolving package versions...
 [...]
Unreachable reached at 0x7f75351968fa

signal (4): Illegal instruction
in expression starting at [...]/packages/SolidStateDetectors/7T4zz/test/runtests.jl:8
construct_semiconductor at [...]/packages/SolidStateDetectors/7T4zz/src/SolidStateDetector/SolidStateDetector.jl:72
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2197
construct_objects at [...]/packages/SolidStateDetectors/7T4zz/src/SolidStateDetector/SolidStateDetector.jl:86
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2197
[...]

Note: Needs TypedTables#master until a new version of TypedTables is released (see JuliaData/TypedTables.jl#52).

On Julia v1.1.1 all is fine.

Not sure if this is related to JuliaLang/LinearAlgebra.jl#637 - it also happens with the current nightly though, which has JuliaLang/LinearAlgebra.jl#637 fixed.

We've (@oschulz, @lmh91, @hervasa2) run into this on Ubuntu 16, Ubuntu 19, CentOS-7 and OS-X, so doesn't seem to be OS-related.

@ararslan ararslan added the bug Indicates an unexpected problem or unintended behavior label Jun 21, 2019
@JeffBezanson
Copy link
Member

This is slightly similar to JuliaLang/LinearAlgebra.jl#637 in that I can't immediately figure out what's going on...

@oschulz
Copy link
Contributor Author

oschulz commented Jun 21, 2019

Sorry that we didn't provide an MWE, didn't manage to put one together so far that reproduces the error. Will be above be Ok for you to track it down?

@JeffBezanson
Copy link
Member

Yes, the above is enough.

@oschulz
Copy link
Contributor Author

oschulz commented Jun 21, 2019

Thanks, Jeff!

@JeffBezanson JeffBezanson added the regression Regression in behavior compared to a previous version label Jun 25, 2019
@JeffBezanson JeffBezanson added this to the 1.2 milestone Jun 25, 2019
@JeffBezanson
Copy link
Member

Seems to be caused by e3de220. Looks like this is a simpler issue than JuliaLang/LinearAlgebra.jl#637 after all.

@JeffBezanson JeffBezanson self-assigned this Jun 25, 2019
JeffBezanson added a commit that referenced this issue Jun 25, 2019
@oschulz
Copy link
Contributor Author

oschulz commented Jun 26, 2019

Ah, e3de220 isn't part of v1.2-rc1, right? Will it go into rc2?

@JeffBezanson
Copy link
Member

No I believe it is in 1.2-rc1.

@oschulz
Copy link
Contributor Author

oschulz commented Jun 26, 2019

Ah, sorry, silly me - of course it is, otherwise we wouldn't have run into this issue, after all.

(Did look in the history of 1.2-rc1 via gitk, and must have overlooked it.)

@oschulz
Copy link
Contributor Author

oschulz commented Jun 26, 2019

So I guess fix #31993 needs a fix itself? :-)

Sorry I can't be of any help, here, I'm really not familiar enough with the internals of the Julia compiler ...

JeffBezanson added a commit that referenced this issue Jun 26, 2019
JeffBezanson added a commit that referenced this issue Jun 27, 2019
might be related to #24333 and/or #21153
(cherry picked from commit 0d9c72d)
@oschulz
Copy link
Contributor Author

oschulz commented Jun 27, 2019

Thanks a lot for the speedy fix, @JeffBezanson!

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 regression Regression in behavior compared to a previous version
Projects
None yet
Development

No branches or pull requests

3 participants