[PATCH] D22105: [X86][SSE] Reimplement SSE fp2si conversion intrinsics instead of using generic IR

Simon Pilgrim via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 19 11:41:29 PDT 2016


RKSimon added a comment.

In https://reviews.llvm.org/D22105#488566, @eli.friedman wrote:

> The x86-specific operation is affected by the rounding mode... but so is a C cast.  This is specified by Annex F in the C standard.
>
> Of course, you're going to end up with undefined behavior if you actually modify the rounding mode because LLVM and clang don't support FENV_ACCESS at the moment.


OK I'm going to pull the sitofp conversions from this patch - I have other concerns about them (i.e. we don't treat scalar + vector the same) that will need to be looked at as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D22105





More information about the cfe-commits mailing list