Skip to content

Conversation

DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Aug 30, 2025

This reverts commit 3b04664 (which backports #58837 to 1.11.x).

For more details, see #59336 (comment) and #59336 (comment).

Targets backports-release-1.11 (#59336).

@DilumAluthge DilumAluthge changed the title Revert "fix null comparisons for non-standard address spaces (#58837)" [backports-release-1.11]: Revert "fix null comparisons for non-standard address spaces (#58837)" Aug 30, 2025
@DilumAluthge DilumAluthge added the release Release management and versioning. label Aug 30, 2025
@DilumAluthge
Copy link
Member Author

cc: @simeonschaub @gbaraldi @vtjnash

@simeonschaub
Copy link
Member

simeonschaub commented Aug 30, 2025

Can we backport #59259 instead? Can't test it right now, but it should fix these issues

@DilumAluthge
Copy link
Member Author

Can we backport #59259 instead? Can't test it right now, but that should fix these issues

Sure, let's try that out. I'll make a separate PR first, and if that works, then we can add the backport label so that Kristoffer's script does the backport.

@DilumAluthge
Copy link
Member Author

Can we backport #59259 instead? Can't test it right now, but that should fix these issues

Sure, let's try that out. I'll make a separate PR first, and if that works, then we can add the backport label so that Kristoffer's script does the backport.

#59446

@DilumAluthge
Copy link
Member Author

DilumAluthge commented Aug 31, 2025

#59446 doesn't seem to be sufficient. Can we proceed with this revert for now, and then we can try to re-land later?

@DilumAluthge DilumAluthge marked this pull request as ready for review August 31, 2025 01:00
Copy link
Member

@simeonschaub simeonschaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems ok for now, seeing as the AMDGPU issues were only observed on 1.12+

@DilumAluthge
Copy link
Member Author

Windows failures are errors when cleaning up temp directories.

llvmpasses is a different failure that I'm working on bisecting.

@DilumAluthge DilumAluthge merged commit ddcbe4d into backports-release-1.11 Aug 31, 2025
9 of 12 checks passed
@DilumAluthge DilumAluthge deleted the dpa/revert-3b04664---backports-release-1.11 branch August 31, 2025 14:04
gbaraldi added a commit that referenced this pull request Sep 9, 2025
…ion for null ptr comparisons") to Julia 1.11.x (#59446)

Backports #59259 to 1.11 (cherry picked from commit
920df7a).

For more details, see
#59445 (comment).

Targets `backports-release-1.11` (#59336).

---------

Co-authored-by: Gabriel Baraldi <[email protected]>
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.

2 participants