[PATCH] D35610: [Polly] Avoid use of `getStmtFor(BB)` in ScopBuilder. [NFC]

[Nandini Singhal via Phabricator via llvm-commits]

nandini12396 updated this revision to Diff 107458.
nandini12396 retitled this revision from "[Polly] Avoid use of `getStmtFor(BB)` in ScopBuilder. NFC" to "[Polly] Avoid use of `getStmtFor(BB)` in ScopBuilder. [NFC]".
nandini12396 edited the summary of this revision.
nandini12396 added subscribers: pollydev, llvm-commits.
nandini12396 added a comment.

Addressed comments as suggested by Michael Sir.


https://reviews.llvm.org/D35610

Files:
  lib/Analysis/ScopBuilder.cpp


Index: lib/Analysis/ScopBuilder.cpp
===================================================================
--- lib/Analysis/ScopBuilder.cpp
+++ lib/Analysis/ScopBuilder.cpp
@@ -880,11 +880,11 @@
 /// happened yet, such that virtual and physical uses are equivalent.
 static void verifyUses(Scop *S, LoopInfo &LI, DominatorTree &DT) {
   for (auto *BB : S->getRegion().blocks()) {
-    auto *Stmt = S->getStmtFor(BB);
-    if (!Stmt)
-      continue;
-
     for (auto &Inst : *BB) {
+      auto *Stmt = S->getStmtFor(&Inst);
+      if (!Stmt)
+        continue;
+
       if (isIgnoredIntrinsic(&Inst))
         continue;
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35610.107458.patch
Type: text/x-patch
Size: 619 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170720/a98dcded/attachment.bin>