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

release-1.7: Backports for 1.7.2 #43667

Merged
merged 25 commits into from
Jan 28, 2022
Merged

Conversation

KristofferC
Copy link
Member

@KristofferC KristofferC commented Jan 5, 2022

aviatesk and others added 8 commits January 5, 2022 16:15
The last entry sentinel for Win64 SEH is `~0L` not NULL. Apparently
this doesn't cause issues on windows proper, but does crash wine.
Arguably if Windows doesn't have issues then we should just fix
this in wine, but since we control the source and nobody else
ever seems to have run into this, let's just fix it and save
the good Wine folks some headache. Fixes #43569.

(cherry picked from commit 722f9d4)
Beacuse `shm_open()` is a variadic function, if we don't declare it as
such, the kernel receives trash as the `permissions` value, which
occasionally results in errors when creating shared memory segments.
This did not happen on `x86_64` because the calling convention doesn't
change so much between variadic and non-vadiadic functions.

(cherry picked from commit ffed6b8)
@KristofferC KristofferC added the release Release management and versioning. label Jan 5, 2022
vchuravy and others added 17 commits January 7, 2022 11:16
Prevents loading of `libjulia-codegen` into `opt`.

Co-authored-by: Jameson Nash <vtjnash@gmail.com>
(cherry picked from commit 0402dfa)
Constructing the Regex touched in this commit can represent a
significant fraction (e.g. half or better) of the runtime of
the DateFormat method touched in this commit. To make this
DateFormat method more efficient, let's lift that Regex
construction out of that method body.

(cherry picked from commit 7fb5bb8)
See comment in diff for explanation. This fixes test/stacktraces.jl
on aarch64 macOS 12, and according to an OpenJDK issue where they
ran into the same problem, https://git.openjdk.java.net/jdk/pull/6193,
probably also x86_64 macOS 12.

(cherry picked from commit 7f27dea)
Co-authored-by: Shuhei Kadowaki <40514306+aviatesk@users.noreply.github.com>
(cherry picked from commit 4409c82)
Just explicitely check for `CodeInfo` objects and use an explicit
`return` in this case inside the `@generated` macro.

Co-authored-by: Jeff Bezanson <jeff.bezanson@gmail.com>
(cherry picked from commit 92c84bf)
Phi nodes are allowed to recursively depend on themselves, so
we need to keep track of which ones we have seen to avoid an
infinite recursion here. Apparently this doesn't usually come
up, but I did see if with tsan enabled.

(cherry picked from commit a9a1d80)
To avoid the compiler potentially picking the wrong size depending on standards versions.

(cherry picked from commit e8d1167)
Co-authored-by: Simeon Schaub <schaub@mit.edu>
(cherry picked from commit 8536522)
Apple reintroduced the old bug that we previously worked around in
fad04d3 with a similar patch to this.

This is needed anywhere that we may attempt to stop threads.

Fixes #43578

(cherry picked from commit 2939272)
…sue, since 12.1

Later, we should probably switch to using mach_exc_server generated from
`mig mach_exc.defs`.

(cherry picked from commit 267b124)
@KristofferC

This comment has been minimized.

@KristofferC

This comment has been minimized.

@KristofferC

This comment has been minimized.

@nanosoldier

This comment has been minimized.

@nanosoldier

This comment has been minimized.

@vtjnash

This comment has been minimized.

@KristofferC
Copy link
Member Author

@nanosoldier runbenchmarks(ALL, vs=":release-1.7")

@KristofferC
Copy link
Member Author

@nanosoldier runtests(ALL, vs = ":release-1.7")

@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="/nanosoldier/workdir/NanosoldierReports"), ProcessExited(1)) [1]

Unfortunately, the logs could not be uploaded.

@nanosoldier
Copy link
Collaborator

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

@KristofferC
Copy link
Member Author

@nanosoldier runtests(["ADNLPModels", "AutomotiveVisualization", "BioMASS", "Caesar", "CanopyLayers", "CompressedSensing", "DistributedArrays", "GadgetIO", "GaussianMixtureAlignment", "GraphNeuralNetworks", "IntervalConstraintProgramming", "JuMP", "LoopThrottle", "MadNLP", "MiniQhull", "NavAbilitySDK", "OceanRobots", "OptimalTransmissionRouting", "Probably", "RoME", "SLEEFPirates", "SpatialJackknife", "Spectra", "StaticKernels", "StochasticRounding", "SymmetryReduceBZ", "TensorKit", "TraitSimulation", "VoronoiGraph", "WavePropBase"], vs = ":release-1.7")

@nanosoldier
Copy link
Collaborator

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release Release management and versioning.
Projects
None yet
Development

Successfully merging this pull request may close these issues.