Skip to content
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

Open
wants to merge 394 commits into
base: master
Choose a base branch
from
Open

GCC 6.x seems working #78

wants to merge 394 commits into from

Conversation

bebbo
Copy link
Contributor

@bebbo bebbo commented Mar 26, 2017

It's time to merge the changes:

  • gcc 6.3.1 seems working
  • gcc 6.3.1 produces good code
  • enabled also objective c
  • various fixes here and there

pick what you want :-)

@cahirwpz
Copy link
Member

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.

bebbo added 28 commits April 26, 2017 10:34
…xed propagate moves, @b fixed eliminate dead assignments
…converted int operations on address reg
…h ellipsis are not inline candidates
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.

6 participants