[Beignet] [PATCH v2] Fix bug: Initialize bti of LoadInstuctionPattern::shootByteGatherMsg().

Song, Ruiling ruiling.song at intel.com
Thu Nov 24 06:56:20 UTC 2016


Looks good. Thanks for the fix.

- Ruiling

> -----Original Message-----
> From: Beignet [mailto:beignet-bounces at lists.freedesktop.org] On Behalf Of
> yan.wang at linux.intel.com
> Sent: Thursday, November 24, 2016 2:55 PM
> To: beignet at lists.freedesktop.org
> Cc: Yan Wang <yan.wang at linux.intel.com>
> Subject: [Beignet] [PATCH v2] Fix bug: Initialize bti of
> LoadInstuctionPattern::shootByteGatherMsg().
> 
> From: Yan Wang <yan.wang at linux.intel.com>
> 
> If it isn't initialized, Luxmark hotel scene will display wrong.
> ---
>  backend/src/backend/gen_insn_selection.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/backend/src/backend/gen_insn_selection.cpp
> b/backend/src/backend/gen_insn_selection.cpp
> index c14e0bc..c9a5be7 100644
> --- a/backend/src/backend/gen_insn_selection.cpp
> +++ b/backend/src/backend/gen_insn_selection.cpp
> @@ -4487,7 +4487,7 @@ extern bool OCL_DEBUGINFO; // first defined by
> calling BVAR in program.cpp
>        using namespace ir;
>        unsigned addrBytes = typeSize(addr.type);
>        AddressMode AM = insn.getAddressMode();
> -      vector<GenRegister> btiTemp;
> +      vector<GenRegister> btiTemp = sel.getBTITemps(AM);
>        if (AM == AM_DynamicBti || AM == AM_StaticBti) {
>          if (AM == AM_DynamicBti) {
>            Register btiReg = insn.getBtiReg();
> --
> 2.7.4
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list