[PATCH] D50911: Update MemorySSA in LoopSimplifyCFG.

Chandler Carruth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 20 13:44:08 PDT 2018


chandlerc added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopSimplifyCFG.cpp:77
                                            LPMUpdater &) {
-  if (!simplifyLoopCFG(L, AR.DT, AR.LI, AR.SE))
+  std::unique_ptr<MemorySSAUpdater> MSSAU;
+  if (EnableMSSALoopDependency && AR.MSSA)
----------------
Same comment as on other patch -- could we use an Optional here to avoid heap allocation?


Repository:
  rL LLVM

https://reviews.llvm.org/D50911





More information about the llvm-commits mailing list