[llvm] [mlir] Poly canonicalization (PR #91410)

Jeremy Kun via llvm-commits llvm-commits at lists.llvm.org
Fri May 17 10:17:40 PDT 2024


================
@@ -1,3 +1,7 @@
+set(LLVM_TARGET_DEFINITIONS PolynomialCanonicalization.td)
+mlir_tablegen(PolynomialCanonicalization.inc -gen-rewriters)
+add_public_tablegen_target(MLIRPolynomialCanonicalizationIncGen)
----------------
j2kun wrote:

In principle I agree (added in 0e25a328d9d05c080bef4e81df276b4f5a2b3a77).

In practice I am baffled by how CI and my local tests are passing without this dependency added. Excuse my build system naïveté, but maybe someone could help me understand why? Without that, I can't even be 100% sure I added the dependency in the right place.

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


More information about the llvm-commits mailing list