[Mlir-commits] [mlir] [mlir][Vector] Update VectorEmulateNarrowType.cpp (2/N) (PR #123527)
Alan Li
llvmlistbot at llvm.org
Tue Feb 4 09:53:46 PST 2025
lialan wrote:
> I'm +1 on having them in a single PR because the change is not big, and they all touch the same file. Having a clear PR description is good enough to me. IMHO, it also makes the codebase state and commits tracking easier. (2) + (3) is like +107 -78 lines, which is a reasonable size of PR to me.
+1 too.
https://github.com/llvm/llvm-project/pull/123527
More information about the Mlir-commits
mailing list