[llvm] r342916 - [X86] Remove shift/rotate by CL memory (RMW) overrides

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 24 13:24:17 PDT 2018


I'm not sure I understand why you have to bump the uops by +1. Is this
because WriteRMW isn't listed as 2 uops?
~Craig


On Mon, Sep 24, 2018 at 1:13 PM Simon Pilgrim via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: rksimon
> Date: Mon Sep 24 13:11:50 2018
> New Revision: 342916
>
> URL: http://llvm.org/viewvc/llvm-project?rev=342916&view=rev
> Log:
> [X86] Remove shift/rotate by CL memory (RMW) overrides
>
> The uops are slightly different to the register variant, so requires a
> +1uop tweak
>
> Modified:
>     llvm/trunk/lib/Target/X86/X86SchedBroadwell.td
>     llvm/trunk/lib/Target/X86/X86SchedHaswell.td
>     llvm/trunk/lib/Target/X86/X86SchedSandyBridge.td
>     llvm/trunk/lib/Target/X86/X86SchedSkylakeClient.td
>     llvm/trunk/lib/Target/X86/X86SchedSkylakeServer.td
>     llvm/trunk/test/CodeGen/X86/bmi2-schedule.ll
>     llvm/trunk/test/tools/llvm-mca/X86/Generic/resources-bmi2.s
>
> Modified: llvm/trunk/lib/Target/X86/X86SchedBroadwell.td
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86SchedBroadwell.td?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86SchedBroadwell.td (original)
> +++ llvm/trunk/lib/Target/X86/X86SchedBroadwell.td Mon Sep 24 13:11:50 2018
> @@ -84,7 +84,7 @@ def : ReadAdvance<ReadAfterLd, 5>;
>  multiclass BWWriteResPair<X86FoldableSchedWrite SchedRW,
>                            list<ProcResourceKind> ExePorts,
>                            int Lat, list<int> Res = [1], int UOps = 1,
> -                          int LoadLat = 5> {
> +                          int LoadLat = 5, int LoadUOps = 1> {
>    // Register variant is using a single cycle on ExePort.
>    def : WriteRes<SchedRW, ExePorts> {
>      let Latency = Lat;
> @@ -97,7 +97,7 @@ multiclass BWWriteResPair<X86FoldableSch
>    def : WriteRes<SchedRW.Folded, !listconcat([BWPort23], ExePorts)> {
>      let Latency = !add(Lat, LoadLat);
>      let ResourceCycles = !listconcat([1], Res);
> -    let NumMicroOps = !add(UOps, 1);
> +    let NumMicroOps = !add(UOps, LoadUOps);
>    }
>  }
>
> @@ -173,9 +173,9 @@ defm : BWWriteResPair<WritePOPCNT,
>
>  // Integer shifts and rotates.
>  defm : BWWriteResPair<WriteShift,    [BWPort06],  1>;
> -defm : BWWriteResPair<WriteShiftCL,  [BWPort06,BWPort0156],  3, [2,1], 3>;
> +defm : BWWriteResPair<WriteShiftCL,  [BWPort06,BWPort0156], 3, [2,1], 3,
> 5, 2>;
>  defm : BWWriteResPair<WriteRotate,   [BWPort06],  2, [2], 2>;
> -defm : BWWriteResPair<WriteRotateCL, [BWPort06,BWPort0156],  3, [2,1], 3>;
> +defm : BWWriteResPair<WriteRotateCL, [BWPort06,BWPort0156], 3, [2,1], 3,
> 5, 2>;
>
>  // SHLD/SHRD.
>  defm : X86WriteRes<WriteSHDrri, [BWPort1], 3, [1], 1>;
> @@ -1167,11 +1167,6 @@ def BWWriteResGroup100 : SchedWriteRes<[
>    let ResourceCycles = [1,1,1,2,1];
>  }
>  def : SchedAlias<WriteADCRMW, BWWriteResGroup100>;
> -def: InstRW<[BWWriteResGroup100], (instregex "ROL(8|16|32|64)mCL",
> -                                             "ROR(8|16|32|64)mCL",
> -                                             "SAR(8|16|32|64)mCL",
> -                                             "SHL(8|16|32|64)mCL",
> -                                             "SHR(8|16|32|64)mCL")>;
>
>  def BWWriteResGroup101 : SchedWriteRes<[BWPort1,BWPort23]> {
>    let Latency = 9;
>
> Modified: llvm/trunk/lib/Target/X86/X86SchedHaswell.td
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86SchedHaswell.td?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86SchedHaswell.td (original)
> +++ llvm/trunk/lib/Target/X86/X86SchedHaswell.td Mon Sep 24 13:11:50 2018
> @@ -89,7 +89,7 @@ def : ReadAdvance<ReadAfterLd, 5>;
>  multiclass HWWriteResPair<X86FoldableSchedWrite SchedRW,
>                            list<ProcResourceKind> ExePorts,
>                            int Lat, list<int> Res = [1], int UOps = 1,
> -                          int LoadLat = 5> {
> +                          int LoadLat = 5, int LoadUOps = 1> {
>    // Register variant is using a single cycle on ExePort.
>    def : WriteRes<SchedRW, ExePorts> {
>      let Latency = Lat;
> @@ -102,7 +102,7 @@ multiclass HWWriteResPair<X86FoldableSch
>    def : WriteRes<SchedRW.Folded, !listconcat([HWPort23], ExePorts)> {
>      let Latency = !add(Lat, LoadLat);
>      let ResourceCycles = !listconcat([1], Res);
> -    let NumMicroOps = !add(UOps, 1);
> +    let NumMicroOps = !add(UOps, LoadUOps);
>    }
>  }
>
> @@ -144,9 +144,9 @@ defm : X86WriteRes<WriteXCHG, [HWPort015
>
>  // Integer shifts and rotates.
>  defm : HWWriteResPair<WriteShift,    [HWPort06],  1>;
> -defm : HWWriteResPair<WriteShiftCL,  [HWPort06, HWPort0156],  3, [2,1],
> 3>;
> +defm : HWWriteResPair<WriteShiftCL,  [HWPort06, HWPort0156], 3, [2,1], 3,
> 6, 2>;
>  defm : HWWriteResPair<WriteRotate,   [HWPort06],  2, [2], 2>;
> -defm : HWWriteResPair<WriteRotateCL, [HWPort06, HWPort0156],  3, [2,1],
> 3>;
> +defm : HWWriteResPair<WriteRotateCL, [HWPort06, HWPort0156], 3, [2,1], 3,
> 6, 2>;
>
>  // SHLD/SHRD.
>  defm : X86WriteRes<WriteSHDrri, [HWPort1], 3, [1], 1>;
> @@ -1306,11 +1306,6 @@ def HWWriteResGroup69 : SchedWriteRes<[H
>    let NumMicroOps = 6;
>    let ResourceCycles = [1,1,1,2,1];
>  }
> -def: InstRW<[HWWriteResGroup69], (instregex "ROL(8|16|32|64)mCL",
> -                                            "ROR(8|16|32|64)mCL",
> -                                            "SAR(8|16|32|64)mCL",
> -                                            "SHL(8|16|32|64)mCL",
> -                                            "SHR(8|16|32|64)mCL")>;
>  def: SchedAlias<WriteADCRMW, HWWriteResGroup69>;
>
>  def HWWriteResGroup70 : SchedWriteRes<[HWPort0,HWPort1]> {
>
> Modified: llvm/trunk/lib/Target/X86/X86SchedSandyBridge.td
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86SchedSandyBridge.td?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86SchedSandyBridge.td (original)
> +++ llvm/trunk/lib/Target/X86/X86SchedSandyBridge.td Mon Sep 24 13:11:50
> 2018
> @@ -79,7 +79,7 @@ def : ReadAdvance<ReadAfterLd, 5>;
>  multiclass SBWriteResPair<X86FoldableSchedWrite SchedRW,
>                            list<ProcResourceKind> ExePorts,
>                            int Lat, list<int> Res = [1], int UOps = 1,
> -                          int LoadLat = 5> {
> +                          int LoadLat = 5, int LoadUOps = 1> {
>    // Register variant is using a single cycle on ExePort.
>    def : WriteRes<SchedRW, ExePorts> {
>      let Latency = Lat;
> @@ -92,7 +92,7 @@ multiclass SBWriteResPair<X86FoldableSch
>    def : WriteRes<SchedRW.Folded, !listconcat([SBPort23], ExePorts)> {
>      let Latency = !add(Lat, LoadLat);
>      let ResourceCycles = !listconcat([1], Res);
> -    let NumMicroOps = !add(UOps, 1);
> +    let NumMicroOps = !add(UOps, LoadUOps);
>    }
>  }
>
> @@ -144,10 +144,10 @@ defm : X86WriteRes<WriteSHDrrcl,[SBPort0
>  defm : X86WriteRes<WriteSHDmri, [SBPort4,SBPort23,SBPort05,SBPort015], 8,
> [1, 2, 1, 1], 5>;
>  defm : X86WriteRes<WriteSHDmrcl,[SBPort4,SBPort23,SBPort05,SBPort015],
> 10, [1, 2, 3, 1], 7>;
>
> -defm : SBWriteResPair<WriteShift,    [SBPort05],  1>;
> -defm : SBWriteResPair<WriteShiftCL,  [SBPort05],  3, [3], 3>;
> -defm : SBWriteResPair<WriteRotate,   [SBPort05],  2, [2], 2>;
> -defm : SBWriteResPair<WriteRotateCL, [SBPort05],  3, [3], 3>;
> +defm : SBWriteResPair<WriteShift,    [SBPort05], 1, [1], 1, 6, 2>;
> +defm : SBWriteResPair<WriteShiftCL,  [SBPort05], 3, [3], 3, 6, 2>;
> +defm : SBWriteResPair<WriteRotate,   [SBPort05], 2, [2], 2, 6, 2>;
> +defm : SBWriteResPair<WriteRotateCL, [SBPort05], 3, [3], 3, 6, 2>;
>
>  defm : SBWriteResPair<WriteJump,  [SBPort5],   1>;
>  defm : SBWriteResPair<WriteCRC32, [SBPort1],   3, [1], 1, 5>;
> @@ -924,10 +924,7 @@ def SBWriteResGroup69 : SchedWriteRes<[S
>  }
>  def: InstRW<[SBWriteResGroup69], (instregex "BTC(16|32|64)mi8",
>                                              "BTR(16|32|64)mi8",
> -                                            "BTS(16|32|64)mi8",
> -                                            "SAR(8|16|32|64)m(1|i)",
> -                                            "SHL(8|16|32|64)m(1|i)",
> -                                            "SHR(8|16|32|64)m(1|i)")>;
> +                                            "BTS(16|32|64)mi8")>;
>
>  def SBWriteResGroup77 : SchedWriteRes<[SBPort0,SBPort1,SBPort23]> {
>    let Latency = 8;
> @@ -960,14 +957,6 @@ def SBWriteResGroup84 : SchedWriteRes<[S
>  }
>  def: InstRW<[SBWriteResGroup84], (instrs FLDCW16m)>;
>
> -def SBWriteResGroup85 : SchedWriteRes<[SBPort4,SBPort23,SBPort05]> {
> -  let Latency = 8;
> -  let NumMicroOps = 5;
> -  let ResourceCycles = [1,2,2];
> -}
> -def: InstRW<[SBWriteResGroup85], (instregex "ROL(8|16|32|64)m(1|i)",
> -                                            "ROR(8|16|32|64)m(1|i)")>;
> -
>  def SBWriteResGroup86 : SchedWriteRes<[SBPort4,SBPort23,SBPort015]> {
>    let Latency = 8;
>    let NumMicroOps = 5;
> @@ -1005,17 +994,6 @@ def SBWriteResGroup97 : SchedWriteRes<[S
>  def: InstRW<[SBWriteResGroup97], (instregex "IST_F(16|32)m",
>                                              "IST_FP(16|32|64)m")>;
>
> -def SBWriteResGroup97_2 : SchedWriteRes<[SBPort4,SBPort23,SBPort05]> {
> -  let Latency = 9;
> -  let NumMicroOps = 6;
> -  let ResourceCycles = [1,2,3];
> -}
> -def: InstRW<[SBWriteResGroup97_2], (instregex "ROL(8|16|32|64)mCL",
> -                                              "ROR(8|16|32|64)mCL",
> -                                              "SAR(8|16|32|64)mCL",
> -                                              "SHL(8|16|32|64)mCL",
> -                                              "SHR(8|16|32|64)mCL")>;
> -
>  def SBWriteResGroup98 : SchedWriteRes<[SBPort4,SBPort23,SBPort015]> {
>    let Latency = 9;
>    let NumMicroOps = 6;
>
> Modified: llvm/trunk/lib/Target/X86/X86SchedSkylakeClient.td
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86SchedSkylakeClient.td?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86SchedSkylakeClient.td (original)
> +++ llvm/trunk/lib/Target/X86/X86SchedSkylakeClient.td Mon Sep 24 13:11:50
> 2018
> @@ -83,7 +83,7 @@ def : ReadAdvance<ReadAfterLd, 5>;
>  multiclass SKLWriteResPair<X86FoldableSchedWrite SchedRW,
>                            list<ProcResourceKind> ExePorts,
>                            int Lat, list<int> Res = [1], int UOps = 1,
> -                          int LoadLat = 5> {
> +                          int LoadLat = 5, int LoadUOps = 1> {
>    // Register variant is using a single cycle on ExePort.
>    def : WriteRes<SchedRW, ExePorts> {
>      let Latency = Lat;
> @@ -96,7 +96,7 @@ multiclass SKLWriteResPair<X86FoldableSc
>    def : WriteRes<SchedRW.Folded, !listconcat([SKLPort23], ExePorts)> {
>      let Latency = !add(Lat, LoadLat);
>      let ResourceCycles = !listconcat([1], Res);
> -    let NumMicroOps = !add(UOps, 1);
> +    let NumMicroOps = !add(UOps, LoadUOps);
>    }
>  }
>
> @@ -169,10 +169,10 @@ defm : SKLWriteResPair<WriteTZCNT,
>  defm : SKLWriteResPair<WritePOPCNT,         [SKLPort1], 3>;
>
>  // Integer shifts and rotates.
> -defm : SKLWriteResPair<WriteShift,    [SKLPort06],  1>;
> -defm : SKLWriteResPair<WriteShiftCL,  [SKLPort06],  3, [3], 3>;
> -defm : SKLWriteResPair<WriteRotate,   [SKLPort06],  2, [2], 2>;
> -defm : SKLWriteResPair<WriteRotateCL, [SKLPort06],  3, [3], 3>;
> +defm : SKLWriteResPair<WriteShift,    [SKLPort06], 1>;
> +defm : SKLWriteResPair<WriteShiftCL,  [SKLPort06], 3, [3], 3, 5, 2>;
> +defm : SKLWriteResPair<WriteRotate,   [SKLPort06], 2, [2], 2>;
> +defm : SKLWriteResPair<WriteRotateCL, [SKLPort06], 3, [3], 3, 5, 2>;
>
>  // SHLD/SHRD.
>  defm : X86WriteRes<WriteSHDrri, [SKLPort1], 3, [1], 1>;
> @@ -1246,17 +1246,6 @@ def SKLWriteResGroup116 : SchedWriteRes<
>  def: InstRW<[SKLWriteResGroup116], (instregex "RCL(8|16|32|64)m(1|i)",
>                                                "RCR(8|16|32|64)m(1|i)")>;
>
> -def SKLWriteResGroup117 :
> SchedWriteRes<[SKLPort4,SKLPort23,SKLPort237,SKLPort06]> {
> -  let Latency = 8;
> -  let NumMicroOps = 6;
> -  let ResourceCycles = [1,1,1,3];
> -}
> -def: InstRW<[SKLWriteResGroup117], (instregex "ROL(8|16|32|64)mCL",
> -                                              "ROR(8|16|32|64)mCL",
> -                                              "SAR(8|16|32|64)mCL",
> -                                              "SHL(8|16|32|64)mCL",
> -                                              "SHR(8|16|32|64)mCL")>;
> -
>  def SKLWriteResGroup119 :
> SchedWriteRes<[SKLPort4,SKLPort23,SKLPort237,SKLPort06,SKLPort0156]> {
>    let Latency = 8;
>    let NumMicroOps = 6;
>
> Modified: llvm/trunk/lib/Target/X86/X86SchedSkylakeServer.td
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86SchedSkylakeServer.td?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/lib/Target/X86/X86SchedSkylakeServer.td (original)
> +++ llvm/trunk/lib/Target/X86/X86SchedSkylakeServer.td Mon Sep 24 13:11:50
> 2018
> @@ -83,7 +83,7 @@ def : ReadAdvance<ReadAfterLd, 5>;
>  multiclass SKXWriteResPair<X86FoldableSchedWrite SchedRW,
>                            list<ProcResourceKind> ExePorts,
>                            int Lat, list<int> Res = [1], int UOps = 1,
> -                          int LoadLat = 5> {
> +                          int LoadLat = 5, int LoadUOps = 1> {
>    // Register variant is using a single cycle on ExePort.
>    def : WriteRes<SchedRW, ExePorts> {
>      let Latency = Lat;
> @@ -96,7 +96,7 @@ multiclass SKXWriteResPair<X86FoldableSc
>    def : WriteRes<SchedRW.Folded, !listconcat([SKXPort23], ExePorts)> {
>      let Latency = !add(Lat, LoadLat);
>      let ResourceCycles = !listconcat([1], Res);
> -    let NumMicroOps = !add(UOps, 1);
> +    let NumMicroOps = !add(UOps, LoadUOps);
>    }
>  }
>
> @@ -163,10 +163,10 @@ def  : WriteRes<WriteLAHFSAHF, [SKXPort0
>  def  : WriteRes<WriteBitTest,  [SKXPort06]>; //
>
>  // Integer shifts and rotates.
> -defm : SKXWriteResPair<WriteShift,    [SKXPort06],  1>;
> -defm : SKXWriteResPair<WriteShiftCL,  [SKXPort06],  3, [3], 3>;
> -defm : SKXWriteResPair<WriteRotate,   [SKXPort06],  2, [2], 2>;
> -defm : SKXWriteResPair<WriteRotateCL, [SKXPort06],  3, [3], 3>;
> +defm : SKXWriteResPair<WriteShift,    [SKXPort06], 1>;
> +defm : SKXWriteResPair<WriteShiftCL,  [SKXPort06], 3, [3], 3, 5, 2>;
> +defm : SKXWriteResPair<WriteRotate,   [SKXPort06], 2, [2], 2>;
> +defm : SKXWriteResPair<WriteRotateCL, [SKXPort06], 3, [3], 3, 5, 2>;
>
>  // SHLD/SHRD.
>  defm : X86WriteRes<WriteSHDrri, [SKXPort1], 3, [1], 1>;
> @@ -1599,17 +1599,6 @@ def SKXWriteResGroup127 : SchedWriteRes<
>  def: InstRW<[SKXWriteResGroup127], (instregex "RCL(8|16|32|64)m(1|i)",
>                                                "RCR(8|16|32|64)m(1|i)")>;
>
> -def SKXWriteResGroup128 :
> SchedWriteRes<[SKXPort4,SKXPort23,SKXPort237,SKXPort06]> {
> -  let Latency = 8;
> -  let NumMicroOps = 6;
> -  let ResourceCycles = [1,1,1,3];
> -}
> -def: InstRW<[SKXWriteResGroup128], (instregex "ROL(8|16|32|64)mCL",
> -                                              "ROR(8|16|32|64)mCL",
> -                                              "SAR(8|16|32|64)mCL",
> -                                              "SHL(8|16|32|64)mCL",
> -                                              "SHR(8|16|32|64)mCL")>;
> -
>  def SKXWriteResGroup130 :
> SchedWriteRes<[SKXPort4,SKXPort23,SKXPort237,SKXPort06,SKXPort0156]> {
>    let Latency = 8;
>    let NumMicroOps = 6;
>
> Modified: llvm/trunk/test/CodeGen/X86/bmi2-schedule.ll
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/X86/bmi2-schedule.ll?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/CodeGen/X86/bmi2-schedule.ll (original)
> +++ llvm/trunk/test/CodeGen/X86/bmi2-schedule.ll Mon Sep 24 13:11:50 2018
> @@ -430,7 +430,7 @@ define i32 @test_rorx_i32(i32 %a0, i32 %
>  ; GENERIC-LABEL: test_rorx_i32:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    rorxl $5, %edi, %ecx # sched: [1:0.50]
> -; GENERIC-NEXT:    rorxl $5, (%rdx), %eax # sched: [6:0.50]
> +; GENERIC-NEXT:    rorxl $5, (%rdx), %eax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addl %ecx, %eax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -483,7 +483,7 @@ define i64 @test_rorx_i64(i64 %a0, i64 %
>  ; GENERIC-LABEL: test_rorx_i64:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    rorxq $5, %rdi, %rcx # sched: [1:0.50]
> -; GENERIC-NEXT:    rorxq $5, (%rdx), %rax # sched: [6:0.50]
> +; GENERIC-NEXT:    rorxq $5, (%rdx), %rax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addq %rcx, %rax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -536,7 +536,7 @@ define i32 @test_sarx_i32(i32 %a0, i32 %
>  ; GENERIC-LABEL: test_sarx_i32:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    sarxl %esi, %edi, %ecx # sched: [1:0.50]
> -; GENERIC-NEXT:    sarxl %esi, (%rdx), %eax # sched: [6:0.50]
> +; GENERIC-NEXT:    sarxl %esi, (%rdx), %eax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addl %ecx, %eax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -585,7 +585,7 @@ define i64 @test_sarx_i64(i64 %a0, i64 %
>  ; GENERIC-LABEL: test_sarx_i64:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    sarxq %rsi, %rdi, %rcx # sched: [1:0.50]
> -; GENERIC-NEXT:    sarxq %rsi, (%rdx), %rax # sched: [6:0.50]
> +; GENERIC-NEXT:    sarxq %rsi, (%rdx), %rax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addq %rcx, %rax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -634,7 +634,7 @@ define i32 @test_shlx_i32(i32 %a0, i32 %
>  ; GENERIC-LABEL: test_shlx_i32:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    shlxl %esi, %edi, %ecx # sched: [1:0.50]
> -; GENERIC-NEXT:    shlxl %esi, (%rdx), %eax # sched: [6:0.50]
> +; GENERIC-NEXT:    shlxl %esi, (%rdx), %eax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addl %ecx, %eax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -683,7 +683,7 @@ define i64 @test_shlx_i64(i64 %a0, i64 %
>  ; GENERIC-LABEL: test_shlx_i64:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    shlxq %rsi, %rdi, %rcx # sched: [1:0.50]
> -; GENERIC-NEXT:    shlxq %rsi, (%rdx), %rax # sched: [6:0.50]
> +; GENERIC-NEXT:    shlxq %rsi, (%rdx), %rax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addq %rcx, %rax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -732,7 +732,7 @@ define i32 @test_shrx_i32(i32 %a0, i32 %
>  ; GENERIC-LABEL: test_shrx_i32:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    shrxl %esi, %edi, %ecx # sched: [1:0.50]
> -; GENERIC-NEXT:    shrxl %esi, (%rdx), %eax # sched: [6:0.50]
> +; GENERIC-NEXT:    shrxl %esi, (%rdx), %eax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addl %ecx, %eax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
> @@ -781,7 +781,7 @@ define i64 @test_shrx_i64(i64 %a0, i64 %
>  ; GENERIC-LABEL: test_shrx_i64:
>  ; GENERIC:       # %bb.0:
>  ; GENERIC-NEXT:    shrxq %rsi, %rdi, %rcx # sched: [1:0.50]
> -; GENERIC-NEXT:    shrxq %rsi, (%rdx), %rax # sched: [6:0.50]
> +; GENERIC-NEXT:    shrxq %rsi, (%rdx), %rax # sched: [7:0.50]
>  ; GENERIC-NEXT:    addq %rcx, %rax # sched: [1:0.33]
>  ; GENERIC-NEXT:    retq # sched: [1:1.00]
>  ;
>
> Modified: llvm/trunk/test/tools/llvm-mca/X86/Generic/resources-bmi2.s
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-mca/X86/Generic/resources-bmi2.s?rev=342916&r1=342915&r2=342916&view=diff
>
> ==============================================================================
> --- llvm/trunk/test/tools/llvm-mca/X86/Generic/resources-bmi2.s (original)
> +++ llvm/trunk/test/tools/llvm-mca/X86/Generic/resources-bmi2.s Mon Sep 24
> 13:11:50 2018
> @@ -75,21 +75,21 @@ shrx        %rax, (%rbx), %rcx
>  # CHECK-NEXT:  1      1     0.33                        pextq  %rax,
> %rbx, %rcx
>  # CHECK-NEXT:  2      6     0.50    *                   pextq  (%rax),
> %rbx, %rcx
>  # CHECK-NEXT:  1      1     0.50                        rorxl  $1, %eax,
> %ecx
> -# CHECK-NEXT:  2      6     0.50    *                   rorxl  $1,
> (%rax), %ecx
> +# CHECK-NEXT:  3      7     0.50    *                   rorxl  $1,
> (%rax), %ecx
>  # CHECK-NEXT:  1      1     0.50                        rorxq  $1, %rax,
> %rcx
> -# CHECK-NEXT:  2      6     0.50    *                   rorxq  $1,
> (%rax), %rcx
> +# CHECK-NEXT:  3      7     0.50    *                   rorxq  $1,
> (%rax), %rcx
>  # CHECK-NEXT:  1      1     0.50                        sarxl  %eax,
> %ebx, %ecx
> -# CHECK-NEXT:  2      6     0.50    *                   sarxl  %eax,
> (%rbx), %ecx
> +# CHECK-NEXT:  3      7     0.50    *                   sarxl  %eax,
> (%rbx), %ecx
>  # CHECK-NEXT:  1      1     0.50                        sarxq  %rax,
> %rbx, %rcx
> -# CHECK-NEXT:  2      6     0.50    *                   sarxq  %rax,
> (%rbx), %rcx
> +# CHECK-NEXT:  3      7     0.50    *                   sarxq  %rax,
> (%rbx), %rcx
>  # CHECK-NEXT:  1      1     0.50                        shlxl  %eax,
> %ebx, %ecx
> -# CHECK-NEXT:  2      6     0.50    *                   shlxl  %eax,
> (%rbx), %ecx
> +# CHECK-NEXT:  3      7     0.50    *                   shlxl  %eax,
> (%rbx), %ecx
>  # CHECK-NEXT:  1      1     0.50                        shlxq  %rax,
> %rbx, %rcx
> -# CHECK-NEXT:  2      6     0.50    *                   shlxq  %rax,
> (%rbx), %rcx
> +# CHECK-NEXT:  3      7     0.50    *                   shlxq  %rax,
> (%rbx), %rcx
>  # CHECK-NEXT:  1      1     0.50                        shrxl  %eax,
> %ebx, %ecx
> -# CHECK-NEXT:  2      6     0.50    *                   shrxl  %eax,
> (%rbx), %ecx
> +# CHECK-NEXT:  3      7     0.50    *                   shrxl  %eax,
> (%rbx), %ecx
>  # CHECK-NEXT:  1      1     0.50                        shrxq  %rax,
> %rbx, %rcx
> -# CHECK-NEXT:  2      6     0.50    *                   shrxq  %rax,
> (%rbx), %rcx
> +# CHECK-NEXT:  3      7     0.50    *                   shrxq  %rax,
> (%rbx), %rcx
>
>  # CHECK:      Resources:
>  # CHECK-NEXT: [0]   - SBDivider
>
>
> _______________________________________________
> 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/20180924/400ba365/attachment-0001.html>


More information about the llvm-commits mailing list