[PATCH] D83421: [RFC] MemorySSAUpdater: Simplify applyUpdates
Nicolai Hähnle via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 9 08:52:13 PDT 2020
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
More information about the llvm-commits
mailing list