Index.Basic: avoid Name.quickCmp when comparing IndexMatchLocations

Name.quickCmp makes name comparisons less predictable since for names
containing a numeric part, such as `_uniq.123`, the numeric part is
considered first. This was probably the cause of flakiness in
`AesopTest.RulePattern`, where the order of forward hypotheses would
change due to seemingly unrelated changes.

Aesop/Index/Basic.lean

@@ -76,7 +76,7 @@ instance : Ord IndexMatchLocation where
     | none, hyp .. => .lt
     | hyp .., target => .gt
     | hyp .., none => .gt
-    | hyp ldecl₁, hyp ldecl₂ => ldecl₁.fvarId.name.quickCmp ldecl₂.fvarId.name
+    | hyp ldecl₁, hyp ldecl₂ => ldecl₁.fvarId.name.cmp ldecl₂.fvarId.name
 
 instance : Hashable IndexMatchLocation where
   hash