diff options
author | Muhammad Mahad <mahadtxt@gmail.com> | 2023-08-15 20:19:36 +0500 |
---|---|---|
committer | CohenArthur <arthur.cohen@embecosm.com> | 2023-08-17 13:33:23 +0000 |
commit | d4bf5e3f23bfbbfc3b42ca523fc6f46cae064808 (patch) | |
tree | 49a82a00409fdf38abff4851d889478c138794de | |
parent | 9ef8689c7faeec4f6833bff39aa2301f14209c9a (diff) | |
download | gcc-d4bf5e3f23bfbbfc3b42ca523fc6f46cae064808.zip gcc-d4bf5e3f23bfbbfc3b42ca523fc6f46cae064808.tar.gz gcc-d4bf5e3f23bfbbfc3b42ca523fc6f46cae064808.tar.bz2 |
gccrs: [E0034] found more than one items for method
Multiple items found with same prototype.
Fixes: https://github.com/Rust-GCC/gccrs/issues/2366
gcc/rust/ChangeLog:
* typecheck/rust-hir-path-probe.h:
Fixes issue & added rich location message.
* typecheck/rust-hir-type-check-expr.cc (TypeCheckExpr::visit):
Added rich location and error code.
gcc/testsuite/ChangeLog:
* rust/compile/generics6.rs: Updated dejagnu comment.
* rust/compile/generics7.rs: likewise.
* rust/compile/issue-925.rs: likewise.
Signed-off-by: Muhammad Mahad <mahadtxt@gmail.com>
-rw-r--r-- | gcc/rust/typecheck/rust-hir-path-probe.h | 5 | ||||
-rw-r--r-- | gcc/rust/typecheck/rust-hir-type-check-expr.cc | 9 | ||||
-rw-r--r-- | gcc/testsuite/rust/compile/generics6.rs | 2 | ||||
-rw-r--r-- | gcc/testsuite/rust/compile/generics7.rs | 2 | ||||
-rw-r--r-- | gcc/testsuite/rust/compile/issue-925.rs | 2 |
5 files changed, 15 insertions, 5 deletions
diff --git a/gcc/rust/typecheck/rust-hir-path-probe.h b/gcc/rust/typecheck/rust-hir-path-probe.h index 09874f0..d829d9a 100644 --- a/gcc/rust/typecheck/rust-hir-path-probe.h +++ b/gcc/rust/typecheck/rust-hir-path-probe.h @@ -165,8 +165,11 @@ public: for (auto &c : candidates) r.add_range (c.locus); + std::string rich_msg = "multiple " + query.as_string () + " found"; + r.add_fixit_replace (rich_msg.c_str ()); + rust_error_at (r, ErrorCode::E0034, - "multiple applicable items in scope for: %s", + "multiple applicable items in scope for: %qs", query.as_string ().c_str ()); } }; diff --git a/gcc/rust/typecheck/rust-hir-type-check-expr.cc b/gcc/rust/typecheck/rust-hir-type-check-expr.cc index 3a252a6..1ce54c4 100644 --- a/gcc/rust/typecheck/rust-hir-type-check-expr.cc +++ b/gcc/rust/typecheck/rust-hir-type-check-expr.cc @@ -1065,11 +1065,18 @@ TypeCheckExpr::visit (HIR::MethodCallExpr &expr) if (candidates.size () > 1) { rich_location r (line_table, expr.get_method_name ().get_locus ()); + std::string rich_msg + = "multiple " + expr.get_method_name ().get_segment ().as_string () + + " found"; + for (auto &c : candidates) r.add_range (c.candidate.locus); + r.add_fixit_replace (rich_msg.c_str ()); + rust_error_at ( - r, "multiple candidates found for method %<%s%>", + r, ErrorCode::E0034, + "multiple applicable items in scope for method %qs", expr.get_method_name ().get_segment ().as_string ().c_str ()); return; } diff --git a/gcc/testsuite/rust/compile/generics6.rs b/gcc/testsuite/rust/compile/generics6.rs index 00b93c3..33093cf 100644 --- a/gcc/testsuite/rust/compile/generics6.rs +++ b/gcc/testsuite/rust/compile/generics6.rs @@ -26,7 +26,7 @@ impl Foo<f32> { } fn main() { - let a: i32 = Foo::test(); // { dg-error "multiple applicable items in scope for: test" } + let a: i32 = Foo::test(); // { dg-error "multiple applicable items in scope for: .test." } // { dg-error {Failed to resolve expression of function call} "" { target *-*-* } .-1 } } diff --git a/gcc/testsuite/rust/compile/generics7.rs b/gcc/testsuite/rust/compile/generics7.rs index f2cd607..ae0dc7e 100644 --- a/gcc/testsuite/rust/compile/generics7.rs +++ b/gcc/testsuite/rust/compile/generics7.rs @@ -26,5 +26,5 @@ impl<T> Foo<T> { fn main() { let a = Foo { a: 123 }; a.bar(); - // { dg-error "multiple candidates found for method .bar." "" { target *-*-* } .-1 } + // { dg-error "multiple applicable items in scope for method .bar." "" { target *-*-* } .-1 } } diff --git a/gcc/testsuite/rust/compile/issue-925.rs b/gcc/testsuite/rust/compile/issue-925.rs index 2d536a5..d5eed7e 100644 --- a/gcc/testsuite/rust/compile/issue-925.rs +++ b/gcc/testsuite/rust/compile/issue-925.rs @@ -21,5 +21,5 @@ impl B for S { fn test() { let a = S; a.foo(); - // { dg-error "multiple candidates found for method .foo." "" { target *-*-* } .-1 } + // { dg-error "multiple applicable items in scope for method .foo." "" { target *-*-* } .-1 } } |