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

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers added inline comments.


================
Comment at: llvm/lib/CodeGen/CallBrPrepare.cpp:181
+    for (Use &U : CBR->uses()) {
+      PrintDebugDomInfo(DT, U, LandingPad, /*IsDefaultDest*/ false);
+      PrintDebugDomInfo(DT, U, DefaultDest, /*IsDefaultDest*/ true);
----------------
aeubanks wrote:
> I'd just `ifndef NDEBUG` around these two lines and remove the empty `PrintDebugDomInfo` implementation
Slight disagree, but could be convinced otherwise. Do you feel strongly about this?


================
Comment at: llvm/lib/CodeGen/CallBrPrepare.cpp:187-189
+        SSAUpdate.Initialize(U->getType(), U->getName());
+        SSAUpdate.AddAvailableValue(LandingPad, Intrinsic);
+        SSAUpdate.RewriteUseAfterInsertions(U);
----------------
aeubanks wrote:
> maybe I'm misunderstanding what is going on, but can this be `U.set(Intrinsic)`?
oh! yeah, nice!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139970