[Beignet] [PATCH 4/7] add special register constoffst expressing curbe offset

[Xing, Homer]

OK. Will send version two of this patch.

-----Original Message-----
From: beignet-bounces+homer.xing=intel.com at lists.freedesktop.org [mailto:beignet-bounces+homer.xing=intel.com at lists.freedesktop.org] On Behalf Of Zhigang Gong
Sent: Saturday, April 27, 2013 6:40 PM
To: Xing, Homer
Cc: beignet at lists.freedesktop.org
Subject: Re: [Beignet] [PATCH 4/7] add special register constoffst expressing curbe offset

On Thu, Apr 25, 2013 at 02:21:30PM +0800, Homer Hsing wrote:
> 
> Signed-off-by: Homer Hsing <homer.xing at intel.com>
> ---
>  backend/src/ir/profile.cpp | 2 ++
>  backend/src/ir/profile.hpp | 3 ++-
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/backend/src/ir/profile.cpp b/backend/src/ir/profile.cpp 
> index 61531be..c1dc650 100644
> --- a/backend/src/ir/profile.cpp
> +++ b/backend/src/ir/profile.cpp
> @@ -40,6 +40,7 @@ namespace ir {
>          "stack_pointer",
>          "block_ip",
>          "barrier_id", "thread_number",
> +        "const_curbe_offset",
>      };
>  
>  #if GBE_DEBUG
> @@ -74,6 +75,7 @@ namespace ir {
>        DECL_NEW_REG(FAMILY_WORD, blockip);
>        DECL_NEW_REG(FAMILY_DWORD, barrierid);
>        DECL_NEW_REG(FAMILY_DWORD, threadn);
> +      DECL_NEW_REG(FAMILY_DWORD, constoffst);
>      }
>  #undef DECL_NEW_REG
>  
> diff --git a/backend/src/ir/profile.hpp b/backend/src/ir/profile.hpp 
> index 12050ff..cf35040 100644
> --- a/backend/src/ir/profile.hpp
> +++ b/backend/src/ir/profile.hpp
> @@ -63,7 +63,8 @@ namespace ir {
>      static const Register blockip = Register(19);  // blockip
>      static const Register barrierid = Register(20);// barrierid
>      static const Register threadn = Register(21);  // number of threads
> -    static const uint32_t regNum = 22;             // number of special registers
> +    static const Register constoffst = Register(22); // offset of 
> + const curbe
       could you change the comment to "//offset of global constant array's curbe."?
As we also have the input constant array in the const curbe.
> +    static const uint32_t regNum = 23;             // number of special registers
>      extern const char *specialRegMean[];           // special register name.
>    } /* namespace ocl */
>  
> --
> 1.8.1.2
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet
_______________________________________________
Beignet mailing list
Beignet at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/beignet