[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