[PATCH] D147312: [Assignment Tracking][SROA] Handle DIArgList in migrateDebugInfo

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 31 04:05:06 PDT 2023


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

LGTM; could you expand the comment where you set the kill location a little more, to explicitly explain why we can't use the DIArgList. As I understand it, it's because we'd have to compute the value, then start fragmenting it up, which is a large amount of effort for a rare scenario.


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

https://reviews.llvm.org/D147312



More information about the llvm-commits mailing list