[ARROW-10321] [C++] Building AVX512 code when we should not

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/26311
• Assignee: Frank Du
• Reporter: Neal Richardson
• Created: 15/Oct/20 23:17
• Updated: 11/Jan/23 08:12
• Resolved: 16/Oct/20 17:47

Description

On https://github.com/autobrew/homebrew-core/pull/31, where we are packaging Arrow for an old macOS SDK version, we found what I believe are 2 different problems:

1. The check for AVX512 support was returning true when in fact the compiler did not support it
2. Even when we manually set the runtime SIMD level to less-than-AVX512, it was still trying to compile one of the AVX512 files, which failed. I added a patch that made that file conditional, but there's probably a proper cmake way to tell it not to compile that file at all

cc yibo apitrou

Issue Links

• links to: GitHub Pull Request #8478