[PATCH] Allow code generation of ARM usat/ssat instructions

Weiming Zhao weimingz at codeaurora.org
Wed Apr 1 09:11:02 PDT 2015


Hi James,

Right. After Ahmed's generic saturation intrinsic patch is merged, let's revisit it.

Thanks,
Weiming

In http://reviews.llvm.org/D8371#150356, @jmolloy wrote:

> Hi Weiming,
>
> I've been looking at clamp() too, specifically when we don't have the right bounds for a saturation, so we need to emit a sequence of min + max instructions.
>
> I think your patch is a good basis for this, but please do bear in mind the other use case when designing your helper functions so it is possible to add this behaviour in the future!
>
> Cheers,
>
> James





http://reviews.llvm.org/D8371

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list