[PATCH] D84410: [LAA] Dynamically allocate RuntimeCheckingPtrGroup (NFC).

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

I don't understand the motivation. Sharing `RuntimeCheckingPtrGroup` between multiple objects will increase coupling between them.

Summary:

> Currently parts in RuntimePointerChecking owns RuntimeCheckingPtrGroup directly, which is reference from other datastructures in RuntimePointerChecking.

The grammar of this sentence seems off.



================
Comment at: llvm/include/llvm/Analysis/LoopAccessAnalysis.h:21-23
 #include "llvm/Pass.h"
 
+#include <memory>
----------------



================
Comment at: llvm/lib/Analysis/LoopAccessAnalysis.cpp:66-68
 #include <vector>
 
+#include <memory>
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84410/new/

https://reviews.llvm.org/D84410