[PATCH] D46863: [X86] Use __builtin_convertvector to implement some of the packed integer to packed float conversion intrinsics.

David Kreitzer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 16 13:26:57 PDT 2018


DavidKreitzer added a comment.

This looks good to me, Craig. I am not worried about the constant folding issue, as I think constant folding these conversion intrinsics (assuming round-to-nearest) is a perfectly valid optimization in the absence of FENV_ACCESS. (FWIW, we don't do this constant folding in icc, but that is only because we have never gotten around to implementing it.) I am also not worried about the spurious 'inexact' exceptions that the changes to the mask intrinsics will cause.


Repository:
  rC Clang

https://reviews.llvm.org/D46863





More information about the cfe-commits mailing list