Skip to content

Add --as-flags#34

Merged
mkst merged 6 commits intomainfrom
as-flags
Feb 18, 2026
Merged

Add --as-flags#34
mkst merged 6 commits intomainfrom
as-flags

Conversation

@mkst
Copy link
Owner

@mkst mkst commented Feb 16, 2026

No description provided.

Copy link
Contributor

@laura-a-n-n laura-a-n-n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay thanks so much! this does solve my issue :)

there seems to be one problem, which is that I can only pass one arg to as_flags. if I pass it as a string, like as_flags="-x -y", it isn't interpreted correctly, and omitting quotes makes -y get passed to MWCC instead of AS. this doesn't affect my work currently, but worth noting -- am I doing something wrong?

@mkst
Copy link
Owner Author

mkst commented Feb 17, 2026

Foolishly I had only tested with --as-flags foo bar baz rather than real flags with dashes in them.

I've tweaked the logic as best I could, but it means --as-flags needs to be right at the end of your mwccgap.py command, and all flags will be treated as arguments for the assembler.

e.g.

python3 mwccgap.py foo.c foo.o -foo -bar baz --mwcc-path mwccpsp.exe --as-flags -G0 -hello -laura

will pass -foo -bar -baz arguments to the MWCC compiler, and -G0 -hello -laura to GNU as...

but if you did

python3 mwccgap.py foo.c foo.o --as-flags -G0 -hello -laura --mwcc-path mwccpsp.exe -foo -bar baz

then -G0 -hello -laura --mwcc-path mwccpsp.exe -foo -bar baz will be passed to GNU as!

Copy link
Contributor

@laura-a-n-n laura-a-n-n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this works for me! i had to rearrange the flags a bit, but i can pass more than one now :D

@mkst mkst merged commit 9335bca into main Feb 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments