Changes

Summary

  1. [OpAsmParser] Refactor parseOptionalInteger to support wide integers, NFC. (details)
  2. [LLD] Improve reporting unresolved symbols in shared libraries (details)
  3. [mlir][linalg] Remove IndexedGenericOp support from Tiling... (details)
  4. [clangd][index-sever] Limit results in repsonse (details)
  5. [clangd][remote-client] Set HasMore to true for failure (details)
Commit 2b09a89daf956795d82076d983c3d78b96e1af4b by clattner
[OpAsmParser] Refactor parseOptionalInteger to support wide integers, NFC.

OpAsmParser (and DialectAsmParser) supports a pair of
parseInteger/parseOptionalInteger methods, which allow parsing a bare
integer into a C type of your choice (e.g. int8_t) using templates.  It
was implemented in terms of a virtual method call that is hard coded to
int64_t because "that should be big enough".

Change the virtual method hook to return an APInt instead.  This allows
asmparsers for custom ops to parse large integers if they want to, without
changing any of the clients of the fixed size C API.

Differential Revision: https://reviews.llvm.org/D102120
The file was modifiedmlir/lib/Parser/Parser.h
The file was modifiedmlir/include/mlir/IR/OpImplementation.h
The file was modifiedmlir/lib/Parser/DialectSymbolParser.cpp
The file was modifiedmlir/include/mlir/IR/DialectImplementation.h
The file was modifiedmlir/lib/Parser/Parser.cpp
Commit 70c23e232e50b190c9e0d8e4b5b6a8ddfc19b80c by ikudrin
[LLD] Improve reporting unresolved symbols in shared libraries

Currently, when reporting unresolved symbols in shared libraries, if an
undefined symbol is firstly seen in a regular object file that shadows
the reference for the same symbol in a shared object. As a result, the
error for the unresolved symbol in the shared library is not reported.
If referencing sections in regular object files are discarded because of
'--gc-sections', no reports about such symbols are generated, and the
linker finishes successfully, generating an output image that fails on
the run.

The patch fixes the issue by keeping symbols, which should be checked,
for each shared library separately.

Differential Revision: https://reviews.llvm.org/D101996
The file was modifiedlld/test/ELF/allow-shlib-undefined.s
The file was modifiedlld/ELF/Writer.cpp
The file was modifiedlld/ELF/InputFiles.cpp
The file was modifiedlld/ELF/InputFiles.h
Commit d69bccf1ed30d16e043d4bb71b4ebd6100efa75b by gysit
[mlir][linalg] Remove IndexedGenericOp support from Tiling...

after introducing the IndexedGenericOp to GenericOp canonicalization (https://reviews.llvm.org/D101612).

Differential Revision: https://reviews.llvm.org/D102176
The file was modifiedmlir/test/Dialect/Linalg/tile-tensors.mlir
The file was modifiedmlir/lib/Dialect/Linalg/Transforms/Tiling.cpp
The file was removedmlir/test/Dialect/Linalg/tile-indexed-generic.mlir
Commit daf3cb3b8a5868d9089a69025c556b564615b844 by kadircet
[clangd][index-sever] Limit results in repsonse

This is to prevent server from being DOS'd by possible malicious
parties issuing requests that can yield huge responses.

One possible drawback is on rename workflow. As it really requests all
occurences, but it has an internal limit on 50 files currently.
We are putting the limit on 10000 elements per response So for rename to regress
one should have 10k refs to a symbol in less than 50 files. This seems unlikely
and we fix it if there are complaints by giving up on the response based on the
number of files covered instead.

Differential Revision: https://reviews.llvm.org/D101914
The file was addedclang-tools-extra/clangd/test/remote-index/result-limiting.test
The file was modifiedclang-tools-extra/clangd/index/remote/server/Server.cpp
Commit 888307ee625b50b060317e2100fb16e0be2626b7 by kadircet
[clangd][remote-client] Set HasMore to true for failure

Currently client was setting the HasMore to true iff stream said so.
Hence if we had a broken stream for whatever reason (e.g. hitting deadline for a
huge response), HasMore would be false, which is semantically incorrect (e.g.
will throw rename off).

Differential Revision: https://reviews.llvm.org/D101915
The file was modifiedclang-tools-extra/clangd/index/remote/Client.cpp