[PATCH] D91576: [MemCpyOpt] Correctly merge alias scopes during call slot optimization

Tarique Islam via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 1 11:19:12 PST 2020


tislam added inline comments.


================
Comment at: llvm/test/Transforms/MemCpyOpt/callslot_badaa.ll:11
+;  !2 = distinct !{!2, !"callee1"}
+;  !3 = !{!1, !4, !5, !"callee0: %a"}
+;  !4 = distinct !{!4, !5, !"callee0: %a"}
----------------
Regarding the alias.scope metadata `!3 = !{!1, !4, !5, !"callee0: %a"}`. Should this be `!3 = !{!1, !4}`? An alias.scope metadata should be a list of scope metadata and should not include any domain metadata (!5, for example).


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

https://reviews.llvm.org/D91576



More information about the llvm-commits mailing list