[Mlir-commits] [mlir] ed7dce2 - Cleanup the MLIR LSP doc to remove references to the passes
Mehdi Amini
llvmlistbot at llvm.org
Thu Jun 15 09:56:38 PDT 2023
Author: Mehdi Amini
Date: 2023-06-15T18:56:19+02:00
New Revision: ed7dce23c2d234e4bf8ad573ce3453d9acab47f0
URL: https://github.com/llvm/llvm-project/commit/ed7dce23c2d234e4bf8ad573ce3453d9acab47f0
DIFF: https://github.com/llvm/llvm-project/commit/ed7dce23c2d234e4bf8ad573ce3453d9acab47f0.diff
LOG: Cleanup the MLIR LSP doc to remove references to the passes
The MLIR LSP server initially had plans to support running passes from the
client, but this was never implemented. The doc unnecessarily advise to
register passes and makes reference to LSP server having some interaction
with passes.
Differential Revision: https://reviews.llvm.org/D153046
Added:
Modified:
mlir/docs/Tools/MLIRLSP.md
Removed:
################################################################################
diff --git a/mlir/docs/Tools/MLIRLSP.md b/mlir/docs/Tools/MLIRLSP.md
index 733b2cecd0830..b6c367f249b16 100644
--- a/mlir/docs/Tools/MLIRLSP.md
+++ b/mlir/docs/Tools/MLIRLSP.md
@@ -17,16 +17,16 @@ MLIR provides an implementation of an LSP language server for `.mlir` text files
in the form of the `mlir-lsp-server` tool. This tool interacts with the MLIR C++
API to support rich language queries, such as "Find Definition".
-### Supporting custom dialects and passes
+### Supporting custom dialects
`mlir-lsp-server`, like many other MLIR based tools, relies on having the
appropriate dialects registered to be able to parse in the custom assembly
formats used in the textual .mlir files. The `mlir-lsp-server` found within the
-main MLIR repository provides support for all of the upstream MLIR dialects and
-passes. Downstream and out-of-tree users will need to provide a custom
+main MLIR repository provides support for all of the upstream MLIR dialects.
+Downstream and out-of-tree users will need to provide a custom
`mlir-lsp-server` executable that registers the entities that they are
interested in. The implementation of `mlir-lsp-server` is provided as a library,
-making it easy for downstream users to register their dialect/passes and simply
+making it easy for downstream users to register their dialect and simply
call into the main implementation. A simple example is shown below:
```c++
@@ -35,7 +35,6 @@ call into the main implementation. A simple example is shown below:
int main(int argc, char **argv) {
mlir::DialectRegistry registry;
registerMyDialects(registry);
- registerMyPasses();
return mlir::failed(mlir::MlirLspServerMain(argc, argv, registry));
}
```
More information about the Mlir-commits
mailing list