[PATCH] D115160: [GVN] MemorySSA for GVN: remove MemDep form GVNPass:ValueTable

[Momchil Velikov via Phabricator via llvm-commits]

chill added a comment.

In D115160#3259460 <https://reviews.llvm.org/D115160#3259460>, @asbirlea wrote:

> A very broad comment for a smooth transition
>
> - the patches should be split into changes to GVN independent of the switch, e.g. a) factoring out components/methods where the differences between MSSA and MD occur and b) things that need to be added for the transition
> - we should keep the MD implementation and have a flag for switching between the current one and the MSSA one being added
> - once the flag flip sticks, then we'd remove the MD implementation

Ack.


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

https://reviews.llvm.org/D115160