265400@main updated the CMake list of LibWebRTC sources after update to M115 (265320@main). In the sources update, a new block was added to include specialized files that make use of AVX2 extensions if the host is X64. This condition is too broad, since not all X64 computers have support for AVX2 extensions (rare nowadays, but it may happen). Another issue we found out was that since the AVX2 files are built with `-march=native` the host may emit AVX512 instructions if the host supports that extension. This may cause compatibility issues if a binary is built in a host with AVX512 support and it is executed in a host without AVX512 suport (the result will be a crash due to 'invalid instruction'). The build flag `m=no-avx512f` prevents emitting AVX512 instructions. https://stackoverflow.com/questions/60815316/disabling-all-avx512-extensions
Pull request: https://github.com/webkit/webkit/pull/15432
Committed 265646@main (d95cad7579b5): <https://commits.webkit.org/265646@main> Reviewed commits have been landed. Closing PR #15432 and removing active labels.
<rdar://problem/111556402>
After chatting with Carlos Lopez in a private conversation, we concluded that the LibWebRTC's AVX2 files should be included always, regardless of whether the host machine that builds WebKit has support for it or not. The flags `-march=machine -mno-avx512f` added to CMAKE_CXX_FLAGS should be removed because they affect all files. Instead, these flags should only apply to the set of AVX2 files. To enable compilation with AVX2 extension is enough with adding '-mavx2'. This is how Chromium does it: https://webrtc.googlesource.com/src/+/refs/heads/main/modules/audio_processing/aec3/BUILD.gn#269 CMake macro `set_source_files_properties` can be used to apply build flags to a set of files. https://cmake.org/cmake/help/latest/prop_sf/COMPILE_OPTIONS.html#prop_sf:COMPILE_OPTIONS
Pull request: https://github.com/webkit/webkit/pull/15527
Committed 265754@main (0de2717c0029): <https://commits.webkit.org/265754@main> Reviewed commits have been landed. Closing PR #15527 and removing active labels.
Re-opening for pull request https://github.com/webkit/webkit/pull/15586
Committed 265790@main (48d7dce49219): <https://commits.webkit.org/265790@main> Reviewed commits have been landed. Closing PR #15586 and removing active labels.
Re-opening for pull request https://github.com/webkit/webkit/pull/15622
Committed 265832@main (aff1c9e1b48a): <https://commits.webkit.org/265832@main> Reviewed commits have been landed. Closing PR #15622 and removing active labels.