diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2017-04-25 12:02:09 -0300 |
---|---|---|
committer | Thiago Macieira <thiago.macieira@intel.com> | 2017-11-22 17:58:21 +0000 |
commit | d813c66bfcfac1837814ec4d174d0389172f0d4c (patch) | |
tree | 0fb335520ed92297e7db89dbe8b4b95826a1ac90 /src/concurrent | |
parent | b71b7461b0b95bbf02c215019381b19e4070e07c (diff) |
Fix the build when AVX2 is enabled but __F16C__ isn't defined
If -mavx2 is used, __AVX2__ is defined, which enables the F16C code
after commit 280e321e52fd4e86545f3f0d4bd4e047786a897e, but that was
wrong since we aren't allowed to use the F16C intrinsics with either
Clang or GCC (we can only do that with GCC 4.9 and Clang 4.8, and only
with an __attribute__ decoration).
With ICC and MSVC, we are allowed to use the intrinsics, but the
#include was missing.
[ChangeLog][QtCore] Fixed a compilation issue with qfloat16 if AVX2
support is enabled in the compiler. Since all processors that support
AVX2 also support F16C, for GCC and Clang it is recommended to either
add -mf16c to your build or to use the corresponding -march= switch.
Task-number: QTBUG-64529
Change-Id: I84e363d735b443cb9beefffd14b8ac1fd4baa978
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
Diffstat (limited to 'src/concurrent')
0 files changed, 0 insertions, 0 deletions