fix: Update custom_functions xsimd library#26320
fix: Update custom_functions xsimd library#26320mohsaka wants to merge 1 commit intoprestodb:masterfrom
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideRefactors CMake integration for xsimd in the custom_functions library by replacing direct target references with the ${XSIMD} variable and updating include/link settings for compatibility. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
b77e6db to
66911b7
Compare
aditi-pandit
left a comment
There was a problem hiding this comment.
Thanks @mohsaka.
Does this help with the compilation issues we saw with custom functions earlier ?
Hi @aditi-pandit There are a few issues that came up afterwards. 1 was that BUNDLED was required. @minhancao and @pramodsatya both pointed this out to me. @minhancao Has a different fix where we use We are not sure which one is better to use or which one is more universal. I'm also trying to fix the issue with the const expr since I'm waiting for reviews for TVF. |
c0c2508 to
18df56d
Compare
Co-authored-by: minhancao <minhan.duc.cao@gmail.com> Co-authored-by: czentgr <82243552+czentgr@users.noreply.github.com>
18df56d to
066c778
Compare
|
Replaced by #26324 |
Description
Update xsimd usage in custom_functions library.
Motivation and Context
Impact
Test Plan
Contributor checklist
Release Notes
Please follow release notes guidelines and fill in the release notes below.
If release note is NOT required, use: