phosek added a comment.
In D114782#3184317 <https://reviews.llvm.org/D114782#3184317>, @pengfei wrote:
> Hi @phosek , thanks for reporting it.
> I would consider it as a misuse of the header file when SSE is not usable,
> especially we now have the specific header "x86gprintrin.h" for such
> scenarios. Can you help to try if changing to "x86gprintrin.h" works?
It's a normal practice to compile with `-mno-{sse,avx}` and then use SSE/AVX
intrinsics explicitly inside runtime cpuid conditionals. The function calling
the intrinsic might be marked with `__attribute__((target("avx")))`, but the
source file's ambient `-mo-{sse,avx}` will be in force when `#include
<immintrin.h>` is done. This change breaks that practice which is likely going
to affect existing codebases.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114782/new/
https://reviews.llvm.org/D114782
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
