-
Notifications
You must be signed in to change notification settings - Fork 49
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
GCC 6.x seems working #78
base: master
Are you sure you want to change the base?
Conversation
Hi bebbo! I'm suprised to see that someone works on gcc 6.x. That's great news! I find your work to be valuable for the community and I'd like it to be merged. However I find the sheer size and scope of the pull request to be overwhelming. I could review and merge changes provided that they're relatively small (up to 100 lines?) and goal focused (no unnecessary changes, one funcionality at a time). Do you have any testing procedure so that you know your changes are correct? How can you prove nothing got broken by a modification? After all binutils and gcc are complex beasts. For know I find your PR to be a proof of concept rather than a complete solution. For the starter I'd like you to merge changes from submodules with respective master branches (e.g. binutils-2.14 and so on). Thanks for good work! I'm looking forward to your feedback. |
…o enable make via command line
…h ellipsis are not inline candidates
…h asm parameters
…orks Update instructions
update mcc_nlist.sdk
It's time to merge the changes:
pick what you want :-)