Builder ffmpegsos-solaris10-i386 Build #14273
Results:
Failed
SourceStamp:
| Project | ffmpeg |
| Repository | https://git.ffmpeg.org/ffmpeg.git |
| Branch | master |
| Revision | 17e6ed9dc0106fbf837bda5d675fae22360e6c3e |
| Changes | 1 change |
BuildSlave:
unstable10xReason:
The SingleBranchScheduler scheduler named 'schedule-ffmpegsos-solaris10-i386' triggered this build
Steps and Logfiles:
Build Properties:
| Name | Value | Source |
|---|---|---|
| branch | master | Build |
| builddir | /export/home/buildbot/slave/ffmpegsos-solaris10-i386 | slave |
| buildername | ffmpegsos-solaris10-i386 | Builder |
| buildnumber | 14273 | Build |
| codebase | Build | |
| project | ffmpeg | Build |
| repository | https://git.ffmpeg.org/ffmpeg.git | Build |
| revision | 17e6ed9dc0106fbf837bda5d675fae22360e6c3e | Build |
| scheduler | schedule-ffmpegsos-solaris10-i386 | Scheduler |
| slavename | unstable10x | BuildSlave |
| workdir | /export/home/buildbot/slave/ffmpegsos-solaris10-i386 | slave (deprecated) |
Forced Build Properties:
| Name | Label | Value |
|---|
Responsible Users:
- Niklas Haasgit@haasn.dev
Timing:
| Start | Tue Sep 15 13:36:53 2026 |
| End | Tue Sep 15 13:36:59 2026 |
| Elapsed | 6 secs |
All Changes:
:
Change #281873
Category ffmpeg Changed by Niklas Haas <git@haasn.dev> Changed at Tue 15 Sep 2026 15:12:45 Repository https://git.ffmpeg.org/ffmpeg.git Project ffmpeg Branch master Revision 17e6ed9dc0106fbf837bda5d675fae22360e6c3e Comments
avfilter/vf_colordetectdsp: rewrite C implementations as macros In the process, also add the range assumption and cast to the detect_alpha intermediates, which seems to help codegen: checkasm: - CPU: AMD Ryzen 9 9950X3D 16-Core Processor (00B40F40) - Timing source: x86 (rdtsc) - Bench duration: 100000 µs per function (442374016 cycles) - Random seed: 2809724650 Benchmark results: (old) name cycles (vs ref) detect_alpha_8_full_c: 7582.7 detect_alpha_8_full_off_c: 7587.4 detect_alpha_8_limited_c: 13306.7 detect_alpha_16_full_c: 3482.5 detect_alpha_16_full_off_c: 3480.4 detect_alpha_16_limited_c: 8055.1 detect_range_8_c: 1271.4 detect_range_16_c: 1213.0 Benchmark results: (new) name cycles (vs ref) detect_alpha_8_full_c: 4716.6 detect_alpha_8_full_off_c: 4715.4 detect_alpha_8_limited_c: 7943.2 detect_alpha_16_full_c: 2920.2 detect_alpha_16_full_off_c: 2918.9 detect_alpha_16_limited_c: 9952.0 detect_range_8_c: 1253.1 detect_range_16_c: 1194.8 The ~20% slowdown for detect_alpha_16_limited_c is because the previous version of the code short-circuited this condition, which is faster when the compiler is not able to vectorize the loop body and falls back to a pure scalar loop. However, I don't think optimizing around this is necessary as it's a pure compiler optimization question. In particular, it actually _prevents_ better compilers from being able to vectorize this in the future. (As an example, doing the same short-circuiting for 8-bit would be a 2x regression on my end) People running into performance issues should just use the SIMD versions. Sponsored-by: nxtedition AB Signed-off-by: Niklas Haas <git@haasn.dev>
Changed files
- libavfilter/vf_colordetectdsp.h