Skip to content
Snippets Groups Projects
  • Alex Bennée's avatar
    1d3daf95
    softfloat: use QEMU_FLATTEN to avoid mistaken isra inlining · 1d3daf95
    Alex Bennée authored
    
    Balton discovered that asserts for the extract/deposit calls had a
    significant impact on a lame benchmark on qemu-ppc. Replicating with:
    
      ./qemu-ppc64 ~/lsrc/tests/lame.git-svn/builds/ppc64/frontend/lame \
        -h pts-trondheim-3.wav pts-trondheim-3.mp3
    
    showed up the pack/unpack routines not eliding the assert checks as it
    should have done causing them to prominently figure in the profile:
    
      11.44%  qemu-ppc64  qemu-ppc64               [.] unpack_raw64.isra.0
      11.03%  qemu-ppc64  qemu-ppc64               [.] parts64_uncanon_normal
       8.26%  qemu-ppc64  qemu-ppc64               [.] helper_compute_fprf_float64
       6.75%  qemu-ppc64  qemu-ppc64               [.] do_float_check_status
       5.34%  qemu-ppc64  qemu-ppc64               [.] parts64_muladd
       4.75%  qemu-ppc64  qemu-ppc64               [.] pack_raw64.isra.0
       4.38%  qemu-ppc64  qemu-ppc64               [.] parts64_canonicalize
       3.62%  qemu-ppc64  qemu-ppc64               [.] float64r32_round_pack_canonical
    
    After this patch the same test runs 31 seconds faster with a profile
    where the generated code dominates more:
    
    +   14.12%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000619420
    +   13.30%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000616850
    +   12.58%    12.19%  qemu-ppc64  qemu-ppc64               [.] parts64_uncanon_normal
    +   10.62%     0.00%  qemu-ppc64  [unknown]                [.] 0x000000400061bf70
    +    9.91%     9.73%  qemu-ppc64  qemu-ppc64               [.] helper_compute_fprf_float64
    +    7.84%     7.82%  qemu-ppc64  qemu-ppc64               [.] do_float_check_status
    +    6.47%     5.78%  qemu-ppc64  qemu-ppc64               [.] parts64_canonicalize.constprop.0
    +    6.46%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000620130
    +    6.42%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000619400
    +    6.17%     6.04%  qemu-ppc64  qemu-ppc64               [.] parts64_muladd
    +    5.85%     0.00%  qemu-ppc64  [unknown]                [.] 0x00000040006167e0
    +    5.74%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000b693fcffffd3
    +    5.45%     4.78%  qemu-ppc64  qemu-ppc64               [.] float64r32_round_pack_canonical
    
    Suggested-by: default avatarRichard Henderson <richard.henderson@linaro.org>
    Message-Id: <ec9cfe5a-d5f2-466d-34dc-c35817e7e010@linaro.org>
    [AJB: Patchified rth's suggestion]
    Signed-off-by: default avatarAlex Bennée <alex.bennee@linaro.org>
    Cc: BALATON Zoltan <balaton@eik.bme.hu>
    Reviewed-by: default avatarRichard Henderson <richard.henderson@linaro.org>
    Reviewed-by: default avatarPhilippe Mathieu-Daudé <philmd@linaro.org>
    Tested-by: default avatarBALATON Zoltan <balaton@eik.bme.hu>
    Message-Id: <20230523131107.3680641-1-alex.bennee@linaro.org>
    Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
    1d3daf95
    History
    softfloat: use QEMU_FLATTEN to avoid mistaken isra inlining
    Alex Bennée authored
    
    Balton discovered that asserts for the extract/deposit calls had a
    significant impact on a lame benchmark on qemu-ppc. Replicating with:
    
      ./qemu-ppc64 ~/lsrc/tests/lame.git-svn/builds/ppc64/frontend/lame \
        -h pts-trondheim-3.wav pts-trondheim-3.mp3
    
    showed up the pack/unpack routines not eliding the assert checks as it
    should have done causing them to prominently figure in the profile:
    
      11.44%  qemu-ppc64  qemu-ppc64               [.] unpack_raw64.isra.0
      11.03%  qemu-ppc64  qemu-ppc64               [.] parts64_uncanon_normal
       8.26%  qemu-ppc64  qemu-ppc64               [.] helper_compute_fprf_float64
       6.75%  qemu-ppc64  qemu-ppc64               [.] do_float_check_status
       5.34%  qemu-ppc64  qemu-ppc64               [.] parts64_muladd
       4.75%  qemu-ppc64  qemu-ppc64               [.] pack_raw64.isra.0
       4.38%  qemu-ppc64  qemu-ppc64               [.] parts64_canonicalize
       3.62%  qemu-ppc64  qemu-ppc64               [.] float64r32_round_pack_canonical
    
    After this patch the same test runs 31 seconds faster with a profile
    where the generated code dominates more:
    
    +   14.12%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000619420
    +   13.30%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000616850
    +   12.58%    12.19%  qemu-ppc64  qemu-ppc64               [.] parts64_uncanon_normal
    +   10.62%     0.00%  qemu-ppc64  [unknown]                [.] 0x000000400061bf70
    +    9.91%     9.73%  qemu-ppc64  qemu-ppc64               [.] helper_compute_fprf_float64
    +    7.84%     7.82%  qemu-ppc64  qemu-ppc64               [.] do_float_check_status
    +    6.47%     5.78%  qemu-ppc64  qemu-ppc64               [.] parts64_canonicalize.constprop.0
    +    6.46%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000620130
    +    6.42%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000004000619400
    +    6.17%     6.04%  qemu-ppc64  qemu-ppc64               [.] parts64_muladd
    +    5.85%     0.00%  qemu-ppc64  [unknown]                [.] 0x00000040006167e0
    +    5.74%     0.00%  qemu-ppc64  [unknown]                [.] 0x0000b693fcffffd3
    +    5.45%     4.78%  qemu-ppc64  qemu-ppc64               [.] float64r32_round_pack_canonical
    
    Suggested-by: default avatarRichard Henderson <richard.henderson@linaro.org>
    Message-Id: <ec9cfe5a-d5f2-466d-34dc-c35817e7e010@linaro.org>
    [AJB: Patchified rth's suggestion]
    Signed-off-by: default avatarAlex Bennée <alex.bennee@linaro.org>
    Cc: BALATON Zoltan <balaton@eik.bme.hu>
    Reviewed-by: default avatarRichard Henderson <richard.henderson@linaro.org>
    Reviewed-by: default avatarPhilippe Mathieu-Daudé <philmd@linaro.org>
    Tested-by: default avatarBALATON Zoltan <balaton@eik.bme.hu>
    Message-Id: <20230523131107.3680641-1-alex.bennee@linaro.org>
    Signed-off-by: default avatarRichard Henderson <richard.henderson@linaro.org>
Code owners
Assign users and groups as approvers for specific file changes. Learn more.