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

Orlando Cazalet-Hyams via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 11 09:18:15 PDT 2023


Orlando added a comment.

In D148020#4258393 <https://reviews.llvm.org/D148020#4258393>, @jmorse wrote:

> 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?

Exactly. Thanks for the review(s).


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

https://reviews.llvm.org/D148020



More information about the llvm-commits mailing list