The macro-in-macro check seems insufficient.

Original discussion about disallowing macro in macro: https://github.com/HaxeFoundation/haxe/issues/7348

It seems the check only occurs when actually calling a macro from a macro:

```haxe
class Test {
static function main() {
#if !macro
for (v in getValues())
trace(v);
#end
}
public static macro function getValues() {
getValues();// Uncaught exception macro-in-macro
return macro [];
}
}
```

But not when calling a macro in macro context:

```haxe
class Test {
static function main() {
for (v in getValues()) // You can't iterate on a Dynamic value, please specify Iterator or Iterable
trace(v);
}
public static macro function getValues() {
return macro [];
}
}
```

Does that really make sense?