[PATCH] D133312: [Assignment Tracking][17/*] Account for assignment tracking in memcpyopt

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 9 06:43:18 PDT 2022


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

LGTM, if you feel the call of duty then you could remove the tbaa metadata from the patch.



================
Comment at: llvm/test/DebugInfo/Generic/assignment-tracking/memcpyopt/merge-stores.ll:22
+
+;; Check that the memcpy that memcpyopt creates to merge 4 stores merges the
+;; DIASsignIDs from the stores.
----------------
More of a memset than a memcpy


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

https://reviews.llvm.org/D133312



More information about the llvm-commits mailing list