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

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 20 12:34:45 PST 2022


asbirlea added a comment.

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


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

https://reviews.llvm.org/D115160



More information about the llvm-commits mailing list