[PATCH] D71480: [BasicBlockUtils] Fix dbg.value elimination problem in MergeBlockIntoPredecessor

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 13 15:36:39 PST 2019


vsk accepted this revision.
vsk added a comment.

LGTM as well.



================
Comment at: llvm/lib/Transforms/Utils/BasicBlockUtils.cpp:288
+  // Eliminate duplicate/redundant dbg.values. This seems to be a good place to
+  // do that sine we might end up with redundant dbg.values describing the entry
+  // PHI node post-splice.
----------------
nit, since


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71480





More information about the llvm-commits mailing list