[PATCH] D152859: [MemorySSA] Always perform MemoryUses liveOnEntry optimization on MSSA construction

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea added inline comments.


================
Comment at: llvm/lib/Analysis/MemorySSA.cpp:368
+template <typename AliasAnalysisType>
+static bool isUseTriviallyOptimizableToLiveOnEntry(AliasAnalysisType &AA,
                                                    const Instruction *I) {
----------------
Can you check if the the callsite in `optimizeUses` can be removed with this change?
I think the one in getClobbering needs to stay.


================
Comment at: llvm/lib/Passes/PassBuilder.cpp:1001
+Expected<bool> parseMemorySSAPrinterPassOptions(StringRef Params) {
+  return parseSinglePassOption(Params, "no-ensure-optimized-uses",
+                               "MemorySSAPrinterPass");
----------------
nit: Simpify the test option to "no-optimized-uses".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152859