[PATCH] D87661: [MemorySSA] Rename uses starting at defining access

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 14 18:18:10 PDT 2020


asbirlea created this revision.
asbirlea added a reviewer: george.burgess.iv.
Herald added subscribers: hiraditya, Prazek.
Herald added a project: LLVM.
asbirlea requested review of this revision.

Uses should be renamed starting at the defining access of the added
access. In the given testcase, the block in which the access is added
does not dominate the exit block. The exit block has an access that
needs to be renamed.

This patch is also reseting the "MustAlias" flag when renaming. The optimized
access is reset, so the access sees a defining access as not optimized
internally, but when prinitng we were still seeing an unoptimized access
as MustAlias due to the flag not being reset.

Resolves PR45927.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D87661

Files:
  llvm/lib/Analysis/MemorySSA.cpp
  llvm/lib/Analysis/MemorySSAUpdater.cpp
  llvm/test/Analysis/MemorySSA/pr45927.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D87661.291751.patch
Type: text/x-patch
Size: 4625 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200915/dc7b02e5/attachment.bin>


More information about the llvm-commits mailing list