[PATCH] D91576: [MemCpyOpt] Bail on call slot optimization if it merges alias scopes

[Di Mo via Phabricator via llvm-commits]

modimo added inline comments.


================
Comment at: llvm/test/Transforms/MemCpyOpt/callslot_badaa.ll:34
+!0 = !{!0}
+!1 = !{!1}
----------------
nikic wrote:
> This alias.scope metadata looks corrupt. alias.scope accepts a list of scopes. I don't see any scopes or domains here.
Yeah the metadata in my first example was wrong. I've dug through how scopes and domains work and the current example should be representative of legal metadata.


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

https://reviews.llvm.org/D91576