[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 12:44:42 PDT 2020
asbirlea added a comment.
Yes, the load should have the Phi as the defining access. I'm still looking into where this information should come from, but it's not hitting the phi translation.
Thank you for the revert, I'll update with the fix once I have it available.
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