Hi, On Mon, Jan 23, 2017 at 02:11:37PM +0100, Richard Biener wrote: > On Mon, Jan 23, 2017 at 1:02 PM, Martin Jambor <mjam...@suse.cz> wrote: > > Hi, > > > > > > On Mon, Jan 23, 2017 at 12:56:13PM +0100, Richard Biener wrote: > >> On Fri, Jan 20, 2017 at 6:25 PM, Pekka Jääskeläinen <pe...@parmance.com> > >> wrote: > >> > Hi Richard, > >> > > >> > On Fri, Jan 20, 2017 at 10:26 AM, Richard Biener > >> > <richard.guent...@gmail.com> wrote: > >> >> So the #ifdef ENABLE_BRIG_FE shouldn't be needed anymore (nor the > >> >> configury for it). > >> >> > >> >> Otherwise this looks ok to me then. > >> > > >> > Attached is a patch set with that unnecessary ENABLE_BRIG_FE macro > >> > removed. It has also been refreshed to the latest trunk. > >> > >> Are you set up to commit these to trunk yourself or do you need help here? > >> > > > > Pekka is still in the process of requesting an account at gcc.gnu.org. > > We agreed that I would commit the patches. > > > > Over Saturday and today morning, I have bootstrappet and tested them > > in various configurations (although mostly on x86_64 and only a little > > bit on aarch64) and (together with a "svn mv libgomp/plugin/hsa.h > > include/") I have not found any issue so far. > > > > I suppose I should go ahead (in any case, if there is any fallout, > > reach out to me too, at least until Pekka gets his account). > > Yes, sounds good to me. >
It took me more time because I got distracted by PR 79198 and did not want to commit another huge thing while bootstrap was still broken for so many people with my previous patch. Nevertheless, I have just committed the BRIG FE as revision 244867. I tried to be careful (as well as quick to avoid conflicts) so I hope there are no issues. However if there are some, throw complaints my way at least unless Pekka gets an account at gcc.gnu.org. Thanks, Martin
