[PATCH] D117898: [Clang] Add elementwise saturated add/sub builtins

Simon Pilgrim via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 7 09:41:15 PST 2022


RKSimon added inline comments.


================
Comment at: clang/docs/LanguageExtensions.rst:549
+ T __builtin_elementwise_add_sat(T x, T y) return the sum of x and y, clamped to the range of signed or     integer types
+                                           values representable by the bit width of the arguments.
+ T __builtin_elementwise_sub_sat(T x, T y) return the difference of x and y, clamped to the range of        integer types
----------------
aaron.ballman wrote:
> aaron.ballman wrote:
> > craig.topper wrote:
> > > Not sure if I'm reading this right due to the columns, but is "unsigned" missing after the "signed or"
> > This reads strangely to me as well. "..., clamped to the range of signed or integer types unsigned values representable by.."
> This still seems unaddressed.
I rephrased it - what did you have in mind?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117898



More information about the cfe-commits mailing list