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

Backports for 1.9.0 #49302

Merged
merged 31 commits into from
Apr 25, 2023
Merged

Backports for 1.9.0 #49302

merged 31 commits into from
Apr 25, 2023

Conversation

KristofferC
Copy link
Sponsor Member

@KristofferC KristofferC commented Apr 9, 2023

Backported PRs:

Need manual backport:

Non-merged PRs with backport label:

jishnub and others added 15 commits April 9, 2023 17:24
same for symlinks, adjust docs accordingly and clarify that it refers to the mtime

(cherry picked from commit 93df7e2)
Changes [Distributed._default_worker_pool](https://github.com/JuliaLang/julia/blob/5f5d2040511b42ba74bd7529a0eac9cf817ad496/stdlib/Distributed/src/workerpool.jl#L242) to hold an `AbstractWorkerPool` instead of `WorkerPool`. With this, alternate implementations can be plugged in as the default pool. Helps in cases where a cluster is always meant to use a certain custom pool. Lower level calls can then work without having to pass a custom pool reference with every call.

(cherry picked from commit def2dda)
We generally hit the runtime in pretty specific places when allocations
look funky (because they are missing a typevar bound, so inference is
not too willing to deal with it). Try to throw an error in those cases
before those can get allocated and cause problems later from being
non-concrete objects.

Fix #49203

(cherry picked from commit a1013e7)
Disambiguate scope of the `wp` variable inside `try`-`catch` in distributed tests by qualifying it and suppress the warning reported in the tests.

fixes #49289

(cherry picked from commit 4ccaa1b)
This release of libblastrampoline contains two new features: ILP64 Accelerate support, and LAPACK_jll support.

(cherry picked from commit 4ced71a)
This changes the behavior of `select_platform()` to resolve ambiguities
using a two-step process.  Previously, we would simply sort the
resultant triplets and return the last one (so as to provide asemi-
arbitrary stability, and also to prefer higher version numbers over
lower version numbers in tags).  However, with the proliferation of tags
(and especially the new `sanitize=memory` tags) we need a way to exclude
these "extra" tags when our `HostPlatform()` does not have them.

This new matching algorithm excludes candidates from matching with the
platform if there are other candidates with fewer total tags.  This
results in a "simplest match first" behavior, which better represents
the intent of tags in the first place.

(cherry picked from commit 99938fc)
Subtype: skip more identical check for `X::Tuple <: Y::Tuple`
(cherry picked from commit e3dfcc3)
Fix imaging value for generating sysimages for multiple targets

(cherry picked from commit fc4b079)
@KristofferC
Copy link
Sponsor Member Author

@nanosoldier runtests(ALL, vs = "@7b395153e80672f8cdb18f51dd653a85e28b2070", configuration = (buildflags=["LLVM_ASSERTIONS=1", "FORCE_ASSERTIONS=1"],), vs_configuration=(buildflags=["LLVM_ASSERTIONS=1", "FORCE_ASSERTIONS=1"],))

@nanosoldier
Copy link
Collaborator

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

N5N3 and others added 2 commits April 14, 2023 07:57
Co-authored-by: Jeff Bezanson <jeff.bezanson@gmail.com>
Co-authored-by: Kristoffer Carlsson <kcarlsson89@gmail.com>
(cherry picked from commit 99c0dad)
@KristofferC
Copy link
Sponsor Member Author

@nanosoldier runtests(["OnlineStats", "Recommenders", "RCall", "DocumenterCitations", "SharedMATLABEngine", "StateSpaceSets", "MCMCChains", "Reactive", "BifurcationInference", "Plots", "Mitosis", "Expronicon", "Arrow", "Unitful", "GeometricIntegrators", "CrystalInfoFramework", "PlasmoPlots", "AesKeywrapNettle", "ApproximateGPs", "RAPIDS", "VortexLattice", "AdmittanceModels", "MINLPTests", "PlasmaEquilibriumToolkit", "SPICE", "RadiationSpectra", "KiteModels", "DimensionalData", "AnyMOD"], vs = ":release-1.9", configuration = (buildflags = ["LLVM_ASSERTIONS=1", "FORCE_ASSERTIONS=1"],), vs_configuration = (buildflags = ["LLVM_ASSERTIONS=1", "FORCE_ASSERTIONS=1"],))

@nanosoldier
Copy link
Collaborator

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

@KristofferC
Copy link
Sponsor Member Author

The instruction dominates all uses errors in PkgEval are the same as #48918 and got "introduced" in fe72f81 when they actually started getting checked.

@timholy
Copy link
Sponsor Member

timholy commented Apr 16, 2023

Is skipping verification a viable "solution" on 1.9?

@KristofferC
Copy link
Sponsor Member Author

KristofferC commented Apr 16, 2023

Not really (it only shows up when building with asserts anyway) , but we might just have to release without that fixed (it's really a LLVM bug from what I understand).

DilumAluthgeBot and others added 2 commits April 17, 2023 08:53
Co-authored-by: Dilum Aluthge <dilum@aluthge.com>
(cherry picked from commit 74a52cf)
N5N3 and others added 5 commits April 23, 2023 01:12
* Improve `simple_meet` resolution.

* Fix for many-to-one cases.

* Test disjoint via `jl_has_empty_intersection`
Co-authored-by: Sukera <Seelengrab@users.noreply.github.com>
(cherry picked from commit 4beea98)
* Add missing entry to invalidation log

Addresses timholy/SnoopCompile.jl#357 (comment)
using the observation in the following comment.

Co-authored-by: Jameson Nash <vtjnash@gmail.com>

* Fix indentation

---------

Co-authored-by: Jameson Nash <vtjnash@gmail.com>
(cherry picked from commit 23a5b04)
@KristofferC
Copy link
Sponsor Member Author

@nanosoldier runtests(ALL, vs = "@7b395153e80672f8cdb18f51dd653a85e28b2070", configuration = (buildflags=["LLVM_ASSERTIONS=1", "FORCE_ASSERTIONS=1"],), vs_configuration=(buildflags=["LLVM_ASSERTIONS=1", "FORCE_ASSERTIONS=1"],))

@nanosoldier
Copy link
Collaborator

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

N5N3 and others added 4 commits April 24, 2023 16:32
1. remove duplicated disjoint check.
2. add a fast path for all disjoint case.

(cherry picked from commit f2560c2)
This is the same as #45765 where
we use `jl_error()` too early to get backtraces, but too late to fail
the supposed guard `if` statement that should prevent us from trying to
take a backtrace.  X-ref: #45847

(cherry picked from commit fa21589)
[release-1.9] Avoid usage of `jl_error()` in `check_cmdline()`
@KristofferC
Copy link
Sponsor Member Author

I'm gonna merge this since it is green to get a new "checkpoint". There are some other small thing that should go in but I'll do that in a different PR.

@KristofferC KristofferC merged commit 3d195c0 into release-1.9 Apr 25, 2023
@KristofferC KristofferC deleted the backports-release-1.9 branch April 25, 2023 14:28
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 this pull request may close these issues.

None yet