[Mesa-dev] [PATCH 2/6] R600: CONST_ADDRESS node is not marked as mayLoad anymore

Tom Stellard tom at stellard.net
Tue Feb 19 07:36:20 PST 2013


On Mon, Feb 18, 2013 at 05:27:26PM +0100, Vincent Lejeune wrote:
> mayLoad complexify scheduling and does not bring any usefull info
> as the location is not writeable at all.

Reviewed-by: Tom Stellard <thomas.stellard at amd.com>
> ---
>  lib/Target/R600/R600Instructions.td | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/Target/R600/R600Instructions.td b/lib/Target/R600/R600Instructions.td
> index e4cc06e..0a777f1 100644
> --- a/lib/Target/R600/R600Instructions.td
> +++ b/lib/Target/R600/R600Instructions.td
> @@ -513,7 +513,7 @@ def INTERP_PAIR_ZW :  AMDGPUShaderInst <
>  
>  def CONST_ADDRESS: SDNode<"AMDGPUISD::CONST_ADDRESS",
>    SDTypeProfile<1, -1, [SDTCisInt<0>, SDTCisPtrTy<1>]>,
> -  [SDNPMayLoad, SDNPVariadic]
> +  [SDNPVariadic]
>  >;
>  
>  //===----------------------------------------------------------------------===//
> -- 
> 1.8.1.2
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list