[Beignet] [PATCH] GBE: clear deadprintfs when current function is done.

Zhigang Gong zhigang.gong at linux.intel.com
Wed Aug 27 00:29:35 PDT 2014


Pushed, thanks.

On Tue, Aug 26, 2014 at 04:25:00PM +0800, He Junyan wrote:
> OK,
> Thanks for finding this bug.
> 
> On 二, 2014-08-26 at 15:39 +0800, Ruiling Song wrote:
> > It should be cleared, to prevent invalid pointers staying there
> > when processing next Function.
> > 
> > Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> > ---
> >  backend/src/llvm/llvm_printf_parser.cpp |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/backend/src/llvm/llvm_printf_parser.cpp b/backend/src/llvm/llvm_printf_parser.cpp
> > index e02f5aa..00e1ef8 100644
> > --- a/backend/src/llvm/llvm_printf_parser.cpp
> > +++ b/backend/src/llvm/llvm_printf_parser.cpp
> > @@ -616,6 +616,7 @@ error:
> >          prf.first->eraseFromParent();
> >      }
> >  
> > +    deadprintfs.clear();
> >      delete builder;
> >  
> >      return changed;
> 
> 
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list