[PATCH] D83421: [RFC] MemorySSAUpdater: Simplify applyUpdates

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle abandoned this revision.
nhaehnle added a comment.

Okay, thank you for that context, that makes sense. I'm going to drop this change then obviously.

FYI, I'm considering to at least try out making dominator tree construction based on the CfgInterfaceImpl from D83088 <https://reviews.llvm.org/D83088>, so that GenericDomTreeConstruction is no longer all a giant ball of templates. Do you think this would help you with your changes? Obviously there's the question of how it impacts compile time, but we won't really know for sure until we try it, which is one of my motivations here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83421