about summary refs log tree commit diff
path: root/compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp
diff options
context:
space:
mode:
authorFrancis McKenzie <info@macca.tech>2024-06-29 13:06:28 +0800
committerFrancis McKenzie <info@macca.tech>2024-07-28 00:32:37 +0800
commit1261827eb9332b9595994d9be67ba9da80f1a644 (patch)
tree8fba1154fc33b9256d24ccbe60e341c290d8b8bf /compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp
parent34dc7ac79f6a2fe20e2fb67f106566ded456c7dd (diff)
downloadrust-1261827eb9332b9595994d9be67ba9da80f1a644.tar.gz
rust-1261827eb9332b9595994d9be67ba9da80f1a644.zip
Fix for #17497 - Invalid RA diagnostic error: expected 2 arguments, found 1
The issue occurs because in some configurations of traits where one of them has Deref as a supertrait, RA's type inference algorithm fails to resolve the Deref::Target type, and instead uses a TyKind::BoundVar (i.e. an unknown type). This "autoderefed" type then incorrectly acts as if it implements all traits in scope.

The fix is to re-apply the same sanity-check that is done in iterate_method_candidates_with_autoref(), that is: don't try to resolve methods on unknown types. This same sanity-check is now done on each autoderefed type for which trait methods are about to be checked. If the autoderefed type is unknown, then the iterating of the trait methods for that type is skipped.

Includes a unit test that only passes after applying the fixes in this commit.

Includes a change to the assertion count in test syntax_highlighting::tests::benchmark_syntax_highlighting_parser as suggested by Lukas Wirth during review.

Includes a change to the sanity-check code as suggested by Florian Diebold during review.
Diffstat (limited to 'compiler/rustc_llvm/llvm-wrapper/CoverageMappingWrapper.cpp')
0 files changed, 0 insertions, 0 deletions