[Mesa-dev] [PATCH] isl: annotate isl_get_interleaved_msaa_px_size_sa return value as const

Jason Ekstrand jason at jlekstrand.net
Tue Feb 7 16:11:32 UTC 2017


How does it make any sense to declare const on the return type of a
function that doesn't return a pointer???  It's not returning by reference,
it's returning a copy so const on the return type of the function means
nothing.

On Tue, Feb 7, 2017 at 4:54 AM, Emil Velikov <emil.l.velikov at gmail.com>
wrote:

> Already used as such and annotated in a number of places.
> Should resolve -Wignored-qualifiers warnings, such as:
>
> src/intel/blorp/blorp_blit.c:1857:8: warning: 'const' type qualifier on
> return type has no effect [-Wignored-qualifiers]
>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
>  src/intel/blorp/blorp_blit.c | 4 ++--
>  src/intel/isl/isl.c          | 4 ++--
>  src/intel/isl/isl.h          | 2 +-
>  3 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c
> index b89a112b8c..1d3b52d80e 100644
> --- a/src/intel/blorp/blorp_blit.c
> +++ b/src/intel/blorp/blorp_blit.c
> @@ -1358,7 +1358,7 @@ surf_get_intratile_offset_px(struct
> brw_blorp_surface_info *info,
>                               uint32_t *tile_x_px, uint32_t *tile_y_px)
>  {
>     if (info->surf.msaa_layout == ISL_MSAA_LAYOUT_INTERLEAVED) {
> -      struct isl_extent2d px_size_sa =
> +      const struct isl_extent2d px_size_sa =
>           isl_get_interleaved_msaa_px_size_sa(info->surf.samples);
>        assert(info->tile_x_sa % px_size_sa.width == 0);
>        assert(info->tile_y_sa % px_size_sa.height == 0);
> @@ -1678,7 +1678,7 @@ try_blorp_blit(struct blorp_batch *batch,
>         * If it's UMS, then we have no choice but to set up the rendering
>         * pipeline as multisampled.
>         */
> -      struct isl_extent2d px_size_sa =
> +      const struct isl_extent2d px_size_sa =
>           isl_get_interleaved_msaa_px_size_sa(params->dst.surf.samples);
>        params->x0 = ROUND_DOWN_TO(params->x0, 2) * px_size_sa.width;
>        params->y0 = ROUND_DOWN_TO(params->y0, 2) * px_size_sa.height;
> diff --git a/src/intel/isl/isl.c b/src/intel/isl/isl.c
> index 82ab68dc65..62aa31cf29 100644
> --- a/src/intel/isl/isl.c
> +++ b/src/intel/isl/isl.c
> @@ -329,7 +329,7 @@ isl_choose_msaa_layout(const struct isl_device *dev,
>     }
>  }
>
> -struct isl_extent2d
> +const struct isl_extent2d
>  isl_get_interleaved_msaa_px_size_sa(uint32_t samples)
>  {
>     assert(isl_is_pow2(samples));
> @@ -342,7 +342,7 @@ isl_get_interleaved_msaa_px_size_sa(uint32_t samples)
>      *    MSFMT_DEPTH_STENCIL, W_L and H_L must be adjusted as follows
> before
>      *    proceeding: [...]
>      */
> -   return (struct isl_extent2d) {
> +   return (const struct isl_extent2d) {
>        .width = 1 << ((ffs(samples) - 0) / 2),
>        .height = 1 << ((ffs(samples) - 1) / 2),
>     };
> diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h
> index eff7218748..e9074e526d 100644
> --- a/src/intel/isl/isl.h
> +++ b/src/intel/isl/isl.h
> @@ -1172,7 +1172,7 @@ isl_tiling_is_std_y(enum isl_tiling tiling)
>     return (1u << tiling) & ISL_TILING_STD_Y_MASK;
>  }
>
> -struct isl_extent2d ATTRIBUTE_CONST
> +const struct isl_extent2d ATTRIBUTE_CONST
>  isl_get_interleaved_msaa_px_size_sa(uint32_t samples);
>
>  static inline bool
> --
> 2.11.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170207/84cb2b88/attachment.html>


More information about the mesa-dev mailing list