AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Build fails with clang-cl: SVML _mm_pow_ps gated on _MSC_VER alone

Open Beginner friendly
#2,329 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.