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
refactor: Remove Span operator==, Use std::ranges::equal #29071
base: master
Are you sure you want to change the base?
Conversation
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. Code CoverageFor detailed information about the code coverage, see the test coverage report. ReviewsSee the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
fabd742
to
1ff0c38
Compare
1ff0c38
to
56c9e51
Compare
Looks like this also fails on macOS on GHA. Does anyone know what version(s) of clang can be expected to be available on macOS normally? |
Pretty sure that job should be rerun, because it's failing in the brew install stage. |
GIven our minimum supported macOS (11.x), you could run anything from Xcode 12.5+ onwards, which is roughly LLVM/Clang 12+. Although I'd expect most users compiling from source on macOS, would be using a much more recent Xcode, so likely Clang ~14+ at least. Also, anyone should be able to install and run the latest LLVM/Clang via brew. |
56c9e51
to
adaeeb2
Compare
adaeeb2
to
c45d3b7
Compare
c45d3b7
to
05804fb
Compare
Shouldn't our |
Yes, this change is required to move from |
Concept ACK |
05804fb
to
f5ece54
Compare
🐙 This pull request conflicts with the target branch and needs rebase. |
std::span
removed the comparison operators, so it makes sense to remove them for theSpan
"backport" as well. Usingstd::ranges::equal
also has the benefit that someSpan
temporary constructions can now be dropped.This is required to move from
Span
towardstd::span
.