-
Notifications
You must be signed in to change notification settings - Fork 167
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
Rebasing against GCC #247
Comments
@philberty, do you really intend to |
Good point, I have been assuming that in the event of going upstream to GCC this would require a rebase against GCC so all the front-end work is ontop at that point but if merges are ok then let's do that. |
Eventually integrating this work into GCC upstream would -- per current upstream rules; I didn't invent these ;-) -- not be done as a |
..., and of course, any changes to existing GCC code (bug fixes, enhancements, etc.) shall be contributed individually, at any point in time, outside of the big "Add Rust front end" commit. Also, adding run-time libraries, etc. is often done in separate commits from the one adding the front end proper. We'll sort that out once we get to that point. ;-) |
I think we've been using |
@NalaGinrut: a |
@tschwinge @philberty Well, I realized that you guys are talking about using What I mean is to keep using |
As again mentioned in today's Status Report Call, I'm working on this. Aside from the GCC/Rust-specific files (not existing in GCC upstream), we also carry a lot of changes in |
…[...]' [#247] ... in preparation for a merge from GCC upstream, where the former disappear.
690: A bit of 'RichLocation' C++ tuning [#247], [#97, #374] r=philberty a=tschwinge ... in preparation for a merge from GCC upstream, where we otherwise run into several different build errors. Follow-up to commit ed651fc "Add basic wrapper over gcc rich_location". Co-authored-by: Thomas Schwinge <[email protected]>
689: x86: Instead of 'TARGET_ISA_[...]', 'TARGET_ISA2_[...]', use 'TARGET_[...]' [#247] r=philberty a=tschwinge ... in preparation for a merge from GCC upstream, where the former disappear. Co-authored-by: Thomas Schwinge <[email protected]>
…obs.build-and-check' That's what I use in my local development enviroment, and I suppose I'll be the main one touching this file semi-regularly in context of <Rust-GCC#247> "Rebasing against GCC".
…obs.build-and-check' That's what I use in my local development enviroment, and I suppose I'll be the main one touching this file semi-regularly in context of <Rust-GCC#247> "Rebasing against GCC".
Looks like we can close this issue since we can merge from upstream rather than a rebase |
…obs.build-and-check' That's what I use in my local development enviroment, and I suppose I'll be the main one touching this file semi-regularly in context of <#247> "Rebasing against GCC".
Last time the front-end was rebased against GCC Master was on:
b8ae081
We need a strategy for when we rebase the front-end as this requires a force push back to master
The text was updated successfully, but these errors were encountered: