-
Notifications
You must be signed in to change notification settings - Fork 73
fix: warnings for 'c-run-time.c' #1706
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
If the premise isn't valid, please suggest an alternative and I'll start over. |
This passes 3-stage bootstrap and
|
@cgay |
Creates different arities for DFLN. typedef dylan_value (*DLFN0)(void); typedef dylan_value (*DLFN1)(dylan_value); typedef dylan_value (*DLFN2)(dylan_value, dylan_value); ... etc Changing DLFN to a union typedef union { void* raw; DLFN0 mep0; DLFN1 mep1; DLFN2 mep2; DLFN3 mep3; DLFN4 mep4; DLFN5 mep5; DLFN6 mep6; DLFN7 mep7; DLFN8 mep8; DLFN9 mep9; DLFN10 mep10; DLFN64 mep64; } DLFN; Then rewrite the call logic using the correct type per arity. switch (teb->argument_count) { case 0: return (mep.mep0)(); case 1: return (mep.mep1)(v[0]); ... Closes dylan-lang#1650
Remove 'priority_map' function that is unused.
Setting the default backend compiler to C and doing bootstrap, in the stage-2, it shows this errors:
|
Creates different arities for DFLN.
typedef dylan_value (*DLFN0)(void);
typedef dylan_value (*DLFN1)(dylan_value);
typedef dylan_value (*DLFN2)(dylan_value, dylan_value);
... etc
Changing DLFN to a union
typedef union {
void* raw;
DLFN0 mep0;
DLFN1 mep1;
DLFN2 mep2;
DLFN3 mep3;
DLFN4 mep4;
DLFN5 mep5;
DLFN6 mep6;
DLFN7 mep7;
DLFN8 mep8;
DLFN9 mep9;
DLFN10 mep10;
DLFN64 mep64;
} DLFN;
Then rewrite the call logic using the correct
type per arity.
switch (teb->argument_count) {
case 0: return (mep.mep0)();
case 1: return (mep.mep1)(v[0]);
...
Closes #1650