[llvm] [mlir] [LLVM][MLIR] Move LSP server support library from MLIR into LLVM (PR #157885)
Jan Ječmen via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 16 05:30:16 PDT 2025
JanJecmen wrote:
@mgorny Would it be possible to get the actual output of the test by running `/path/to/mlir-pdll-lsp-server -lit-test < ./mlir/test/mlir-pdll-lsp-server/view-output.test` and posting it here? Then we can compare with the lit test source to see how the expected output differs.
This is a strange failure because the PR should be NFC, just moving code to a different place. Either something really got broken, or it could be some small detail, perhaps the JSON object being serialized in a different order? I checked and it uses DenseMap as storage, and AFAICT there is no guarantee about preserving insertion order.
https://github.com/llvm/llvm-project/pull/157885
More information about the llvm-commits
mailing list