https://github.com/NagyDonat commented:

The differences between the indexing kinds are still very difficult to 
understand, but your change probably reduces the amount of confusion. I have 
several suggestions and I'll give a final review when the rest of the stack is 
merged (or close to it).

By the way, is this change NFC? The PR description looks as if you're changing 
a few things – if it is still NFC, please add an explanation paragraph why the 
changed logic does not change the end result.

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

Reply via email to