Simplify type dispatch with device_storage_dispatch

[codereport]

Resolves #7390

Compile times:

// Before
real	33m29.842s
user	300m0.478s
sys	10m46.871s

// After
real	33m20.127s
user	299m24.825s
sys	10m35.779s

Binary sizes:

Before: -rwxr-xr-x  1 rapids rapids 328M Feb 22 15:10 libcudf_base.so
After:  -rwxr-xr-x  1 rapids rapids 327M Feb 23 07:49 libcudf_base.so

[jrhemstad]

Do you have any measurements of the impact of this change on compile time or binary size?

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (branch-0.19@580f9a2). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             branch-0.19    #7419   +/-   ##
==============================================
  Coverage               ?   82.23%           
==============================================
  Files                  ?      101           
  Lines                  ?    17060           
  Branches               ?        0           
==============================================
  Hits                   ?    14029           
  Misses                 ?     3031           
  Partials               ?        0           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 580f9a2...5239a20. Read the comment docs.

[codereport]

Do you have any measurements of the impact of this change on compile time or binary size?

Compile time no, I will post when I measure. Binary size doesn't seem to have changed (based on current commits)

Before: -rwxr-xr-x  1 rapids rapids 328M Feb 22 15:10 libcudf_base.so
After:  -rwxr-xr-x  1 rapids rapids 328M Feb 22 15:10 libcudf_base.so

[codereport]

Compile times:

// Before
real	33m29.842s
user	300m0.478s
sys	10m46.871s

// After
real	33m20.127s
user	299m24.825s
sys	10m35.779s

Binary sizes:

Before: -rwxr-xr-x  1 rapids rapids 328M Feb 22 15:10 libcudf_base.so
After:  -rwxr-xr-x  1 rapids rapids 327M Feb 23 07:49 libcudf_base.so

[davidwendt]

Looks good. You may need to update some copyrights.

[jrhemstad]

Compile times:

// Before
real	33m29.842s
user	300m0.478s
sys	10m46.871s

// After
real	33m20.127s
user	299m24.825s
sys	10m35.779s

Binary sizes:

Before: -rwxr-xr-x  1 rapids rapids 328M Feb 22 15:10 libcudf_base.so
After:  -rwxr-xr-x  1 rapids rapids 327M Feb 23 07:49 libcudf_base.so

Welp, so much for that idea. Either the compiler was already making the optimization or the switch is just too complicated to detect the identical instantiations. Anyways, I still think this is a clean way to handle the storage type. Thanks for doing this.

[codereport]

Anyways, I still think this is a clean way to handle the storage type. Thanks for doing this.

I agree, I still think this is a lot cleaner for the cases that work. All you have to do is find the type_dispatcher call, add the <dispatch_storage_type and you are done.

[codereport]

@gpucibot merge