bjope wrote:

But isn't it still a bit weird that if I change the datalayout to set the same 
pointer size for the two address spaces, then BasicAA is able to derive that 
there is NoAlias (https://godbolt.org/z/vhcG4jx7a), and the same goes even if 
using same address space for both accesses (https://godbolt.org/z/GoEM7zx68).

So this patch kind of messed things up for targets that have multiple address 
spaces, but with different pointer sizes, since the early out for "index size 
mismatch" is making it skip some checks that could help identify NoAlias.

https://github.com/llvm/llvm-project/pull/119365
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to