[llvm] [RFC][Draft] Extend MemoryEffects to Support Target-Specific Memory L… (PR #148650)

[via llvm-commits]

CarolineConcatto wrote:

Hi @nikic ,

Thank you for your comments, I do have some extra notes.

My concern is that with Data being 32 bits and reserving 2 bits for each location, there are 16 memory locations available. There are 3 already being reserved to ArgMem, InaccessibleMem and Other. Now AArch64 is using 2(AArch64_FPRM and AArch64_ZA, I will probably expand to use for ZT0), if each target adds its own location it  could be out of space with Data.

Any suggestion how to implement the Target  Memory Location? The way it is implemented the Target Memory Locations for now is correct ? Should I change?

Another question, should we treat these extra locations as Inaccessible memory as well?
So, should  onlyAccessesInaccessibleMem()  return true is any Target Memory Location is set.

https://github.com/llvm/llvm-project/pull/148650