averello/swift-mirror
1
0
Fork 0
mirror of https://github.com/apple/swift.git synced 2025-12-21 12:14:44 +01:00
Code Issues Packages Projects Releases Wiki Activity

[load-store-opts] Avoid re-loading values of "let"-variables

Browse Source 
Teach LoadStoreOpts to handle "let" variables properly. Such variables should be loaded only once and their loaded values can be reused. This is safe, because once assigned these variables cannot change their value.

Swift SVN r27915
This commit is contained in:
Roman Levenstein
2015-04-29 18:02:12 +00:00
parent 0c6e412116
commit dcd3055360
4 changed files with 253 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/SILPasses/LoadStoreOpts.cpp
View File

@@ -855,7 +855,7 @@ void LSBBForwarder::invalidateReadFromStores(LSContext &Ctx,
AliasAnalysis *AA = Ctx.getAA();
llvm::SmallVector<SILValue, 4> InvalidatedStoreList;
for (auto &P : Stores)
if (P.second.aliasingRead(AA, Inst))
if (P.second.aliasingRead(AA, Inst) && !isLetPointer(P.first))
InvalidatedStoreList.push_back(P.first);
for (SILValue SIOp : InvalidatedStoreList) {