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

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Ignoring the question of direction, the code change itself seems ok to me.



================
Comment at: llvm/include/llvm/Analysis/LoopAccessAnalysis.h:22
 #include "llvm/Pass.h"
 
+#include <memory>
----------------
I don't recall seeing such separation between llvm and STD headers with newline elsewhere in llvm.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84410