[PATCH] D148020: [Assignment Tracking][SROA] Don't un-poison dbg.assigns using multiple loc ops

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 09:03:03 PDT 2023


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

LGTM, to refresh my understanding this all becomes necessary because we've lost information during SROA but kept a link with the store, and now there's enough information to re-instate the dbg.assign Value, but it'd require a lot more work for us to get the expression correct so we're just not doing it for the moment?


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

https://reviews.llvm.org/D148020



More information about the llvm-commits mailing list