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

Increasing minimum required version of Clang to 17 #4529

Closed
randombit opened this issue Jan 4, 2025 · 1 comment
Closed

Increasing minimum required version of Clang to 17 #4529

randombit opened this issue Jan 4, 2025 · 1 comment

Comments

@randombit
Copy link
Owner

randombit commented Jan 4, 2025

Of our minimum supported compiler versions, LLVM Clang 14 is the most problematic. It's also I think our least important; it's only the default system compiler on BSDs.

[I forgot: Android NDK is also Clang, namely Clang 17]

Bumping minimum to Clang 17 gets us

  • std::source_location
  • P0634R3 is fixed
  • Fixing several bugs in consteval I ran into over the summer
  • Apparently some amount of ranges works (I haven't looked into this)
  • std::format (but we'd still be blocked by GCC, since it wasn't added there until GCC 13)

#4528 contains proposed changes

The plan would be in 3.7.0 to have a configure time warning if an older Clang is detected, as well as a prominent mention in the release notes. Followed by merging #4528 for 3.8.0

Leave a comment here if removing support for Clang 14 through 16 would be an issue for you.

@randombit randombit changed the title Increasing minimum required version of Clang to 18 Increasing minimum required version of Clang to 17 Jan 4, 2025
@Ashishbsharma
Copy link

Hi @randombit, I’d like to contribute to this issue by suggesting conditional fallbacks for features like std::source_location to ease the transition for users of older Clang versions while moving towards full adoption of Clang 17

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

No branches or pull requests

2 participants