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

ARROW-10321: [C++] Use check_cxx_source_compiles for AVX512 detect in compiler #8478

Closed
wants to merge 1 commit into from

Conversation

frankdjx
Copy link
Contributor

Also build the SIMD files as ARROW_RUNTIME_SIMD_LEVEL.

Signed-off-by: Frank Du frank.du@intel.com

@github-actions
Copy link

… compiler

Also build the SIMD file as ARROW_RUNTIME_SIMD_LEVEL.

Signed-off-by: Frank Du <frank.du@intel.com>
@frankdjx
Copy link
Contributor Author

Seems CI not related, push again

Copy link
Contributor

@nealrichardson nealrichardson left a comment

Choose a reason for hiding this comment

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

LGTM. I applied this patch on autobrew/homebrew-core#31 and it did the right things there.

@frankdjx frankdjx deleted the avx512_runtime_level_build branch October 19, 2020 01:07
kszucs pushed a commit that referenced this pull request Oct 19, 2020
… compiler

Also build the SIMD files as ARROW_RUNTIME_SIMD_LEVEL.

Signed-off-by: Frank Du <frank.du@intel.com>

Closes #8478 from jianxind/avx512_runtime_level_build

Authored-by: Frank Du <frank.du@intel.com>
Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
GeorgeAp pushed a commit to sirensolutions/arrow that referenced this pull request Jun 7, 2021
… compiler

Also build the SIMD files as ARROW_RUNTIME_SIMD_LEVEL.

Signed-off-by: Frank Du <frank.du@intel.com>

Closes apache#8478 from jianxind/avx512_runtime_level_build

Authored-by: Frank Du <frank.du@intel.com>
Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
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.

None yet

2 participants