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

[LLVM] bump to 12.0.1-4 #42512

Merged
merged 3 commits into from
Oct 6, 2021
Merged

[LLVM] bump to 12.0.1-4 #42512

merged 3 commits into from
Oct 6, 2021

Conversation

vchuravy
Copy link
Member

@vchuravy vchuravy commented Oct 5, 2021

@vchuravy vchuravy requested review from vtjnash and tkf October 6, 2021 00:34
@DilumAluthge
Copy link
Member

@staticfloat Looks like we are getting some Buildkite failures on amdci7 but not on amdci5. An example:

file                                (7) |         failed at 2021-10-06T00:49:38.236
Error During Test at /cache/build/amdci7-2/julialang/julia-master/julia-19134592a9/share/julia/test/testdefs.jl:21
  Got exception outside of a @test
  LoadError: IOError: chown("/tmp/jl_FQMcP7/bfile.txt", -2, -1): invalid argument (EINVAL)
  Stacktrace:
    [1] uv_error
      @ ./libuv.jl:97 [inlined]
    [2] chown(path::String, owner::Int64, group::Int64)
      @ Base.Filesystem ./file.jl:1168
    [3] top-level scope
      @ /cache/build/amdci7-2/julialang/julia-master/julia-19134592a9/share/julia/test/file.jl:535
    [4] include
      @ ./Base.jl:416 [inlined]
    [5] macro expansion
      @ /cache/build/amdci7-2/julialang/julia-master/julia-19134592a9/share/julia/test/testdefs.jl:24 [inlined]
    [6] macro expansion
      @ /cache/build/amdci7-5/julialang/julia-master/usr/share/julia/stdlib/v1.8/Test/src/Test.jl:1321 [inlined]
    [7] macro expansion
      @ /cache/build/amdci7-2/julialang/julia-master/julia-19134592a9/share/julia/test/testdefs.jl:23 [inlined]
    [8] macro expansion
      @ ./timing.jl:380 [inlined]
    [9] runtests(name::String, path::String, isolate::Bool; seed::UInt128)
      @ Main /cache/build/amdci7-2/julialang/julia-master/julia-19134592a9/share/julia/test/testdefs.jl:21
   [10] invokelatest(::Any, ::Any, ::Vararg{Any}; kwargs::Base.Pairs{Symbol, UInt128, Tuple{Symbol}, NamedTuple{(:seed,), Tuple{UInt128}}})
      @ Base ./essentials.jl:720
   [11] (::Distributed.var"#106#108"{Distributed.CallMsg{:call_fetch}})()
      @ Distributed /cache/build/amdci7-5/julialang/julia-master/usr/share/julia/stdlib/v1.8/Distributed/src/process_messages.jl:278
   [12] run_work_thunk(thunk::Distributed.var"#106#108"{Distributed.CallMsg{:call_fetch}}, print_error::Bool)
      @ Distributed /cache/build/amdci7-5/julialang/julia-master/usr/share/julia/stdlib/v1.8/Distributed/src/process_messages.jl:63
   [13] macro expansion
      @ /cache/build/amdci7-5/julialang/julia-master/usr/share/julia/stdlib/v1.8/Distributed/src/process_messages.jl:278 [inlined]
   [14] (::Distributed.var"#105#107"{Distributed.CallMsg{:call_fetch}, Distributed.MsgHeader, Sockets.TCPSocket})()
      @ Distributed ./task.jl:411
  in expression starting at /cache/build/amdci7-2/julialang/julia-master/julia-19134592a9/share/julia/test/file.jl:532

From https://buildkite.com/julialang/julia-master/builds/4398#4f9c9e09-f7da-4e0e-9f66-f8f4fa046843, which ran on amdci7.2.

The jobs that run on amdci5 don't seem to have the same error.

@DilumAluthge
Copy link
Member

DilumAluthge commented Oct 6, 2021 •
edited
Loading

This looks similar to the problem we were trying to fix in #41682.

I'm not sure why it would be machine-dependent.

Also cc: @Keno since you helped out with this previously.

@DilumAluthge
Copy link
Member

Seeing the same thing on amdci8, e.g. https://buildkite.com/julialang/julia-master/builds/4399#c580a08b-d431-4b17-adbb-1cea1a61a79a

@vchuravy vchuravy merged commit 538541a into master Oct 6, 2021
@vchuravy vchuravy deleted the vc/bump_llvm branch October 6, 2021 11:33
vchuravy added a commit that referenced this pull request Oct 6, 2021
@vchuravy
Revert "[LLVM] bump to 12.0.1-4 (#42512)"
61252c1 
This reverts commit 538541a.
@vchuravy vchuravy mentioned this pull request Oct 6, 2021
Closed
@vchuravy vchuravy added upstream The issue is with an upstream dependency, e.g. LLVM and removed backport 1.7 labels Oct 6, 2021
This was referenced Oct 7, 2021
Merged
Merged
vchuravy added a commit that referenced this pull request Oct 7, 2021
@vchuravy
[LLVM] bump to 12.0.1-4 (#42512)
b86f78b 
* [LLVM] bump to 12.0.1-4

* Clang is now installed into usr/tools

(cherry picked from commit 538541a)
vchuravy added a commit that referenced this pull request Oct 9, 2021
@vchuravy
[LLVM] bump to 12.0.1-4 (#42512)
85430ac 
* [LLVM] bump to 12.0.1-4

* Clang is now installed into usr/tools

(cherry picked from commit 538541a)
vchuravy added a commit that referenced this pull request Oct 13, 2021
@vchuravy
[LLVM] bump to 12.0.1-4 (#42512)
c5ff713 
* [LLVM] bump to 12.0.1-4

* Clang is now installed into usr/tools

(cherry picked from commit 538541a)
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
@vchuravy @LilithHafner
[LLVM] bump to 12.0.1-4 (JuliaLang#42512)
a6bcd46 
* [LLVM] bump to 12.0.1-4

* Clang is now installed into usr/tools
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
@vchuravy @LilithHafner
[LLVM] bump to 12.0.1-4 (JuliaLang#42512)
2388613 
* [LLVM] bump to 12.0.1-4

* Clang is now installed into usr/tools
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tkf tkf tkf left review comments

@vtjnash vtjnash Awaiting requested review from vtjnash

Assignees
No one assigned
Labels
upstream The issue is with an upstream dependency, e.g. LLVM
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vchuravy @DilumAluthge @tkf