[PATCH] D21563: [PM] refactoring of LoopAccessInfo /NFC

Sean Silva via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 22 15:48:53 PDT 2016


silvas added a subscriber: silvas.

================
Comment at: include/llvm/Analysis/LoopAccessAnalysis.h:617-623
@@ -616,9 +616,9 @@
 
   /// We need to check that all of the pointers in this list are disjoint
   /// at runtime.
-  RuntimePointerChecking PtrRtChecking;
+  std::unique_ptr<RuntimePointerChecking> PtrRtChecking;
 
   /// \brief the Memory Dependence Checker which can determine the
   /// loop-independent and loop-carried dependences between memory accesses.
-  MemoryDepChecker DepChecker;
+  std::unique_ptr<MemoryDepChecker> DepChecker;
 
----------------
anemet wrote:
> Please add a comment saying that these are pointers to facilitate the move ctor.
This might not be relevant, but FYI there is a gotcha with MSVC 2013 where it can't synthesize a move ctor. Just a heads up in case there are bot problems with this patch.

This caused me problems when porting JumpThreading. See http://reviews.llvm.org/rL272607 in particular the manually defined move constructor with the comment "// Hack for MSVC 2013 which seems like it can't synthesize this.".

The fix there is based on the earlier r216244 that gave insight into the problem.


http://reviews.llvm.org/D21563





More information about the llvm-commits mailing list