[mlir] [MLIR][Vector] Add a missing builder implementation for Vector_WarpEx… (PR #110106)
Andrzej Warzyński
llvmlistbot at llvm.org
Tue Oct 15 01:27:17 PDT 2024
banach-space wrote:
I agree with Ben.
If the builder is not used, it should be deleted rather than implemented. If it is to be added, there should also be examples of it being used (these would demonstrate that this is indeed needed).
As a rule of thumb, we should avoid implementing features that are not used - that's potential maintenance burden. While this is a rather small change, there's just no evidence of anyone needing it. Please correct me if I'm wrong :)
https://github.com/llvm/llvm-project/pull/110106
More information about the Mlir-commits
mailing list