[PATCH] D87163: [DSE] Switch to MemorySSA-backed DSE by default.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 15 13:32:49 PDT 2020


asbirlea added a comment.

Right!
I believe the solution is to set the location size to unknown after a phi translation with the same ptr, but I need to properly verify that.

  // include/llvm/Analysis/MemorySSA.h:1233
  CurrentPair.second = Location.getWithNewSize(LocationSize::unknown());
  return


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87163



More information about the llvm-commits mailing list