diff options
author | Chad Rosier <mcrosier@apple.com> | 2011-11-15 23:34:05 +0000 |
---|---|---|
committer | Chad Rosier <mcrosier@apple.com> | 2011-11-15 23:34:05 +0000 |
commit | 291ce47db730d2896da17232ece46371de4b9317 (patch) | |
tree | 88f3e5300e0453c5fa8aa77e0ef33de72344289c | |
parent | 78286b24fd85e30b519f3c333cdad0872aaa1f9c (diff) | |
download | bcm5719-llvm-291ce47db730d2896da17232ece46371de4b9317.tar.gz bcm5719-llvm-291ce47db730d2896da17232ece46371de4b9317.zip |
GEPs with all zero indices are trivially coalesced by fast-isel. For example,
%arrayidx135 = getelementptr inbounds [4 x [4 x [4 x [4 x i32]]]]* %M0, i32 0, i64 0
%arrayidx136 = getelementptr inbounds [4 x [4 x [4 x i32]]]* %arrayidx135, i32 0, i64 %idxprom134
Prior to this commit, the GEP instruction that defines %arrayidx136 thought that
%arrayidx135 was a trivial kill. The GEP that defines %arrayidx135 doesn't
generate any code and thus %M0 gets folded into the second GEP. Thus, we need
to look through GEPs with all zero indices.
rdar://10443319
llvm-svn: 144730
-rw-r--r-- | llvm/lib/CodeGen/SelectionDAG/FastISel.cpp | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp b/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp index e8f8c73d688..c58d730454f 100644 --- a/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/FastISel.cpp @@ -96,6 +96,11 @@ bool FastISel::hasTrivialKill(const Value *V) const { !hasTrivialKill(Cast->getOperand(0))) return false; + // GEPs with all zero indices are trivially coalesced by fast-isel. + if (const GetElementPtrInst *GEP = dyn_cast<GetElementPtrInst>(I)) + if (GEP->hasAllZeroIndices() && !hasTrivialKill(GEP->getOperand(0))) + return false; + // Only instructions with a single use in the same basic block are considered // to have trivial kills. return I->hasOneUse() && |