[PATCH] D17250: [Polly] [Refactor] Enable llvm's isa/cast/dyn_cast on MemAccInst

Michael Kruse via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 08:49:09 PST 2016


Meinersbur accepted this revision.
Meinersbur added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: lib/Analysis/ScopInfo.cpp:619
@@ -618,5 +618,3 @@
 void MemoryAccess::buildMemIntrinsicAccessRelation() {
-  auto MAI = MemAccInst(getAccessInstruction());
-  (void)MAI;
-  assert(MAI.isMemIntrinsic());
+  assert(isa<MemIntrinsic>(MemAccInst(getAccessInstruction())));
   assert(Subscripts.size() == 2 && Sizes.size() == 0);
----------------
Does this work was well?
```
assert(isa<MemIntrinsic>(getAccessInstruction()));
```


Repository:
  rL LLVM

http://reviews.llvm.org/D17250





More information about the llvm-commits mailing list