[PATCH] D145688: [StructurizeCFG] Use UniformityAnalysis instead of DivergenceAnalysis

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 9 06:06:58 PST 2023


foad added a subscriber: nhaehnle.
foad added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/StructurizeCFG.cpp:802
-  if (DA)
-    DA->removeValue(Term);
   Term->eraseFromParent();
----------------
Does UA need something like removeValue? I see it was added by @nhaehnle in D43743 "to avoid dangling pointers as a matter of defensive programming".


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145688



More information about the llvm-commits mailing list