https://github.com/NagyDonat commented:

The change seems to be correct, but I feel that a different approach might be 
better.

Instead of "going back" to the `SuperR` and then calculating indices/offsets 
from that point, you could switch to calculating the indices from the second 
argument pointer itself (in particular, just dereference it to see the starting 
element).

This more straigthforward approach could avoid some fragile symbolic 
calculations.

https://github.com/llvm/llvm-project/pull/198346
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to