rjmccall added inline comments.
================ Comment at: clang/lib/Basic/Targets/X86.cpp:372 + // Turn on _float16 for x86 (feature sse+) + HasFloat16 = SSELevel >= SSE2; ---------------- pengfei wrote: > rjmccall wrote: > > Out of curiosity, why SSE2? SSE2 adds double-precision, but we just need > > single-precision, which is in the original SSE. > > > > It doesn't much matter anymore, of course, since both are ubiquitous. > https://lists.llvm.org/pipermail/llvm-dev/2021-July/151618.html Ah, makes sense. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D114099/new/ https://reviews.llvm.org/D114099 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits