Skip to content

Conversation

@vchuravy
Copy link
Member

@vchuravy vchuravy commented Jul 1, 2021

Originally found by @pchintalapudi

@vchuravy vchuravy added the compiler:codegen Generation of LLVM IR and native code label Jul 1, 2021
@vchuravy vchuravy requested a review from vtjnash July 1, 2021 20:13
@vtjnash
Copy link
Member

vtjnash commented Jul 1, 2021

Wow, that seemed to cause a surprising amount of CI chaos

@vtjnash
Copy link
Member

vtjnash commented Jul 22, 2021

Bisected this to https://reviews.llvm.org/D106613

@vchuravy vchuravy added this to the 1.7 milestone Jul 23, 2021
@vchuravy
Copy link
Member Author

Adding this to the milestone so that we pick up https://reviews.llvm.org/D106613 at least, even if we don't backport this commit.

@JeffBezanson
Copy link
Member

The LLVM patch is probably not release-blocking, since AFAICT we don't actually hit the bug in 1.7. Once it's settled will be good to backport though.

@JeffBezanson JeffBezanson removed this from the 1.7 milestone Aug 3, 2021
@vtjnash
Copy link
Member

vtjnash commented Aug 25, 2021

FWIW, it seems like we are potentially waiting for the next LLVM version, which disables the offending logic (via a better cost model), since I don't know how to fix it.

@vchuravy
Copy link
Member Author

vchuravy commented Oct 8, 2021

@nanosoldier runbenchmarks(ALL, vs=":master")

@vchuravy
Copy link
Member Author

vchuravy commented Oct 8, 2021

@nanosoldier runtests(ALL, vs = ":master")

@nanosoldier
Copy link
Collaborator

Something went wrong when running your job:

NanosoldierError: error when preparing/pushing to report repo: failed process: Process(setenv(`git push`; dir="/run/media/system/data/nanosoldier/workdir/NanosoldierReports"), ProcessExited(1)) [1]

Unfortunately, the logs could not be uploaded.
cc @christopher-dG

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here.

@vchuravy
Copy link
Member Author

vchuravy commented Oct 9, 2021

PkgEval failed due to the network outage yesterday...

@vchuravy
Copy link
Member Author

vchuravy commented Oct 9, 2021

@nanosoldier runtests(ALL, vs = ":master")

@nanosoldier
Copy link
Collaborator

Your package evaluation job has completed - possible new issues were detected. A full report can be found here.

@vchuravy vchuravy merged commit 62daed1 into master Oct 10, 2021
@vchuravy vchuravy deleted the vc/wb branch October 10, 2021 20:07
@vtjnash
Copy link
Member

vtjnash commented Oct 12, 2021

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compiler:codegen Generation of LLVM IR and native code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants