这是indexloc提供的服务,不要输入任何密码
Skip to content

Update search path for highway version #3644

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

Closed
wants to merge 1 commit into from

Conversation

donny-dont
Copy link
Contributor

In version 1.2.0 of highway the defines used to determine the version moved from highway.h to base.h. See if the version is in base.h and if not look in highway.h.

Description

Modified the FindHWY.cmake module to search for version numbers in base.h then highway.h. The change in highway occurred at google/highway@1bf05d6

Pull Request Checklist

  • CLA Signed: Have you signed the Contributor License Agreement (individual or corporate, as appropriate)? Only contributions from signed contributors can be accepted.
  • Authors: Have you considered adding your name to the AUTHORS file?
  • Code Style: Have you ensured your code adheres to the project's coding style guidelines? You can use ./ci.sh lint for automatic code formatting.

Please review the full contributing guidelines for more details.

In version 1.2.0 of highway the defines used to determine the version moved from `highway.h` to `base.h`. See if the version is in `base.h` and if not look in `highway.h`.
@mo271
Copy link
Member

mo271 commented Jun 14, 2024

thanks! duplicate of #3642

@donny-dont
Copy link
Contributor Author

Sorry didn't see that @mo271 ! If you want to steal anything from this feel free. Think the changes are a bit simpler in this one.

@mo271
Copy link
Member

mo271 commented Jun 18, 2024

closed in favour of #3642, but many thanks nonetheless!

@mo271 mo271 closed this Jun 18, 2024
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.

2 participants