Rollup merge of #122820 - oli-obk:no_ord_def_id, r=estebank

Stop using `<DefId as Ord>` in various diagnostic situations

work towards rust-lang/rust#90317

Reverts part of rust-lang/rust#106281, as it sorts constants and that's problematic since it can contain `ParamConst`, which contains `DefId`s

Author: matthiaskrgr