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. Richard. > Thanks, > > Martin >