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

Reply via email to