GridTools / gridtools

Libraries and utilities to develop performance portable applications for weather and climate.
https://gridtools.github.io/gridtools
Other
62 stars 21 forks source link

To figure out if strides_kind (aka storage_id) feature is useful. #1461

Open anstaf opened 4 years ago

anstaf commented 4 years ago

This feature is for sure useless for non cuda backends. For cuda the performance benefit can come from: 1) saving on the overall strides size while passing the data to the kernel; 2) calculating offsets for several fields in one go. The later was checked once on dycore — the result is that it doesn’t affect performance.

Getting rid from this feature support allows to simplify SID concept definition and the sid::composite implementation (which is the most complex metaprogramming part of our data base).

havogt commented 4 years ago

Postpone investigation until we can test it again on a possible GT2.0-dycore (and maybe can also investigate on different hardware, e.g. AMD GPUs)