Skip to content

[mlir] fix polynomial docs for MLIR website #92348

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

j2kun
Copy link
Contributor

@j2kun j2kun commented May 16, 2024

I built it and confirmed this fixes the issue locally.

@j2kun j2kun requested a review from joker-eph May 16, 2024 05:01
@llvmbot llvmbot added the mlir label May 16, 2024
@llvmbot
Copy link
Member

llvmbot commented May 16, 2024

@llvm/pr-subscribers-mlir

Author: Jeremy Kun (j2kun)

Changes

I built it and confirmed this fixes the issue locally.


Full diff: https://github.com/llvm/llvm-project/pull/92348.diff

2 Files Affected:

  • (removed) mlir/docs/Dialects/Polynomial.md (-2)
  • (modified) mlir/include/mlir/Dialect/Polynomial/IR/CMakeLists.txt (-1)
diff --git a/mlir/docs/Dialects/Polynomial.md b/mlir/docs/Dialects/Polynomial.md
deleted file mode 100644
index 6fdd74ab30aa6..0000000000000
--- a/mlir/docs/Dialects/Polynomial.md
+++ /dev/null
@@ -1,2 +0,0 @@
-[include "Dialects/PolynomialDialect.td"]
-[include "Dialects/PolynomialOps.td"]
diff --git a/mlir/include/mlir/Dialect/Polynomial/IR/CMakeLists.txt b/mlir/include/mlir/Dialect/Polynomial/IR/CMakeLists.txt
index c8655ba998701..e8f48f13be2b8 100644
--- a/mlir/include/mlir/Dialect/Polynomial/IR/CMakeLists.txt
+++ b/mlir/include/mlir/Dialect/Polynomial/IR/CMakeLists.txt
@@ -1,6 +1,5 @@
 add_mlir_dialect(Polynomial polynomial)
 add_mlir_doc(Polynomial PolynomialDialect Dialects/ -gen-dialect-doc -dialect=polynomial)
-add_mlir_doc(Polynomial PolynomialOps Dialects/ -gen-op-doc)
 
 set(LLVM_TARGET_DEFINITIONS Polynomial.td)
 mlir_tablegen(PolynomialAttributes.cpp.inc -gen-attrdef-defs -attrdefs-dialect=polynomial)

Copy link

⚠️ We detected that you are using a GitHub private e-mail address to contribute to the repo.
Please turn off Keep my email addresses private setting in your account.
See LLVM Discourse for more information.

@j2kun j2kun merged commit 5bd8091 into llvm:main May 16, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants