[PATCH] D83715: AMDGPU/GlobalISel: Use clamp modifier for uaddsat/usubsat

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 14 01:56:56 PDT 2020


foad added a comment.

> There's an annoying asymmetry where we have signed saturating for
>  packed operations, but not for scalar 16-bit.

My Vega Instruction Set Architecture Reference Guide says:
"V_ADD_I16
D.i16 = S0.i16 + S1.i16.
Supports saturation (signed 16-bit integer domain)."


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

https://reviews.llvm.org/D83715





More information about the llvm-commits mailing list