[PATCH] D17641: [Polly] [MemAccInst] Delete all the assignment operators, as they are not need.

Hongbin Zheng via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 08:50:26 PST 2016


etherzhhb added inline comments.

================
Comment at: include/polly/Support/ScopHelper.h:68
@@ -67,3 +67,3 @@
 private:
-  llvm::Instruction *I;
+  llvm::Instruction *const I;
 
----------------
Meinersbur wrote:
> The constness of the Instruction pointed to should be determined by the user of MemAccInst by declaring it const, e.g. `const MemAccInst`.
This make sense. Could we use MemAccInst just like "const MemAccInst"?


Repository:
  rL LLVM

http://reviews.llvm.org/D17641





More information about the llvm-commits mailing list