[PATCH] D144483: [Assignment Tracking] Choose better passes for RemoveRedundantDbgInstrs call

Scott Linder via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 10:25:04 PST 2023


scott.linder accepted this revision.
scott.linder added a comment.
This revision is now accepted and ready to land.

Seems reasonable to me! LGTM with a small nit



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:10559-10560
+    if (isAssignmentTrackingEnabled(*F.getParent())) {
+      for (auto &BB : F)
+      RemoveRedundantDbgInstrs(&BB);
+    }
----------------
nit, formatting


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

https://reviews.llvm.org/D144483



More information about the llvm-commits mailing list