Skip to content

remove ABSL_HAVE_PTHREAD_CPU_NUMBER_NP#2067

Closed
h-vetinari wants to merge 1 commit into
abseil:masterfrom
h-vetinari:var_rm2
Closed

remove ABSL_HAVE_PTHREAD_CPU_NUMBER_NP#2067
h-vetinari wants to merge 1 commit into
abseil:masterfrom
h-vetinari:var_rm2

Conversation

@h-vetinari
Copy link
Copy Markdown
Contributor

The conditions are always satisfied based on the macOS lower bound that has long moved past the versions being considered here.

// Checks whether pthread_cpu_number_np is available.
#ifdef ABSL_HAVE_PTHREAD_CPU_NUMBER_NP
#error ABSL_HAVE_PTHREAD_CPU_NUMBER_NP cannot be directly set
#elif defined(__APPLE__) && defined(__has_include) && \
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Don't we still need the __APPLE__ test?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Depends a bit on the pthread setup. I can restore it, but then ABSL_HAVE_PTHREAD_CPU_NUMBER_NP won't be removable

@copybara-service copybara-service Bot closed this in bbca5fe Jun 3, 2026
copybara-service Bot pushed a commit that referenced this pull request Jun 3, 2026
Imported from GitHub PR #2067

The conditions are always satisfied based on the macOS [lower bound](https://github.com/google/oss-policies-info/blob/main/foundational-cxx-support-matrix.md) that has long moved past the versions being considered here.
Merge 49ed819 into e3555a1

Merging this change closes #2067

PiperOrigin-RevId: 926238049
Change-Id: Ib90a33dd1d95eb3acea25a5c09bf367f2b64cfaa
@h-vetinari h-vetinari deleted the var_rm2 branch June 3, 2026 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants