Auto merge of #120771 - oli-obk:useless_non_ensure_query_call, r=davi…

…dtwco

Use `ensure` when the result of the query is not needed beyond its `Result`ness

while I would like to just remove the `tcx` methods for ensure-only queries, that is hard to do without another query annotation or by turning the `define_callbacks` macro into a proc macro to get more control

should fix perf regression of #120558

compiler/rustc_hir_typeck/src/callee.rs

@@ -53,7 +53,7 @@ pub fn check_legal_trait_for_method_call(
         };
         return Err(tcx.dcx().emit_err(errors::ExplicitDestructorCall { span, sugg }));
     }
-    tcx.coherent_trait(trait_id)
+    tcx.ensure().coherent_trait(trait_id)
 }
 
 #[derive(Debug)]

compiler/rustc_trait_selection/src/traits/specialize/mod.rs

@@ -446,7 +446,7 @@ fn report_conflicting_impls<'tcx>(
     match used_to_be_allowed {
         None => {
             let reported = if overlap.with_impl.is_local()
-                || tcx.orphan_check_impl(impl_def_id).is_ok()
+                || tcx.ensure().orphan_check_impl(impl_def_id).is_ok()
             {
                 let mut err = tcx.dcx().struct_span_err(impl_span, msg);
                 err.code(E0119);