[PATCH] D130895: [RISCV] Make VL choosing for a splat-like VMV based on its users

[Anton Sidorenko via Phabricator via llvm-commits]

asi-sc added a comment.

A gentle ping

In D130895#3719413 <https://reviews.llvm.org/D130895#3719413>, @craig.topper wrote:

> Sounds fine to me. But the more important case to fix is RISCVISD::*_VL nodes. I'm not sure how common it is to have splats combined with intrinsics. intrinsics come from C code where the splats should have been done with a vmv.v.x initrinsic that would already have the correct VL.

New patch generalized implementation to support RISCVISD::*_VL nodes.

Also, attaching statistics showing the number of VL shinks

                    test-suite :: External/SPEC/CINT2017rate/502.gcc_r/502.gcc_r.test 167.00
                   test-suite :: External/SPEC/CINT2017speed/602.gcc_s/602.gcc_s.test 167.00
       test-suite :: External/SPEC/CINT2017speed/623.xalancbmk_s/623.xalancbmk_s.test 164.00
        test-suite :: External/SPEC/CINT2017rate/523.xalancbmk_r/523.xalancbmk_r.test 164.00
               test-suite :: External/SPEC/CFP2017rate/510.parest_r/510.parest_r.test  75.00
                       test-suite :: MultiSource/Benchmarks/mafft/pairlocalalign.test  70.00
             test-suite :: External/SPEC/CFP2017rate/526.blender_r/526.blender_r.test  47.00
                 test-suite :: External/SPEC/CINT2017speed/625.x264_s/625.x264_s.test  31.00
                  test-suite :: External/SPEC/CINT2017rate/525.x264_r/525.x264_r.test  31.00
                        test-suite :: MultiSource/Benchmarks/7zip/7zip-benchmark.test  27.00
                            test-suite :: MultiSource/Applications/oggenc/oggenc.test  22.00
              test-suite :: MultiSource/Benchmarks/mediabench/jpeg/jpeg-6a/cjpeg.test  19.00
        test-suite :: MultiSource/Benchmarks/MiBench/consumer-jpeg/consumer-jpeg.test  19.00
                         test-suite :: MultiSource/Applications/JM/lencod/lencod.test  16.00
             test-suite :: External/SPEC/CFP2017rate/538.imagick_r/538.imagick_r.test  15.00
            test-suite :: External/SPEC/CFP2017speed/638.imagick_s/638.imagick_s.test  15.00
               test-suite :: External/SPEC/CFP2017rate/511.povray_r/511.povray_r.test  13.00
            test-suite :: MultiSource/Benchmarks/DOE-ProxyApps-C++/miniFE/miniFE.test  12.00
  test-suite :: MultiSource/Benchmarks/DOE-ProxyApps-C++/HACCKernels/HACCKernels.test  12.00
                          test-suite :: MultiSource/Applications/ClamAV/clamscan.test  12.00
           test-suite :: External/SPEC/CINT2017speed/620.omnetpp_s/620.omnetpp_s.test  11.00
            test-suite :: External/SPEC/CINT2017rate/520.omnetpp_r/520.omnetpp_r.test  11.00
                          test-suite :: MultiSource/Applications/sqlite3/sqlite3.test  11.00
                         test-suite :: MultiSource/Applications/JM/ldecod/ldecod.test  10.00
              test-suite :: MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/CLAMR.test  10.00
       test-suite :: External/SPEC/CINT2017speed/600.perlbench_s/600.perlbench_s.test   9.00
        test-suite :: External/SPEC/CINT2017rate/500.perlbench_r/500.perlbench_r.test   9.00
                   test-suite :: MultiSource/Benchmarks/Prolangs-C/bison/mybison.test   7.00
     test-suite :: MicroBenchmarks/LoopVectorization/LoopVectorizationBenchmarks.test   7.00
                    test-suite :: MultiSource/Benchmarks/FreeBench/neural/neural.test   6.00
          test-suite :: MultiSource/Benchmarks/MiBench/telecomm-gsm/telecomm-gsm.test   6.00
                   test-suite :: External/SPEC/CINT2017speed/605.mcf_s/605.mcf_s.test   6.00
                      test-suite :: MultiSource/Benchmarks/tramp3d-v4/tramp3d-v4.test   6.00
     ...


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130895/new/

https://reviews.llvm.org/D130895