[polly] r309401 - [Simplify] Remove empty partial accesses first. NFC.

[Michael Kruse via llvm-commits]

Author: meinersbur
Date: Fri Jul 28 09:57:45 2017
New Revision: 309401

URL: http://llvm.org/viewvc/llvm-project?rev=309401&view=rev
Log:
[Simplify] Remove empty partial accesses first. NFC.

So follow-up cleanup do not need special handling for such accesses.

Modified:
    polly/trunk/lib/Transform/Simplify.cpp
    polly/trunk/test/Simplify/emptyaccessdomain.ll

Modified: polly/trunk/lib/Transform/Simplify.cpp
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Transform/Simplify.cpp?rev=309401&r1=309400&r2=309401&view=diff
==============================================================================
--- polly/trunk/lib/Transform/Simplify.cpp (original)
+++ polly/trunk/lib/Transform/Simplify.cpp Fri Jul 28 09:57:45 2017
@@ -455,15 +455,15 @@ public:
     this->S = &S;
     ScopsProcessed++;
 
+    DEBUG(dbgs() << "Removing partial writes that never happen...\n");
+    removeEmptyPartialAccesses();
+
     DEBUG(dbgs() << "Removing overwrites...\n");
     removeOverwrites();
 
     DEBUG(dbgs() << "Removing redundant writes...\n");
     removeRedundantWrites();
 
-    DEBUG(dbgs() << "Removing partial writes that never happen...\n");
-    removeEmptyPartialAccesses();
-
     DEBUG(dbgs() << "Cleanup unused accesses...\n");
     LoopInfo *LI = &getAnalysis<LoopInfoWrapperPass>().getLoopInfo();
     markAndSweep(LI);

Modified: polly/trunk/test/Simplify/emptyaccessdomain.ll
URL: http://llvm.org/viewvc/llvm-project/polly/trunk/test/Simplify/emptyaccessdomain.ll?rev=309401&r1=309400&r2=309401&view=diff
==============================================================================
--- polly/trunk/test/Simplify/emptyaccessdomain.ll (original)
+++ polly/trunk/test/Simplify/emptyaccessdomain.ll Fri Jul 28 09:57:45 2017
@@ -30,7 +30,7 @@ return:
 
 
 ; CHECK: Statistics {
-; CHECK:     Overwrites removed: 1
+; CHECK:     Access with empty domains removed: 1
 ; CHECK:     Stmts removed: 1
 ; CHECK: }