[PATCH] D86686: [PowerPC] Fix store-fptoi combine of f128 on Power8

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:14098
   // Floating point types smaller than 32 bits are not legal on Power.
-  if (ResVT.getScalarSizeInBits() < 32)
+  // 128-bit floating point type isn't supported on pre-Power9 subtargets.
+  if (!isTypeLegal(ResVT))
----------------
I think, we can remove these two lines comment now as it is not needed any more.


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

https://reviews.llvm.org/D86686