[Mesa-dev] [PATCH 2/2] gallivm, ac: add function attributes at call sites instead of declarations

Marek Olšák maraeo at gmail.com
Tue Feb 28 14:02:08 UTC 2017


Ping.

On Thu, Feb 23, 2017 at 1:01 AM, Marek Olšák <maraeo at gmail.com> wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> They can vary at call sites if the intrinsic is NOT a legacy SI intrinsic.
> We need this to force readnone or inaccessiblememonly on some amdgcn
> intrinsics.
>
> This is only used with LLVM 4.0 and later. Intrinsics only used with
> LLVM <= 3.9 don't need the LEGACY flag.
>
> gallivm and ac code is in the same patch, because splitting would be
> more complicated with all the LEGACY uses all over the place.
> ---
>  src/amd/common/ac_llvm_build.c                    | 23 ++++----
>  src/amd/common/ac_llvm_util.c                     | 31 +++++++----
>  src/amd/common/ac_llvm_util.h                     | 17 +++---
>  src/amd/common/ac_nir_to_llvm.c                   | 63 ++++++++++++++--------
>  src/gallium/auxiliary/draw/draw_llvm.c            |  6 ++-
>  src/gallium/auxiliary/gallivm/lp_bld_intr.c       | 51 ++++++++++++------
>  src/gallium/auxiliary/gallivm/lp_bld_intr.h       | 11 ++--
>  src/gallium/auxiliary/gallivm/lp_bld_sample_soa.c |  3 +-
>  src/gallium/drivers/llvmpipe/lp_state_fs.c        |  3 +-
>  src/gallium/drivers/llvmpipe/lp_state_setup.c     |  7 +--
>  src/gallium/drivers/radeonsi/si_shader.c          | 64 ++++++++++++++---------
>  src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 14 +++--
>  12 files changed, 184 insertions(+), 109 deletions(-)


More information about the mesa-dev mailing list