[Mlir-commits] [mlir] [mlir][linalg][transform] Rename {masked_vectorize => vectorize => vectorize_children_and...}. (PR #66575)
Oleksandr Alex Zinenko
llvmlistbot at llvm.org
Mon Sep 25 02:30:20 PDT 2023
Ingo =?utf-8?q?Müller?= <ingomueller at google.com>,
Ingo =?utf-8?q?Müller?= <ingomueller at google.com>,
Ingo =?utf-8?q?Müller?= <ingomueller at google.com>
Message-ID:
In-Reply-To: <llvm/llvm-project/pull/66575/mlir at github.com>
ftynse wrote:
IMO, small changes like adding parameters is fine to just land. So are simple renamings. Those are rather obvious to detect and fix by simply reading the log of recent commits. What makes this stand out is that it reuses the name for an almost completely different operation. Imagine changing `scf.for` to become a "while" loop while retaining the name, this sounds extreme but conceptually close here.
https://github.com/llvm/llvm-project/pull/66575
More information about the Mlir-commits
mailing list