update review comments

Author: mattarde

llvm/lib/Target/X86/GISel/X86LegalizerInfo.cpp

@@ -448,13 +448,9 @@ X86LegalizerInfo::X86LegalizerInfo(const X86Subtarget &STI,
 
   // fp comparison
   getActionDefinitionsBuilder(G_FCMP)
-      .legalIf([=](const LegalityQuery &Query) {
-        return ((HasSSE1 || UseX87) &&
-                typePairInSet(0, 1, {{s8, s32}})(Query)) ||
-               ((HasSSE2 || UseX87) &&
-                typePairInSet(0, 1, {{s8, s64}})(Query)) ||
-               (UseX87 && typePairInSet(0, 1, {{s8, s80}})(Query));
-      })
+      .legalFor((HasSSE1 || UseX87), {s8, s32})
+      .legalFor((HasSSE2 || UseX87), {s8, s64})
+      .legalFor(UseX87, {s8, s80})
       .clampScalar(0, s8, s8)
       .clampScalar(1, s32, HasSSE2 ? s64 : s32)
       .widenScalarToNextPow2(1);