[PATCH] D139970: [llvm][CallBrPrepare] use SSAUpdater to use intrinsic value

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Instead of having InsertIntrinsicCalls construct a map of SSAUpdaters, then have UpdateSSA iterate through it, can you just perform all the SSA updates involving a given SSAUpdater immediately after constructing it?  The separation seems to add complexity without any benefit.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139970