maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[IslAst] Do not compare arrays in alias check which are known to be identical 

This possibly helps to avoid run-time check failures in the COSMO kernels.

llvm-svn: 311920
This commit is contained in:
Tobias Grosser 2017-08-28 20:17:02 +00:00
parent 3923b319ad
commit ee8ad1c0ff
2 changed files with 47 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
polly/lib/CodeGen/IslAst.cpp
View File

@ -350,6 +350,20 @@ static __isl_give isl_ast_node *AtEachDomain(__isl_take isl_ast_node *Node,
static __isl_give isl_ast_expr *
buildCondition(__isl_keep isl_ast_build *Build, const Scop::MinMaxAccessTy *It0,
const Scop::MinMaxAccessTy *It1) {
isl::id Left =
isl::manage(isl_pw_multi_aff_get_tuple_id(It0->first, isl_dim_set));
isl::id Right =
isl::manage(isl_pw_multi_aff_get_tuple_id(It1->first, isl_dim_set));
const ScopArrayInfo *BaseLeft =
ScopArrayInfo::getFromId(Left)->getBasePtrOriginSAI();
const ScopArrayInfo *BaseRight =
ScopArrayInfo::getFromId(Right)->getBasePtrOriginSAI();
if (BaseLeft && BaseLeft == BaseRight) {
return isl_ast_expr_from_val(
isl_val_int_from_si(isl_ast_build_get_ctx(Build), 1));
}
isl_ast_expr *NonAliasGroup, *MinExpr, *MaxExpr;
MinExpr = isl_ast_expr_address_of(isl_ast_build_access_from_pw_multi_aff(
Build, isl_pw_multi_aff_copy(It0->first)));

33
polly/test/Isl/Ast/aliasing_arrays_with_identical_base.ll Normal file
View File

@ -0,0 +1,33 @@
; RUN: opt %loadPolly -polly-ast -analyze < %s \
; RUN: -polly-invariant-load-hoisting \
; RUN: | FileCheck %s
; CHECK: if (1 && 1 && (&MemRef_X[1] <= &MemRef_BaseA[0] || &MemRef_BaseA[1024] <= &MemRef_X[0]) && (&MemRef_X[1] <= &MemRef_BaseB[0] || &MemRef_BaseB[1024] <= &MemRef_X[0]))
target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
target triple = "x86_64-unknown-linux-gnu"
define void @foo(float** nocapture readonly %X) {
entry:
br label %for.body
for.cond.cleanup:
ret void
for.body:
%i.011 = phi i64 [ 0, %entry ], [ %inc, %for.body ]
%conv = sitofp i64 %i.011 to float
%BaseA = load float*, float** %X, align 8
%BaseB = load float*, float** %X, align 8
%arrayidx = getelementptr inbounds float, float* %BaseA, i64 %i.011
%A = load float, float* %arrayidx, align 4
%add = fadd float %A, %conv
store float %add, float* %arrayidx, align 4
%arrayidxB = getelementptr inbounds float, float* %BaseB, i64 %i.011
%B = load float, float* %arrayidxB, align 4
%addB = fadd float %B, %conv
store float %addB, float* %arrayidxB, align 4
%inc = add nuw nsw i64 %i.011, 1
%exitcond = icmp eq i64 %inc, 1024
br i1 %exitcond, label %for.cond.cleanup, label %for.body
}