[PATCH] D108315: [AMDGPU] Add alias.scope metadata to lowered LDS struct

[Jon Chesterfield via Phabricator via llvm-commits]

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

Change looks good but this comment is strange:

  // Create alias.scope and their lists. Each field in the new structure
  // does not alias with all other fields.

That is true of any structure. Each s/field/pointer in the new?

It seems a shame that moving variables into a struct thwarts AA at present.


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

https://reviews.llvm.org/D108315