[Beignet] [PATCH] delete GEPInst when it is no longer used

Zhigang Gong zhigang.gong at linux.intel.com
Sun Jul 27 23:47:00 PDT 2014


LGTM, pushed, thanks.

On Fri, Jul 25, 2014 at 06:00:27AM +0800, Guo Yejun wrote:
> Signed-off-by: Guo Yejun <yejun.guo at intel.com>
> ---
>  backend/src/llvm/llvm_passes.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/backend/src/llvm/llvm_passes.cpp b/backend/src/llvm/llvm_passes.cpp
> index 3078470..b8ab844 100644
> --- a/backend/src/llvm/llvm_passes.cpp
> +++ b/backend/src/llvm/llvm_passes.cpp
> @@ -378,7 +378,7 @@ namespace gbe
>      //replace uses of the GEP instruction with the newly calculated pointer
>      GEPInst->replaceAllUsesWith(intToPtrInst);
>      GEPInst->dropAllReferences();
> -    GEPInst->removeFromParent();
> +    GEPInst->eraseFromParent();
>  
>  #if FORMER_VERSION
>      //insert new pointer into parent list
> -- 
> 1.8.3.2
> 
> _______________________________________________
> Beignet mailing list
> Beignet at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/beignet


More information about the Beignet mailing list