[PATCH] D122601: Fix MemorySSAUpdater::insertDef for dead code

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 01:40:00 PDT 2022


fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks! Please wait a day or so with committing, in case others have additional comments. In terms of test location, it might be good to place it under `Transforms/GVN` as this is causing by an update through a transformation. But we are not really strict with avoiding running transforms in `Analysis` tests, so whatever should be fine.


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

https://reviews.llvm.org/D122601



More information about the llvm-commits mailing list