[all-commits] [llvm/llvm-project] 1f4672: [polynomial] Move primitive root attribute to ntt/...

[Jeremy Kun via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1f46729a18ef13c3ba4184ead1da4ab3037cb7ae
      https://github.com/llvm/llvm-project/commit/1f46729a18ef13c3ba4184ead1da4ab3037cb7ae
  Author: Jeremy Kun <jkun at google.com>
  Date:   2024-05-30 (Thu, 30 May 2024)

  Changed paths:
    M mlir/include/mlir/Dialect/Polynomial/IR/Polynomial.td
    M mlir/include/mlir/Dialect/Polynomial/IR/PolynomialAttributes.td
    M mlir/lib/Dialect/Polynomial/IR/PolynomialCanonicalization.td
    M mlir/lib/Dialect/Polynomial/IR/PolynomialOps.cpp
    M mlir/test/Dialect/Polynomial/canonicalization.mlir
    M mlir/test/Dialect/Polynomial/ops.mlir
    M mlir/test/Dialect/Polynomial/ops_errors.mlir

  Log Message:
  -----------
  [polynomial] Move primitive root attribute to ntt/intt ops. (#93227)

Better design to put semantics on the ops, and in this case the ntt/intt
op can lower in multiple ways depending on the polynomial ring modulus
(it can need an nth root of unity for cyclic polymul -> ntt, or a 2nth
root for negacyclic polymul -> ntt)

---------

Co-authored-by: Jeremy Kun <j2kun at users.noreply.github.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications