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

Ryotaro Kasuga via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 19 01:36:24 PST 2026


================
@@ -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?
----------------
kasuga-fj wrote:

I see, thanks.

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

That makes sense, replaced.

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


More information about the llvm-commits mailing list