On Tue, 2018-10-23 at 19:21 +0200, Iain Buclaw wrote: > On Tue, 23 Oct 2018 at 15:48, Richard Sandiford > <richard.sandif...@arm.com> wrote: > > > > Iain Buclaw <ibuc...@gdcproject.org> writes: > > > I'm just going to post the diff since the original here, just to > > > show > > > what's been done since review comments. > > > > > > I think I've covered all that's been addressed, except for the > > > couple > > > of notes about the quadratic parts (though I think one of them is > > > actually O(N^2)). I've raised bug reports on improving them > > > later. > > > > > > I've also rebased them against trunk, so there's a couple new > > > things > > > present that are just to support build. > > > > Thanks, this is OK when the frontend is accepted in principle > > (can't remember where things stand with that). > > > > As discussed, the front-end has already been approved by the SC. > > I'm not sure if there's anything else further required, or if any > final review needs to be done. > > Thanks.
I'm wondering what the state of this is [1] Iain: are all of the patches individually approved, with the necessary issues fixed? IIRC, the front-end as a whole was approved, pending approval of all of the individual patches (URL?). If that's done, then presumably this is good to go in - unless there was still some license discussion pending? I take it that you've already got your contributor paperwork in place, right? I see from your maintainers commit that you presumably have svn access. I'm not a global reviewer or steering committee member though; would be nice to get a "go for it" from one of those. Richard is a global reviewer. I'm not sure if it should be one big mega-commit, or split out the same way you split things out for review. Thanks for all your work on this Dave [1] I've been checking the git mirror every few hours to look for a massive commit from you, if I'm honest :)