[PATCH] D87163: [DSE] Switch to MemorySSA-backed DSE by default.
Alina Sbirlea via Phabricator via cfe-commits
cfe-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 cfe-commits
mailing list