fix crash with tyBuiltInTypeClass matching itself (#24462)

fixes #24449

The standalone `seq` type is a `tyBuiltInTypeClass` with a single child
of kind `tySequence`, which itself has no children. This is also the
case for most other `tyBuiltInTypeClass` kinds. However this can cause a
crash in sigmatch when calling `isEmptyContainer` on this child type,
which expects the sequence type to have children. This check was added
in #5557 to prevent empty collections like `@[]` from matching their
respective typeclass, but it's not useful when matching against another
typeclass (which is done here to resolve an ambiguity). So to avoid the
crash, this empty container check is disabled when matching against
another typeclass.

tests/overload/tvartypeclass.nim

@@ -1,11 +1,16 @@
-# issue #13302
+block: # issue #13302
+  proc foo(x: object): int = x.i*2
+  proc foo(x: var object) = x.i*=2
+  type Foo = object
+    i: int
+  let x = Foo(i: 3)
+  var y = Foo(i: 4)
+  doAssert foo(x) == 6
+  foo(y)
+  doAssert y.i == 8
 
-proc foo(x: object): int = x.i*2
-proc foo(x: var object) = x.i*=2
-type Foo = object
-  i: int
-let x = Foo(i: 3)
-var y = Foo(i: 4)
-doAssert foo(x) == 6
-foo(y)
-doAssert y.i == 8
+block: # issue #24449
+  proc p(x: var seq)= discard
+  proc p(x: seq)= discard
+  var s : seq[int]
+  p(s)