[PATCH] D122475: [X86] Refactor X86ScalarSSEf16/32/64 with hasFP16/SSE1/SSE2. NFCI

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86FastISel.cpp:534
   case MVT::f64:
-    if (X86ScalarSSEf32) {
       if (IsNonTemporal && HasSSE4A)
----------------
pengfei wrote:
> This looks a bug to me, but no test is affected by this change.
Agreed - its probably worth adding additional X87/SSE1 only test runs to fast-isel-nontemporal.ll in this patch?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122475/new/

https://reviews.llvm.org/D122475