[PATCH] D73478: [DebugInfo] replaceDbgUsesWithUndef before removing single store alloca

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 08:02:55 PST 2020


probinson added a project: debug-info.
probinson added a comment.

Please add the `debug-info` tag to these reviews.

I know off-list I recommended doing all your transform-related patches separately, but... given that this and D73473 <https://reviews.llvm.org/D73473> are both in SROA, I think the tests at least should be combined into one test file (which reduces testing overhead).  And in general it should be fine to combine patches for the same pass into a single patch.


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

https://reviews.llvm.org/D73478





More information about the llvm-commits mailing list