[llvm] [DA] Remove `DependenceInfo::unifySubscriptType` (PR #181607)

[Nikita Popov via llvm-commits]

================
@@ -3263,7 +3211,10 @@ bool DependenceInfo::tryDelinearize(Instruction *Src, Instruction *Dst,
   for (int I = 0; I < Size; ++I) {
     Pair[I].Src = SrcSubscripts[I];
     Pair[I].Dst = DstSubscripts[I];
-    unifySubscriptType(&Pair[I]);
+
+    // TODO: Is this check necessary?
----------------
nikic wrote:

Ah, I see. SCEV does not (currently at least) support looking through address space casts, so I think the situation I had in mind is not possible.

Probably makes sense to replace this with an assertion then. If some case violates it, we'll find out...

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