[PATCH] D51718: Update MemorySSA in LoopRotate.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added inline comments.


================
Comment at: lib/Transforms/Scalar/LoopRotation.cpp:49
+  bool Changed = LoopRotation(&L, &AR.LI, &AR.TTI, &AR.AC, &AR.DT, &AR.SE,
+                              MSSAU.hasValue() ? MSSAU.getPointer() : nullptr,
+                              SQ, false, Threshold, false);
----------------
greened wrote:
> Why not just pass the Optional?
I'm inclined to prefer using the pointer just due to consistency with the other passes + the hope that this won't be Optional forever :).

If you or other reviewers strongly prefer passing the Optional, please let me know and I'll update here and in all other passes.


Repository:
  rL LLVM

https://reviews.llvm.org/D51718