Vincent Rabaud
Vincent Rabaud
Thx ! Can you please remove the debug info (iostream include and commented out cout). Just curious, does that also fix https://github.com/opencv/opencv_contrib/issues/3316 ?
Ok, it seems on windows, you use vc14 (VS 2015). And it did not implement some AVX2 intrinsics ... Some libraries implement their own versions for older VS, e.g. https://aomedia.googlesource.com/aom/+/b2f80f721fe87547e3bae73be550e3d75ba478ad/aom_ports/msvc.h#57...
Can you please add the patch at https://chromium-review.googlesource.com/c/webm/libwebp/+/7233156 ? Thx
Was the build re-triggered ?
Sorry, I made a mistake. Please change: INLINE -> WEBP_INLINE
In my patch at https://chromium-review.googlesource.com/c/webm/libwebp/+/7233156, I had put INLINE instead of WEBP_INLINE in src/dsp/cpu.h
And I forgot the `#include `, cf patch at https://chromium-review.googlesource.com/c/webm/libwebp/+/7233156 . Sorry for wasting your time. If that does not work, I'll branch your PR myself and fix it till...
I am not sure about the new warning: ``` c:\build\precommit_windows64\4.x\opencv\3rdparty\libwebp\src\dsp\lossless_avx2.c(76): warning C4752: found Intel(R) Advanced Vector Extensions; consider using /arch:AVX [C:\build\precommit_windows64\build\3rdparty\libwebp\libwebp.vcxproj] c:\build\precommit_windows64\4.x\opencv\3rdparty\libwebp\src\dsp\lossless_avx2.c(178): warning C4752: found Intel(R) Advanced Vector Extensions; consider...
On here: https://pullrequest.opencv.org/buildbot/builders/precommit_windows64/builds/111602 Click on warnings: https://pullrequest.opencv.org/buildbot/builders/precommit_windows64/builds/111602/steps/compile%20release/logs/warnings%20%282%29 Why do we use VS 2015? Especially, when there are free versions that are more recent.