AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Build fails with clang-cl: SVML _mm_pow_ps gated on _MSC_VER alone
- Dominant language
- C++
- Stars
- 2.1k
- Forks
- 503
- PR merge metrics
- No merged PRs in 30d
Description
`FixedFunctionOpCPU.cpp` gates the SVML `_mm_pow_ps` path on `_MSC_VER >= 1920` alone. clang-cl
defines `_MSC_VER` (1951 for clang 22.1.8) but ships no SVML, so the build breaks whenever
`OCIO_USE_AVX` is on.
Minimal:
```c++
#include
__m128 f(__m128 x, __m128 e) { return _mm_pow_ps(x, e); }
```
```
> clang-cl -c -arch:AVX t.cpp
t.cpp(2,39): error: use of undeclared identifier '_mm_pow_ps'
```
Excluding the compiler in the gate is enough, and clang-cl then takes the existing `ssePower()` path
like any other non-MSVC compiler:
```diff
-#if (_MSC_VER >= 1920) && (OCIO_USE_AVX)
+#if (_MSC_VER >= 1920) && (OCIO_USE_AVX) && !defined(__clang__)
```
main has 2 such sites; 2.5.2 has 4, the extra two being the dispatch arms in
`GetFixedFunctionCPURenderer`.
clang 22.1.8, x86_64-pc-windows-msvc. Reproduced against 2.5.2 and main.
Contributor guide
Research direction
Start in FixedFunctionOpCPU.cpp and locate the two SVML _mm_pow_ps preprocessor gates using _MSC_VER and OCIO_USE_AVX. Reproduce with clang-cl -c -arch:AVX, then verify the clang-cl build uses the existing ssePower() path and succeeds with OCIO_USE_AVX enabled.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 82/100