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

Drop deprecated / unsupported setups for Halide 20 #8508

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

alexreinking
Copy link
Member

@alexreinking alexreinking commented Dec 16, 2024

This PR drops support for LLVM 17 according to our support policy. It also drops the old names for CMake variables that were renamed and deprecated between Halide 18 and 19.

We also take this opportunity to bump clang-tidy from version 17 to 19. A subsequent PR will do the same for clang-format (so we have a consistent set of tools).

@abadams
Copy link
Member

abadams commented Dec 16, 2024

Looks like there are some new helpful clang-tidy failures. Feel free to fix in this PR or a follow-up if you feel that would be cleaner. I'm also happy to fix in a follow up if you're slammed today.

@alexreinking
Copy link
Member Author

Looks like there are some new helpful clang-tidy failures. Feel free to fix in this PR or a follow-up if you feel that would be cleaner. I'm also happy to fix in a follow up if you're slammed today.

Sounds like @steven-johnson is eager to fix in a follow up PR

@steven-johnson
Copy link
Contributor

Don't analyze generated code in clang-tidy builds

If we are generating untidy code, would be nice to open an issue to fix that someday

@alexreinking
Copy link
Member Author

If we are generating untidy code, would be nice to open an issue to fix that someday

Lots of warnings where it wants us to write auto * where we have auto

@alexreinking
Copy link
Member Author

These failures are quite strange. Will need to investigate.

@abadams
Copy link
Member

abadams commented Dec 17, 2024

FWIW I think I'm seeing the same failures here: #8518

@alexreinking
Copy link
Member Author

FWIW I think I'm seeing the same failures here: #8518

In that case, looks like an LLVM 20 injection?

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

Successfully merging this pull request may close these issues.

3 participants