[PATCH] D71940: [PowerPC] Legalize saturating vector add/sub

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai marked an inline comment as done.
nemanjai added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrAltivec.td:871
           (v4i32 (VRLW v4i32:$vA, v4i32:$vB))>;
+def : Pat<(v16i8 (saddsat v16i8:$vA, v16i8:$vB)), (v16i8 (VADDSBS $vA, $vB))>;
+def : Pat<(v16i8 (uaddsat v16i8:$vA, v16i8:$vB)), (v16i8 (VADDUBS $vA, $vB))>;
----------------
lkail wrote:
> Add `// Saturates.` before this line to separate following instructions from other classes of instructions?
Sounds good.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71940