[Beignet] [PATCH] GBE: Remove TBAA.

Song, Ruiling ruiling.song at intel.com
Wed Oct 18 07:35:37 UTC 2017


Please skip this patch. I have sent another patch to solve the problem.

Ruiling
> -----Original Message-----
> From: Song, Ruiling
> Sent: Wednesday, October 18, 2017 11:03 AM
> To: beignet at lists.freedesktop.org
> Cc: Song, Ruiling <ruiling.song at intel.com>
> Subject: [PATCH] GBE: Remove TBAA.
> 
> At the time we expand llvm.memcpy. we introduce some load/store
> that break the TBAA. This issue comes out in llvm5.0. So we remove
> the TBAA from the compilation passes.
> 
> Signed-off-by: Ruiling Song <ruiling.song at intel.com>
> ---
>  backend/src/llvm/llvm_to_gen.cpp | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/backend/src/llvm/llvm_to_gen.cpp
> b/backend/src/llvm/llvm_to_gen.cpp
> index 8546f73..f679c58 100644
> --- a/backend/src/llvm/llvm_to_gen.cpp
> +++ b/backend/src/llvm/llvm_to_gen.cpp
> @@ -87,7 +87,6 @@ namespace gbe
>      FPM.add(new TargetLibraryInfo(*libraryInfo));
>  #endif
>  #if LLVM_VERSION_MAJOR * 10 + LLVM_VERSION_MINOR >= 38
> -    FPM.add(createTypeBasedAAWrapperPass());
>      FPM.add(createBasicAAWrapperPass());
>  #else
>      FPM.add(createTypeBasedAliasAnalysisPass());
> @@ -129,7 +128,6 @@ namespace gbe
>      MPM.add(new TargetLibraryInfo(*libraryInfo));
>  #endif
>  #if LLVM_VERSION_MAJOR * 10 + LLVM_VERSION_MINOR >= 38
> -    MPM.add(createTypeBasedAAWrapperPass());
>      MPM.add(createBasicAAWrapperPass());
>  #else
>      MPM.add(createTypeBasedAliasAnalysisPass());
> --
> 2.4.1



More information about the Beignet mailing list