Add --compiler-wrapper command line flag#12662
Conversation
This works just like the COMPILER_WRAPPER config settings but works from the command line. Fixes: #12340
|
This change replaces #12608 |
kripken
left a comment
There was a problem hiding this comment.
lgtm, simpler and we can generalize later if needed I guess.
Please document it in emcc.rst and in the changelog.
(Given it's not general, it seems like this could be a setting in settings.js actually? If it's special maybe the PR as is is best.)
That raises an interesting topic for discussion I think. If we want to add a user flag should we add it as an EmccOption or as a settings.js options... My mental model is that if the option only effects that python code in emcc.py then we should lean towards a new command line option. If the setting is needed also in JS then it needs to be in settings.js. As a side node, adding things to settings JS unnecessarily also increases code size for users of |
This works just like the COMPILER_WRAPPER config settings
but works from the command line.
Fixes: #12340