[llvm] r330437 - [DAGCombine] (float)((int) f) --> ftrunc (PR36617)

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 25 16:28:57 PDT 2018


+1

I will say, what i want right now is the flag to go back to the old
behavior. ;] Our releases will end up blocked on this.

On Wed, Apr 25, 2018 at 4:00 PM Stephen Canon <scanon at apple.com> wrote:

> We should also add an intrinsic for fpto[us]I with defined overflow
> behavior, and have the sanitizer call it to people’s attention. Otherwise
> people will roll their own clamping and most of them will get it wrong,
> because INT_MAX isn’t representable as a float.
>
> Ideally we should expose this at the clang level as something like bool
> __builtin_convert_overflow(src, &dst) and make the one builtin work across
> types, as with __builtin_add_overflow and friends.
>
> – Steve
>
>
> On Apr 25, 2018, at 6:11 PM, Chandler Carruth via llvm-commits <
> llvm-commits at lists.llvm.org> wrote:
>
> On Wed, Apr 25, 2018 at 2:58 PM Sanjay Patel <spatel at rotateright.com>
> wrote:
>
>> Damn...
>>
>> 1. I've never actually used the sanitizers for anything more than simple
>> experiments. But a look at the docs and simple test says this will do it:
>> https://clang.llvm.org/docs/UndefinedBehaviorSanitizer.html
>> -fsanitize=float-cast-overflow
>>
>
> If this covers everything, great. Maybe document this in the release notes
> as well?
>
>
>> 2. Yes, that's probably wiser. -fjunk-in-trunc ? :)
>>
>
> -fstrict-fp-trunc-semantics maybe?
>
> Or maybe we should phrase it more:
>
> -ffp-cast-overflow-workaround
>
> To make it clear that enabling this is a bad idea and something that needs
> to go away?
>
> We should be *really* clear around the documentation of the flag that it
> will not be supported long term and will go away in a future release. (And
> be really clear about that in the release notes as well.)
>
> May make sense to make it CC1 only.
>
> Anyways, all of this is a better clang commits list discussion. =]
>
>
>>
>>
>>
>> On Wed, Apr 25, 2018 at 3:38 PM, Chandler Carruth <chandlerc at gmail.com>
>> wrote:
>>
>>> Hey Sanjay, this is hitting a *lot* of code, even outside of Chrome....
>>>
>>> I think we need a more nuanced way to handle this.
>>>
>>> 1) Is there a specific sanitizer that we can enable that will point out
>>> *just* the issues this impacts?
>>>
>>> 2) Can we have a flag to control this behavior at least for a temporary
>>> period of time? that would make it much easier to roll out when there are
>>> very large numbers of issues encountered.
>>>
>>> On Fri, Apr 20, 2018 at 8:11 AM Sanjay Patel via llvm-commits <
>>> llvm-commits at lists.llvm.org> wrote:
>>>
>>>> Author: spatel
>>>> Date: Fri Apr 20 08:07:55 2018
>>>> New Revision: 330437
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=330437&view=rev
>>>> Log:
>>>> [DAGCombine] (float)((int) f) --> ftrunc (PR36617)
>>>>
>>>> This was originally committed at rL328921 and reverted at rL329920 to
>>>> investigate failures in Chrome. This time I've added to the ReleaseNotes
>>>> to warn users of the potential of exposing UB and let me repeat that
>>>> here for more exposure:
>>>>
>>>>   Optimization of floating-point casts is improved. This may cause
>>>> surprising
>>>>   results for code that is relying on undefined behavior. Code
>>>> sanitizers can
>>>>   be used to detect affected patterns such as this:
>>>>
>>>>     int main() {
>>>>       float x = 4294967296.0f;
>>>>       x = (float)((int)x);
>>>>       printf("junk in the ftrunc: %f\n", x);
>>>>       return 0;
>>>>     }
>>>>
>>>>     $ clang -O1 ftrunc.c -fsanitize=undefined ; ./a.out
>>>>     ftrunc.c:5:15: runtime error: 4.29497e+09 is outside the range of
>>>>                    representable values of type 'int'
>>>>     junk in the ftrunc: 0.000000
>>>>
>>>>
>>>> Original commit message:
>>>>
>>>> fptosi / fptoui round towards zero, and that's the same behavior as
>>>> ISD::FTRUNC,
>>>> so replace a pair of casts with the equivalent node. We don't have to
>>>> account for
>>>> special cases (NaN, INF) because out-of-range casts are undefined.
>>>>
>>>> Differential Revision: https://reviews.llvm.org/D44909
>>>>
>>>> Added:
>>>>     llvm/trunk/test/CodeGen/ARM/ftrunc.ll
>>>>       - copied unchanged from r329919,
>>>> llvm/trunk/test/CodeGen/ARM/ftrunc.ll
>>>> Modified:
>>>>     llvm/trunk/docs/ReleaseNotes.rst
>>>>     llvm/trunk/include/llvm/CodeGen/ISDOpcodes.h
>>>>     llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
>>>>     llvm/trunk/test/CodeGen/AArch64/ftrunc.ll
>>>>     llvm/trunk/test/CodeGen/PowerPC/fp-int128-fp-combine.ll
>>>>     llvm/trunk/test/CodeGen/PowerPC/fp-to-int-to-fp.ll
>>>>     llvm/trunk/test/CodeGen/PowerPC/ftrunc-vec.ll
>>>>     llvm/trunk/test/CodeGen/PowerPC/no-extra-fp-conv-ldst.ll
>>>>     llvm/trunk/test/CodeGen/X86/2011-10-19-widen_vselect.ll
>>>>     llvm/trunk/test/CodeGen/X86/ftrunc.ll
>>>>
>>>> Modified: llvm/trunk/docs/ReleaseNotes.rst
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/ReleaseNotes.rst?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/docs/ReleaseNotes.rst (original)
>>>> +++ llvm/trunk/docs/ReleaseNotes.rst Fri Apr 20 08:07:55 2018
>>>> @@ -61,6 +61,26 @@ Non-comprehensive list of changes in thi
>>>>  * The optimization flag to merge constants (-fmerge-all-constants) is
>>>> no longer
>>>>    applied by default.
>>>>
>>>> +* Optimization of floating-point casts is improved. This may cause
>>>> surprising
>>>> +  results for code that is relying on undefined behavior. Code
>>>> sanitizers can
>>>> +  be used to detect affected patterns such as this:
>>>> +
>>>> +.. code-block:: c
>>>> +
>>>> +    int main() {
>>>> +      float x = 4294967296.0f;
>>>> +      x = (float)((int)x);
>>>> +      printf("junk in the ftrunc: %f\n", x);
>>>> +      return 0;
>>>> +    }
>>>> +
>>>> +.. code-block:: bash
>>>> +
>>>> +    clang -O1 ftrunc.c -fsanitize=undefined ; ./a.out
>>>> +    ftrunc.c:5:15: runtime error: 4.29497e+09 is outside the range of
>>>> representable values of type 'int'
>>>> +    junk in the ftrunc: 0.000000
>>>> +
>>>> +
>>>>  * Note..
>>>>
>>>>  .. NOTE
>>>>
>>>> Modified: llvm/trunk/include/llvm/CodeGen/ISDOpcodes.h
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/ISDOpcodes.h?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/include/llvm/CodeGen/ISDOpcodes.h (original)
>>>> +++ llvm/trunk/include/llvm/CodeGen/ISDOpcodes.h Fri Apr 20 08:07:55
>>>> 2018
>>>> @@ -495,7 +495,8 @@ namespace ISD {
>>>>      ZERO_EXTEND_VECTOR_INREG,
>>>>
>>>>      /// FP_TO_[US]INT - Convert a floating point value to a signed or
>>>> unsigned
>>>> -    /// integer.
>>>> +    /// integer. These have the same semantics as fptosi and fptoui in
>>>> IR. If
>>>> +    /// the FP value cannot fit in the integer type, the results are
>>>> undefined.
>>>>      FP_TO_SINT,
>>>>      FP_TO_UINT,
>>>>
>>>>
>>>> Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
>>>> +++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Fri Apr 20
>>>> 08:07:55 2018
>>>> @@ -10890,6 +10890,15 @@ SDValue DAGCombiner::visitSINT_TO_FP(SDN
>>>>      }
>>>>    }
>>>>
>>>> +  // fptosi rounds towards zero, so converting from FP to integer and
>>>> back is
>>>> +  // the same as an 'ftrunc': sitofp (fptosi X) --> ftrunc X
>>>> +  // We only do this if the target has legal ftrunc, otherwise we'd
>>>> likely be
>>>> +  // replacing casts with a libcall.
>>>> +  if (N0.getOpcode() == ISD::FP_TO_SINT &&
>>>> +      N0.getOperand(0).getValueType() == VT &&
>>>> +      TLI.isOperationLegal(ISD::FTRUNC, VT))
>>>> +    return DAG.getNode(ISD::FTRUNC, SDLoc(N), VT, N0.getOperand(0));
>>>> +
>>>>    return SDValue();
>>>>  }
>>>>
>>>> @@ -10929,6 +10938,15 @@ SDValue DAGCombiner::visitUINT_TO_FP(SDN
>>>>      }
>>>>    }
>>>>
>>>> +  // fptoui rounds towards zero, so converting from FP to integer and
>>>> back is
>>>> +  // the same as an 'ftrunc': uitofp (fptoui X) --> ftrunc X
>>>> +  // We only do this if the target has legal ftrunc, otherwise we'd
>>>> likely be
>>>> +  // replacing casts with a libcall.
>>>> +  if (N0.getOpcode() == ISD::FP_TO_UINT &&
>>>> +      N0.getOperand(0).getValueType() == VT &&
>>>> +      TLI.isOperationLegal(ISD::FTRUNC, VT))
>>>> +    return DAG.getNode(ISD::FTRUNC, SDLoc(N), VT, N0.getOperand(0));
>>>> +
>>>>    return SDValue();
>>>>  }
>>>>
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/AArch64/ftrunc.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/AArch64/ftrunc.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/AArch64/ftrunc.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/AArch64/ftrunc.ll Fri Apr 20 08:07:55 2018
>>>> @@ -4,8 +4,7 @@
>>>>  define float @trunc_unsigned_f32(float %x) {
>>>>  ; CHECK-LABEL: trunc_unsigned_f32:
>>>>  ; CHECK:       // %bb.0:
>>>> -; CHECK-NEXT:    fcvtzu w8, s0
>>>> -; CHECK-NEXT:    ucvtf s0, w8
>>>> +; CHECK-NEXT:    frintz s0, s0
>>>>  ; CHECK-NEXT:    ret
>>>>    %i = fptoui float %x to i32
>>>>    %r = uitofp i32 %i to float
>>>> @@ -15,8 +14,7 @@ define float @trunc_unsigned_f32(float %
>>>>  define double @trunc_unsigned_f64(double %x) {
>>>>  ; CHECK-LABEL: trunc_unsigned_f64:
>>>>  ; CHECK:       // %bb.0:
>>>> -; CHECK-NEXT:    fcvtzu x8, d0
>>>> -; CHECK-NEXT:    ucvtf d0, x8
>>>> +; CHECK-NEXT:    frintz d0, d0
>>>>  ; CHECK-NEXT:    ret
>>>>    %i = fptoui double %x to i64
>>>>    %r = uitofp i64 %i to double
>>>> @@ -26,8 +24,7 @@ define double @trunc_unsigned_f64(double
>>>>  define float @trunc_signed_f32(float %x) {
>>>>  ; CHECK-LABEL: trunc_signed_f32:
>>>>  ; CHECK:       // %bb.0:
>>>> -; CHECK-NEXT:    fcvtzs w8, s0
>>>> -; CHECK-NEXT:    scvtf s0, w8
>>>> +; CHECK-NEXT:    frintz s0, s0
>>>>  ; CHECK-NEXT:    ret
>>>>    %i = fptosi float %x to i32
>>>>    %r = sitofp i32 %i to float
>>>> @@ -37,8 +34,7 @@ define float @trunc_signed_f32(float %x)
>>>>  define double @trunc_signed_f64(double %x) {
>>>>  ; CHECK-LABEL: trunc_signed_f64:
>>>>  ; CHECK:       // %bb.0:
>>>> -; CHECK-NEXT:    fcvtzs x8, d0
>>>> -; CHECK-NEXT:    scvtf d0, x8
>>>> +; CHECK-NEXT:    frintz d0, d0
>>>>  ; CHECK-NEXT:    ret
>>>>    %i = fptosi double %x to i64
>>>>    %r = sitofp i64 %i to double
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/PowerPC/fp-int128-fp-combine.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/fp-int128-fp-combine.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/PowerPC/fp-int128-fp-combine.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/PowerPC/fp-int128-fp-combine.ll Fri Apr 20
>>>> 08:07:55 2018
>>>> @@ -5,18 +5,7 @@
>>>>  define float @f_i128_f(float %v) {
>>>>  ; CHECK-LABEL: f_i128_f:
>>>>  ; CHECK:       # %bb.0: # %entry
>>>> -; CHECK-NEXT:    mflr 0
>>>> -; CHECK-NEXT:    std 0, 16(1)
>>>> -; CHECK-NEXT:    stdu 1, -32(1)
>>>> -; CHECK-NEXT:    .cfi_def_cfa_offset 32
>>>> -; CHECK-NEXT:    .cfi_offset lr, 16
>>>> -; CHECK-NEXT:    bl __fixsfti
>>>> -; CHECK-NEXT:    nop
>>>> -; CHECK-NEXT:    bl __floattisf
>>>> -; CHECK-NEXT:    nop
>>>> -; CHECK-NEXT:    addi 1, 1, 32
>>>> -; CHECK-NEXT:    ld 0, 16(1)
>>>> -; CHECK-NEXT:    mtlr 0
>>>> +; CHECK-NEXT:    friz 1, 1
>>>>  ; CHECK-NEXT:    blr
>>>>  entry:
>>>>    %a = fptosi float %v to i128
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/PowerPC/fp-to-int-to-fp.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/fp-to-int-to-fp.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/PowerPC/fp-to-int-to-fp.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/PowerPC/fp-to-int-to-fp.ll Fri Apr 20
>>>> 08:07:55 2018
>>>> @@ -11,8 +11,7 @@ entry:
>>>>    ret float %conv1
>>>>
>>>>  ; FPCVT-LABEL: @fool
>>>> -; FPCVT: fctidz [[REG1:[0-9]+]], 1
>>>> -; FPCVT: fcfids 1, [[REG1]]
>>>> +; FPCVT: friz 1, 1
>>>>  ; FPCVT: blr
>>>>
>>>>  ; PPC64-LABEL: @fool
>>>> @@ -30,8 +29,7 @@ entry:
>>>>    ret double %conv1
>>>>
>>>>  ; FPCVT-LABEL: @foodl
>>>> -; FPCVT: fctidz [[REG1:[0-9]+]], 1
>>>> -; FPCVT: fcfid 1, [[REG1]]
>>>> +; FPCVT: friz 1, 1
>>>>  ; FPCVT: blr
>>>>
>>>>  ; PPC64-LABEL: @foodl
>>>> @@ -48,8 +46,7 @@ entry:
>>>>    ret float %conv1
>>>>
>>>>  ; FPCVT-LABEL: @fooul
>>>> -; FPCVT: fctiduz [[REG1:[0-9]+]], 1
>>>> -; FPCVT: fcfidus 1, [[REG1]]
>>>> +; FPCVT: friz 1, 1
>>>>  ; FPCVT: blr
>>>>  }
>>>>
>>>> @@ -61,8 +58,7 @@ entry:
>>>>    ret double %conv1
>>>>
>>>>  ; FPCVT-LABEL: @fooudl
>>>> -; FPCVT: fctiduz [[REG1:[0-9]+]], 1
>>>> -; FPCVT: fcfidu 1, [[REG1]]
>>>> +; FPCVT: friz 1, 1
>>>>  ; FPCVT: blr
>>>>  }
>>>>
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/PowerPC/ftrunc-vec.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/ftrunc-vec.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/PowerPC/ftrunc-vec.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/PowerPC/ftrunc-vec.ll Fri Apr 20 08:07:55
>>>> 2018
>>>> @@ -4,8 +4,7 @@
>>>>  define <4 x float> @truncf32(<4 x float> %a) {
>>>>  ; CHECK-LABEL: truncf32:
>>>>  ; CHECK:       # %bb.0:
>>>> -; CHECK-NEXT:    xvcvspsxws 0, 34
>>>> -; CHECK-NEXT:    xvcvsxwsp 34, 0
>>>> +; CHECK-NEXT:    xvrspiz 34, 34
>>>>  ; CHECK-NEXT:    blr
>>>>    %t0 = fptosi <4 x float> %a to <4 x i32>
>>>>    %t1 = sitofp <4 x i32> %t0 to <4 x float>
>>>> @@ -15,8 +14,7 @@ define <4 x float> @truncf32(<4 x float>
>>>>  define <2 x double> @truncf64(<2 x double> %a) {
>>>>  ; CHECK-LABEL: truncf64:
>>>>  ; CHECK:       # %bb.0:
>>>> -; CHECK-NEXT:    xvcvdpsxds 34, 34
>>>> -; CHECK-NEXT:    xvcvsxddp 34, 34
>>>> +; CHECK-NEXT:    xvrdpiz 34, 34
>>>>  ; CHECK-NEXT:    blr
>>>>    %t0 = fptosi <2 x double> %a to <2 x i64>
>>>>    %t1 = sitofp <2 x i64> %t0 to <2 x double>
>>>> @@ -26,8 +24,7 @@ define <2 x double> @truncf64(<2 x doubl
>>>>  define <4 x float> @truncf32u(<4 x float> %a) {
>>>>  ; CHECK-LABEL: truncf32u:
>>>>  ; CHECK:       # %bb.0:
>>>> -; CHECK-NEXT:    xvcvspuxws 0, 34
>>>> -; CHECK-NEXT:    xvcvuxwsp 34, 0
>>>> +; CHECK-NEXT:    xvrspiz 34, 34
>>>>  ; CHECK-NEXT:    blr
>>>>    %t0 = fptoui <4 x float> %a to <4 x i32>
>>>>    %t1 = uitofp <4 x i32> %t0 to <4 x float>
>>>> @@ -37,8 +34,7 @@ define <4 x float> @truncf32u(<4 x float
>>>>  define <2 x double> @truncf64u(<2 x double> %a) {
>>>>  ; CHECK-LABEL: truncf64u:
>>>>  ; CHECK:       # %bb.0:
>>>> -; CHECK-NEXT:    xvcvdpuxds 34, 34
>>>> -; CHECK-NEXT:    xvcvuxddp 34, 34
>>>> +; CHECK-NEXT:    xvrdpiz 34, 34
>>>>  ; CHECK-NEXT:    blr
>>>>    %t0 = fptoui <2 x double> %a to <2 x i64>
>>>>    %t1 = uitofp <2 x i64> %t0 to <2 x double>
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/PowerPC/no-extra-fp-conv-ldst.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/no-extra-fp-conv-ldst.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/PowerPC/no-extra-fp-conv-ldst.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/PowerPC/no-extra-fp-conv-ldst.ll Fri Apr 20
>>>> 08:07:55 2018
>>>> @@ -36,11 +36,7 @@ entry:
>>>>    ret float %conv1
>>>>
>>>>  ; CHECK-LABEL: @foo
>>>> -; CHECK-DAG: fctiwz [[REG2:[0-9]+]], 1
>>>> -; CHECK-DAG: addi [[REG1:[0-9]+]], 1,
>>>> -; CHECK: stfiwx [[REG2]], 0, [[REG1]]
>>>> -; CHECK: lfiwax [[REG3:[0-9]+]], 0, [[REG1]]
>>>> -; CHECK: fcfids 1, [[REG3]]
>>>> +; CHECK: friz 1, 1
>>>>  ; CHECK: blr
>>>>  }
>>>>
>>>> @@ -52,11 +48,7 @@ entry:
>>>>    ret double %conv1
>>>>
>>>>  ; CHECK-LABEL: @food
>>>> -; CHECK-DAG: fctiwz [[REG2:[0-9]+]], 1
>>>> -; CHECK-DAG: addi [[REG1:[0-9]+]], 1,
>>>> -; CHECK: stfiwx [[REG2]], 0, [[REG1]]
>>>> -; CHECK: lfiwax [[REG3:[0-9]+]], 0, [[REG1]]
>>>> -; CHECK: fcfid 1, [[REG3]]
>>>> +; CHECK: friz 1, 1
>>>>  ; CHECK: blr
>>>>  }
>>>>
>>>> @@ -68,11 +60,7 @@ entry:
>>>>    ret float %conv1
>>>>
>>>>  ; CHECK-LABEL: @foou
>>>> -; CHECK-DAG: fctiwuz [[REG2:[0-9]+]], 1
>>>> -; CHECK-DAG: addi [[REG1:[0-9]+]], 1,
>>>> -; CHECK: stfiwx [[REG2]], 0, [[REG1]]
>>>> -; CHECK: lfiwzx [[REG3:[0-9]+]], 0, [[REG1]]
>>>> -; CHECK: fcfidus 1, [[REG3]]
>>>> +; CHECK: friz 1, 1
>>>>  ; CHECK: blr
>>>>  }
>>>>
>>>> @@ -84,11 +72,7 @@ entry:
>>>>    ret double %conv1
>>>>
>>>>  ; CHECK-LABEL: @fooud
>>>> -; CHECK-DAG: fctiwuz [[REG2:[0-9]+]], 1
>>>> -; CHECK-DAG: addi [[REG1:[0-9]+]], 1,
>>>> -; CHECK: stfiwx [[REG2]], 0, [[REG1]]
>>>> -; CHECK: lfiwzx [[REG3:[0-9]+]], 0, [[REG1]]
>>>> -; CHECK: fcfidu 1, [[REG3]]
>>>> +; CHECK: friz 1, 1
>>>>  ; CHECK: blr
>>>>  }
>>>>
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/X86/2011-10-19-widen_vselect.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/2011-10-19-widen_vselect.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/X86/2011-10-19-widen_vselect.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/X86/2011-10-19-widen_vselect.ll Fri Apr 20
>>>> 08:07:55 2018
>>>> @@ -71,8 +71,7 @@ define void @full_test() {
>>>>  ; X32-NEXT:    subl $60, %esp
>>>>  ; X32-NEXT:    .cfi_def_cfa_offset 64
>>>>  ; X32-NEXT:    movsd {{.*#+}} xmm2 = mem[0],zero
>>>> -; X32-NEXT:    cvttps2dq %xmm2, %xmm0
>>>> -; X32-NEXT:    cvtdq2ps %xmm0, %xmm1
>>>> +; X32-NEXT:    roundps $11, %xmm2, %xmm1
>>>>  ; X32-NEXT:    xorps %xmm0, %xmm0
>>>>  ; X32-NEXT:    cmpltps %xmm2, %xmm0
>>>>  ; X32-NEXT:    movaps {{.*#+}} xmm3 = <1,1,u,u>
>>>> @@ -93,8 +92,7 @@ define void @full_test() {
>>>>  ; X64-LABEL: full_test:
>>>>  ; X64:       # %bb.0: # %entry
>>>>  ; X64-NEXT:    movsd {{.*#+}} xmm2 = mem[0],zero
>>>> -; X64-NEXT:    cvttps2dq %xmm2, %xmm0
>>>> -; X64-NEXT:    cvtdq2ps %xmm0, %xmm1
>>>> +; X64-NEXT:    roundps $11, %xmm2, %xmm1
>>>>  ; X64-NEXT:    xorps %xmm0, %xmm0
>>>>  ; X64-NEXT:    cmpltps %xmm2, %xmm0
>>>>  ; X64-NEXT:    movaps {{.*#+}} xmm3 = <1,1,u,u>
>>>>
>>>> Modified: llvm/trunk/test/CodeGen/X86/ftrunc.ll
>>>> URL:
>>>> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/ftrunc.ll?rev=330437&r1=330436&r2=330437&view=diff
>>>>
>>>> ==============================================================================
>>>> --- llvm/trunk/test/CodeGen/X86/ftrunc.ll (original)
>>>> +++ llvm/trunk/test/CodeGen/X86/ftrunc.ll Fri Apr 20 08:07:55 2018
>>>> @@ -14,17 +14,12 @@ define float @trunc_unsigned_f32(float %
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_unsigned_f32:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    cvttss2si %xmm0, %rax
>>>> -; SSE41-NEXT:    movl %eax, %eax
>>>> -; SSE41-NEXT:    xorps %xmm0, %xmm0
>>>> -; SSE41-NEXT:    cvtsi2ssq %rax, %xmm0
>>>> +; SSE41-NEXT:    roundss $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_unsigned_f32:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vcvttss2si %xmm0, %rax
>>>> -; AVX1-NEXT:    movl %eax, %eax
>>>> -; AVX1-NEXT:    vcvtsi2ssq %rax, %xmm1, %xmm0
>>>> +; AVX1-NEXT:    vroundss $11, %xmm0, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptoui float %x to i32
>>>>    %r = uitofp i32 %i to float
>>>> @@ -52,35 +47,12 @@ define double @trunc_unsigned_f64(double
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_unsigned_f64:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    movsd {{.*#+}} xmm1 = mem[0],zero
>>>> -; SSE41-NEXT:    movapd %xmm0, %xmm2
>>>> -; SSE41-NEXT:    subsd %xmm1, %xmm2
>>>> -; SSE41-NEXT:    cvttsd2si %xmm2, %rax
>>>> -; SSE41-NEXT:    movabsq $-9223372036854775808, %rcx # imm =
>>>> 0x8000000000000000
>>>> -; SSE41-NEXT:    xorq %rax, %rcx
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rax
>>>> -; SSE41-NEXT:    ucomisd %xmm1, %xmm0
>>>> -; SSE41-NEXT:    cmovaeq %rcx, %rax
>>>> -; SSE41-NEXT:    movq %rax, %xmm0
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm0 =
>>>> xmm0[0],mem[0],xmm0[1],mem[1]
>>>> -; SSE41-NEXT:    subpd {{.*}}(%rip), %xmm0
>>>> -; SSE41-NEXT:    haddpd %xmm0, %xmm0
>>>> +; SSE41-NEXT:    roundsd $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_unsigned_f64:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vmovsd {{.*#+}} xmm1 = mem[0],zero
>>>> -; AVX1-NEXT:    vsubsd %xmm1, %xmm0, %xmm2
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm2, %rax
>>>> -; AVX1-NEXT:    movabsq $-9223372036854775808, %rcx # imm =
>>>> 0x8000000000000000
>>>> -; AVX1-NEXT:    xorq %rax, %rcx
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rax
>>>> -; AVX1-NEXT:    vucomisd %xmm1, %xmm0
>>>> -; AVX1-NEXT:    cmovaeq %rcx, %rax
>>>> -; AVX1-NEXT:    vmovq %rax, %xmm0
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm0 =
>>>> xmm0[0],mem[0],xmm0[1],mem[1]
>>>> -; AVX1-NEXT:    vsubpd {{.*}}(%rip), %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vhaddpd %xmm0, %xmm0, %xmm0
>>>> +; AVX1-NEXT:    vroundsd $11, %xmm0, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptoui double %x to i64
>>>>    %r = uitofp i64 %i to double
>>>> @@ -118,45 +90,12 @@ define <4 x float> @trunc_unsigned_v4f32
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_unsigned_v4f32:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    movshdup {{.*#+}} xmm1 = xmm0[1,1,3,3]
>>>> -; SSE41-NEXT:    cvttss2si %xmm1, %rax
>>>> -; SSE41-NEXT:    cvttss2si %xmm0, %rcx
>>>> -; SSE41-NEXT:    movd %ecx, %xmm1
>>>> -; SSE41-NEXT:    pinsrd $1, %eax, %xmm1
>>>> -; SSE41-NEXT:    movaps %xmm0, %xmm2
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm2 = xmm0[1],xmm2[1]
>>>> -; SSE41-NEXT:    cvttss2si %xmm2, %rax
>>>> -; SSE41-NEXT:    pinsrd $2, %eax, %xmm1
>>>> -; SSE41-NEXT:    shufps {{.*#+}} xmm0 = xmm0[3,1,2,3]
>>>> -; SSE41-NEXT:    cvttss2si %xmm0, %rax
>>>> -; SSE41-NEXT:    pinsrd $3, %eax, %xmm1
>>>> -; SSE41-NEXT:    movdqa {{.*#+}} xmm0 =
>>>> [1258291200,1258291200,1258291200,1258291200]
>>>> -; SSE41-NEXT:    pblendw {{.*#+}} xmm0 =
>>>> xmm1[0],xmm0[1],xmm1[2],xmm0[3],xmm1[4],xmm0[5],xmm1[6],xmm0[7]
>>>> -; SSE41-NEXT:    psrld $16, %xmm1
>>>> -; SSE41-NEXT:    pblendw {{.*#+}} xmm1 =
>>>> xmm1[0],mem[1],xmm1[2],mem[3],xmm1[4],mem[5],xmm1[6],mem[7]
>>>> -; SSE41-NEXT:    addps {{.*}}(%rip), %xmm1
>>>> -; SSE41-NEXT:    addps %xmm0, %xmm1
>>>> -; SSE41-NEXT:    movaps %xmm1, %xmm0
>>>> +; SSE41-NEXT:    roundps $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_unsigned_v4f32:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vmovshdup {{.*#+}} xmm1 = xmm0[1,1,3,3]
>>>> -; AVX1-NEXT:    vcvttss2si %xmm1, %rax
>>>> -; AVX1-NEXT:    vcvttss2si %xmm0, %rcx
>>>> -; AVX1-NEXT:    vmovd %ecx, %xmm1
>>>> -; AVX1-NEXT:    vpinsrd $1, %eax, %xmm1, %xmm1
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm2 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vcvttss2si %xmm2, %rax
>>>> -; AVX1-NEXT:    vpinsrd $2, %eax, %xmm1, %xmm1
>>>> -; AVX1-NEXT:    vpermilps {{.*#+}} xmm0 = xmm0[3,1,2,3]
>>>> -; AVX1-NEXT:    vcvttss2si %xmm0, %rax
>>>> -; AVX1-NEXT:    vpinsrd $3, %eax, %xmm1, %xmm0
>>>> -; AVX1-NEXT:    vpblendw {{.*#+}} xmm1 =
>>>> xmm0[0],mem[1],xmm0[2],mem[3],xmm0[4],mem[5],xmm0[6],mem[7]
>>>> -; AVX1-NEXT:    vpsrld $16, %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vpblendw {{.*#+}} xmm0 =
>>>> xmm0[0],mem[1],xmm0[2],mem[3],xmm0[4],mem[5],xmm0[6],mem[7]
>>>> -; AVX1-NEXT:    vaddps {{.*}}(%rip), %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vaddps %xmm0, %xmm1, %xmm0
>>>> +; AVX1-NEXT:    vroundps $11, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptoui <4 x float> %x to <4 x i32>
>>>>    %r = uitofp <4 x i32> %i to <4 x float>
>>>> @@ -201,61 +140,12 @@ define <2 x double> @trunc_unsigned_v2f6
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_unsigned_v2f64:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    movaps %xmm0, %xmm1
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm1 = xmm0[1],xmm1[1]
>>>> -; SSE41-NEXT:    movsd {{.*#+}} xmm2 = mem[0],zero
>>>> -; SSE41-NEXT:    movaps %xmm1, %xmm3
>>>> -; SSE41-NEXT:    subsd %xmm2, %xmm3
>>>> -; SSE41-NEXT:    cvttsd2si %xmm3, %rax
>>>> -; SSE41-NEXT:    movabsq $-9223372036854775808, %rcx # imm =
>>>> 0x8000000000000000
>>>> -; SSE41-NEXT:    xorq %rcx, %rax
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rdx
>>>> -; SSE41-NEXT:    ucomisd %xmm2, %xmm1
>>>> -; SSE41-NEXT:    cmovaeq %rax, %rdx
>>>> -; SSE41-NEXT:    movaps %xmm0, %xmm1
>>>> -; SSE41-NEXT:    subsd %xmm2, %xmm1
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rax
>>>> -; SSE41-NEXT:    xorq %rcx, %rax
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rcx
>>>> -; SSE41-NEXT:    ucomisd %xmm2, %xmm0
>>>> -; SSE41-NEXT:    cmovaeq %rax, %rcx
>>>> -; SSE41-NEXT:    movq %rcx, %xmm0
>>>> -; SSE41-NEXT:    movdqa {{.*#+}} xmm1 = [1127219200,1160773632,0,0]
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm0 =
>>>> xmm0[0],xmm1[0],xmm0[1],xmm1[1]
>>>> -; SSE41-NEXT:    movapd {{.*#+}} xmm2 = [4.503600e+15,1.934281e+25]
>>>> -; SSE41-NEXT:    subpd %xmm2, %xmm0
>>>> -; SSE41-NEXT:    movq %rdx, %xmm3
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm3 =
>>>> xmm3[0],xmm1[0],xmm3[1],xmm1[1]
>>>> -; SSE41-NEXT:    subpd %xmm2, %xmm3
>>>> -; SSE41-NEXT:    haddpd %xmm3, %xmm0
>>>> +; SSE41-NEXT:    roundpd $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_unsigned_v2f64:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm1 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vmovsd {{.*#+}} xmm2 = mem[0],zero
>>>> -; AVX1-NEXT:    vsubsd %xmm2, %xmm1, %xmm3
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm3, %rax
>>>> -; AVX1-NEXT:    movabsq $-9223372036854775808, %rcx # imm =
>>>> 0x8000000000000000
>>>> -; AVX1-NEXT:    xorq %rcx, %rax
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm1, %rdx
>>>> -; AVX1-NEXT:    vucomisd %xmm2, %xmm1
>>>> -; AVX1-NEXT:    cmovaeq %rax, %rdx
>>>> -; AVX1-NEXT:    vsubsd %xmm2, %xmm0, %xmm1
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm1, %rax
>>>> -; AVX1-NEXT:    xorq %rcx, %rax
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rcx
>>>> -; AVX1-NEXT:    vucomisd %xmm2, %xmm0
>>>> -; AVX1-NEXT:    cmovaeq %rax, %rcx
>>>> -; AVX1-NEXT:    vmovq %rcx, %xmm0
>>>> -; AVX1-NEXT:    vmovdqa {{.*#+}} xmm1 = [1127219200,1160773632,0,0]
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm0 =
>>>> xmm0[0],xmm1[0],xmm0[1],xmm1[1]
>>>> -; AVX1-NEXT:    vmovapd {{.*#+}} xmm2 = [4.503600e+15,1.934281e+25]
>>>> -; AVX1-NEXT:    vsubpd %xmm2, %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vmovq %rdx, %xmm3
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm1 =
>>>> xmm3[0],xmm1[0],xmm3[1],xmm1[1]
>>>> -; AVX1-NEXT:    vsubpd %xmm2, %xmm1, %xmm1
>>>> -; AVX1-NEXT:    vhaddpd %xmm1, %xmm0, %xmm0
>>>> +; AVX1-NEXT:    vroundpd $11, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptoui <2 x double> %x to <2 x i64>
>>>>    %r = uitofp <2 x i64> %i to <2 x double>
>>>> @@ -327,106 +217,13 @@ define <4 x double> @trunc_unsigned_v4f6
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_unsigned_v4f64:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    movaps %xmm1, %xmm3
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm3 = xmm1[1],xmm3[1]
>>>> -; SSE41-NEXT:    movsd {{.*#+}} xmm2 = mem[0],zero
>>>> -; SSE41-NEXT:    movaps %xmm3, %xmm4
>>>> -; SSE41-NEXT:    subsd %xmm2, %xmm4
>>>> -; SSE41-NEXT:    cvttsd2si %xmm4, %rcx
>>>> -; SSE41-NEXT:    movabsq $-9223372036854775808, %rdx # imm =
>>>> 0x8000000000000000
>>>> -; SSE41-NEXT:    xorq %rdx, %rcx
>>>> -; SSE41-NEXT:    cvttsd2si %xmm3, %rax
>>>> -; SSE41-NEXT:    ucomisd %xmm2, %xmm3
>>>> -; SSE41-NEXT:    cmovaeq %rcx, %rax
>>>> -; SSE41-NEXT:    movaps %xmm1, %xmm3
>>>> -; SSE41-NEXT:    subsd %xmm2, %xmm3
>>>> -; SSE41-NEXT:    cvttsd2si %xmm3, %rsi
>>>> -; SSE41-NEXT:    xorq %rdx, %rsi
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rcx
>>>> -; SSE41-NEXT:    ucomisd %xmm2, %xmm1
>>>> -; SSE41-NEXT:    cmovaeq %rsi, %rcx
>>>> -; SSE41-NEXT:    movaps %xmm0, %xmm1
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm1 = xmm0[1],xmm1[1]
>>>> -; SSE41-NEXT:    movaps %xmm1, %xmm3
>>>> -; SSE41-NEXT:    subsd %xmm2, %xmm3
>>>> -; SSE41-NEXT:    cvttsd2si %xmm3, %rsi
>>>> -; SSE41-NEXT:    xorq %rdx, %rsi
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rdi
>>>> -; SSE41-NEXT:    ucomisd %xmm2, %xmm1
>>>> -; SSE41-NEXT:    cmovaeq %rsi, %rdi
>>>> -; SSE41-NEXT:    movaps %xmm0, %xmm1
>>>> -; SSE41-NEXT:    subsd %xmm2, %xmm1
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rsi
>>>> -; SSE41-NEXT:    xorq %rdx, %rsi
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rdx
>>>> -; SSE41-NEXT:    ucomisd %xmm2, %xmm0
>>>> -; SSE41-NEXT:    cmovaeq %rsi, %rdx
>>>> -; SSE41-NEXT:    movq %rdx, %xmm0
>>>> -; SSE41-NEXT:    movdqa {{.*#+}} xmm2 = [1127219200,1160773632,0,0]
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm0 =
>>>> xmm0[0],xmm2[0],xmm0[1],xmm2[1]
>>>> -; SSE41-NEXT:    movapd {{.*#+}} xmm3 = [4.503600e+15,1.934281e+25]
>>>> -; SSE41-NEXT:    subpd %xmm3, %xmm0
>>>> -; SSE41-NEXT:    movq %rdi, %xmm1
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm1 =
>>>> xmm1[0],xmm2[0],xmm1[1],xmm2[1]
>>>> -; SSE41-NEXT:    subpd %xmm3, %xmm1
>>>> -; SSE41-NEXT:    haddpd %xmm1, %xmm0
>>>> -; SSE41-NEXT:    movq %rcx, %xmm1
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm1 =
>>>> xmm1[0],xmm2[0],xmm1[1],xmm2[1]
>>>> -; SSE41-NEXT:    subpd %xmm3, %xmm1
>>>> -; SSE41-NEXT:    movq %rax, %xmm4
>>>> -; SSE41-NEXT:    punpckldq {{.*#+}} xmm4 =
>>>> xmm4[0],xmm2[0],xmm4[1],xmm2[1]
>>>> -; SSE41-NEXT:    subpd %xmm3, %xmm4
>>>> -; SSE41-NEXT:    haddpd %xmm4, %xmm1
>>>> +; SSE41-NEXT:    roundpd $11, %xmm0, %xmm0
>>>> +; SSE41-NEXT:    roundpd $11, %xmm1, %xmm1
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_unsigned_v4f64:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm2 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vmovsd {{.*#+}} xmm1 = mem[0],zero
>>>> -; AVX1-NEXT:    vsubsd %xmm1, %xmm2, %xmm3
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm3, %rcx
>>>> -; AVX1-NEXT:    movabsq $-9223372036854775808, %rdx # imm =
>>>> 0x8000000000000000
>>>> -; AVX1-NEXT:    xorq %rdx, %rcx
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm2, %rax
>>>> -; AVX1-NEXT:    vucomisd %xmm1, %xmm2
>>>> -; AVX1-NEXT:    cmovaeq %rcx, %rax
>>>> -; AVX1-NEXT:    vsubsd %xmm1, %xmm0, %xmm2
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm2, %rcx
>>>> -; AVX1-NEXT:    xorq %rdx, %rcx
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rsi
>>>> -; AVX1-NEXT:    vucomisd %xmm1, %xmm0
>>>> -; AVX1-NEXT:    cmovaeq %rcx, %rsi
>>>> -; AVX1-NEXT:    vextractf128 $1, %ymm0, %xmm0
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm2 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vsubsd %xmm1, %xmm2, %xmm3
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm3, %rcx
>>>> -; AVX1-NEXT:    xorq %rdx, %rcx
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm2, %rdi
>>>> -; AVX1-NEXT:    vucomisd %xmm1, %xmm2
>>>> -; AVX1-NEXT:    cmovaeq %rcx, %rdi
>>>> -; AVX1-NEXT:    vsubsd %xmm1, %xmm0, %xmm2
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm2, %rcx
>>>> -; AVX1-NEXT:    xorq %rdx, %rcx
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rdx
>>>> -; AVX1-NEXT:    vucomisd %xmm1, %xmm0
>>>> -; AVX1-NEXT:    cmovaeq %rcx, %rdx
>>>> -; AVX1-NEXT:    vmovq %rdx, %xmm0
>>>> -; AVX1-NEXT:    vmovdqa {{.*#+}} xmm1 = [1127219200,1160773632,0,0]
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm0 =
>>>> xmm0[0],xmm1[0],xmm0[1],xmm1[1]
>>>> -; AVX1-NEXT:    vmovapd {{.*#+}} xmm2 = [4.503600e+15,1.934281e+25]
>>>> -; AVX1-NEXT:    vsubpd %xmm2, %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vmovq %rdi, %xmm3
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm3 =
>>>> xmm3[0],xmm1[0],xmm3[1],xmm1[1]
>>>> -; AVX1-NEXT:    vsubpd %xmm2, %xmm3, %xmm3
>>>> -; AVX1-NEXT:    vhaddpd %xmm3, %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vmovq %rsi, %xmm3
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm3 =
>>>> xmm3[0],xmm1[0],xmm3[1],xmm1[1]
>>>> -; AVX1-NEXT:    vsubpd %xmm2, %xmm3, %xmm3
>>>> -; AVX1-NEXT:    vmovq %rax, %xmm4
>>>> -; AVX1-NEXT:    vpunpckldq {{.*#+}} xmm1 =
>>>> xmm4[0],xmm1[0],xmm4[1],xmm1[1]
>>>> -; AVX1-NEXT:    vsubpd %xmm2, %xmm1, %xmm1
>>>> -; AVX1-NEXT:    vhaddpd %xmm1, %xmm3, %xmm1
>>>> -; AVX1-NEXT:    vinsertf128 $1, %xmm0, %ymm1, %ymm0
>>>> +; AVX1-NEXT:    vroundpd $11, %ymm0, %ymm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptoui <4 x double> %x to <4 x i64>
>>>>    %r = uitofp <4 x i64> %i to <4 x double>
>>>> @@ -443,15 +240,12 @@ define float @trunc_signed_f32(float %x)
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_signed_f32:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    cvttss2si %xmm0, %eax
>>>> -; SSE41-NEXT:    xorps %xmm0, %xmm0
>>>> -; SSE41-NEXT:    cvtsi2ssl %eax, %xmm0
>>>> +; SSE41-NEXT:    roundss $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_signed_f32:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vcvttss2si %xmm0, %eax
>>>> -; AVX1-NEXT:    vcvtsi2ssl %eax, %xmm1, %xmm0
>>>> +; AVX1-NEXT:    vroundss $11, %xmm0, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptosi float %x to i32
>>>>    %r = sitofp i32 %i to float
>>>> @@ -468,15 +262,12 @@ define double @trunc_signed_f64(double %
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_signed_f64:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rax
>>>> -; SSE41-NEXT:    xorps %xmm0, %xmm0
>>>> -; SSE41-NEXT:    cvtsi2sdq %rax, %xmm0
>>>> +; SSE41-NEXT:    roundsd $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_signed_f64:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rax
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rax, %xmm1, %xmm0
>>>> +; AVX1-NEXT:    vroundsd $11, %xmm0, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptosi double %x to i64
>>>>    %r = sitofp i64 %i to double
>>>> @@ -492,14 +283,12 @@ define <4 x float> @trunc_signed_v4f32(<
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_signed_v4f32:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    cvttps2dq %xmm0, %xmm0
>>>> -; SSE41-NEXT:    cvtdq2ps %xmm0, %xmm0
>>>> +; SSE41-NEXT:    roundps $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_signed_v4f32:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vcvttps2dq %xmm0, %xmm0
>>>> -; AVX1-NEXT:    vcvtdq2ps %xmm0, %xmm0
>>>> +; AVX1-NEXT:    vroundps $11, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptosi <4 x float> %x to <4 x i32>
>>>>    %r = sitofp <4 x i32> %i to <4 x float>
>>>> @@ -520,23 +309,12 @@ define <2 x double> @trunc_signed_v2f64(
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_signed_v2f64:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rax
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm0 = xmm0[1,1]
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rcx
>>>> -; SSE41-NEXT:    xorps %xmm0, %xmm0
>>>> -; SSE41-NEXT:    cvtsi2sdq %rax, %xmm0
>>>> -; SSE41-NEXT:    cvtsi2sdq %rcx, %xmm1
>>>> -; SSE41-NEXT:    movlhps {{.*#+}} xmm0 = xmm0[0],xmm1[0]
>>>> +; SSE41-NEXT:    roundpd $11, %xmm0, %xmm0
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_signed_v2f64:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm1 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm1, %rax
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rcx
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rcx, %xmm2, %xmm0
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rax, %xmm2, %xmm1
>>>> -; AVX1-NEXT:    vmovlhps {{.*#+}} xmm0 = xmm0[0],xmm1[0]
>>>> +; AVX1-NEXT:    vroundpd $11, %xmm0, %xmm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptosi <2 x double> %x to <2 x i64>
>>>>    %r = sitofp <2 x i64> %i to <2 x double>
>>>> @@ -565,39 +343,13 @@ define <4 x double> @trunc_signed_v4f64(
>>>>  ;
>>>>  ; SSE41-LABEL: trunc_signed_v4f64:
>>>>  ; SSE41:       # %bb.0:
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rax
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm1 = xmm1[1,1]
>>>> -; SSE41-NEXT:    cvttsd2si %xmm1, %rcx
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rdx
>>>> -; SSE41-NEXT:    movhlps {{.*#+}} xmm0 = xmm0[1,1]
>>>> -; SSE41-NEXT:    cvttsd2si %xmm0, %rsi
>>>> -; SSE41-NEXT:    xorps %xmm0, %xmm0
>>>> -; SSE41-NEXT:    cvtsi2sdq %rdx, %xmm0
>>>> -; SSE41-NEXT:    xorps %xmm1, %xmm1
>>>> -; SSE41-NEXT:    cvtsi2sdq %rsi, %xmm1
>>>> -; SSE41-NEXT:    movlhps {{.*#+}} xmm0 = xmm0[0],xmm1[0]
>>>> -; SSE41-NEXT:    xorps %xmm1, %xmm1
>>>> -; SSE41-NEXT:    cvtsi2sdq %rax, %xmm1
>>>> -; SSE41-NEXT:    cvtsi2sdq %rcx, %xmm2
>>>> -; SSE41-NEXT:    movlhps {{.*#+}} xmm1 = xmm1[0],xmm2[0]
>>>> +; SSE41-NEXT:    roundpd $11, %xmm0, %xmm0
>>>> +; SSE41-NEXT:    roundpd $11, %xmm1, %xmm1
>>>>  ; SSE41-NEXT:    retq
>>>>  ;
>>>>  ; AVX1-LABEL: trunc_signed_v4f64:
>>>>  ; AVX1:       # %bb.0:
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm1 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm1, %rax
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rcx
>>>> -; AVX1-NEXT:    vextractf128 $1, %ymm0, %xmm0
>>>> -; AVX1-NEXT:    vpermilpd {{.*#+}} xmm1 = xmm0[1,0]
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm1, %rdx
>>>> -; AVX1-NEXT:    vcvttsd2si %xmm0, %rsi
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rsi, %xmm2, %xmm0
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rdx, %xmm2, %xmm1
>>>> -; AVX1-NEXT:    vmovlhps {{.*#+}} xmm0 = xmm0[0],xmm1[0]
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rcx, %xmm2, %xmm1
>>>> -; AVX1-NEXT:    vcvtsi2sdq %rax, %xmm2, %xmm2
>>>> -; AVX1-NEXT:    vmovlhps {{.*#+}} xmm1 = xmm1[0],xmm2[0]
>>>> -; AVX1-NEXT:    vinsertf128 $1, %xmm0, %ymm1, %ymm0
>>>> +; AVX1-NEXT:    vroundpd $11, %ymm0, %ymm0
>>>>  ; AVX1-NEXT:    retq
>>>>    %i = fptosi <4 x double> %x to <4 x i64>
>>>>    %r = sitofp <4 x i64> %i to <4 x double>
>>>>
>>>>
>>>> _______________________________________________
>>>> llvm-commits mailing list
>>>> llvm-commits at lists.llvm.org
>>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>>>
>>>
>> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180425/682bfb62/attachment.html>


More information about the llvm-commits mailing list