Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-03-02 Thread Martin Liška
On 2/27/20 9:44 AM, Jonathan Wakely wrote: Martin, the comment in your patch says "-std=c11" which should be c++11. Sure, thanks for pointing to it. I'm going to install the obvious patch for it. Martin >From 3c633731ade4cdad56f5729eaedb76cb2f727c39 Mon Sep 17 00:00:00 2001 From: Martin Liska

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-28 Thread Paul Smith
On Fri, 2020-02-28 at 17:37 +0100, Martin Jambor wrote: > > Recently Honza, me and others discussed LTO's interaction with > > build systems, and that perhaps the module mapper could be > > generalized for other purposes. (Yes, still need to resurrect my > > Make PoC) > > see also the "Create a g

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-28 Thread Martin Jambor
Hi, On Fri, Feb 28 2020, Nathan Sidwell wrote: > On 2/27/20 12:04 PM, Paul Smith wrote: >> On Thu, 2020-02-27 at 16:58 +, Jonathan Wakely wrote: That's a problem then as were assuming a user's build system for this to work. I mean for now its fine but in the future wouldn't it de a >

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-28 Thread Nathan Sidwell
On 2/27/20 12:04 PM, Paul Smith wrote: On Thu, 2020-02-27 at 16:58 +, Jonathan Wakely wrote: That's a problem then as were assuming a user's build system for this to work. I mean for now its fine but in the future wouldn't it de a good ideal to not assume this? It works fine for everybody.

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-27 Thread Paul Smith
On Thu, 2020-02-27 at 16:58 +, Jonathan Wakely wrote: > > That's a problem then as were assuming a user's build system for this > > to work. I mean for now its fine but in the future wouldn't it de a > > good ideal to not assume this? > > It works fine for everybody. There's just an optimisati

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-27 Thread Jonathan Wakely
On Thu, 27 Feb 2020 at 16:18, Nicholas Krause wrote: > > > > On 2/27/20 3:44 AM, Jonathan Wakely wrote: > > On Thu, 27 Feb 2020 at 06:50, Nicholas Krause wrote: > >> Greetings Martin, > >> > >> This patch: > >> https://gcc.gnu.org/git/?p=gcc.git;a=blobdiff;f=gcc/lto-wrapper.c;h=353187c60434f43a44

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-27 Thread Nicholas Krause
On 2/27/20 3:44 AM, Jonathan Wakely wrote: On Thu, 27 Feb 2020 at 06:50, Nicholas Krause wrote: Greetings Martin, This patch: https://gcc.gnu.org/git/?p=gcc.git;a=blobdiff;f=gcc/lto-wrapper.c;h=353187c60434f43a445e708dcfbf53c857f8cdc1;hp=946897726d03716f7c93f955c438ee4f8190044c;hb=f12fbeb53

Re: Make LTO Patch for Job Server Thread Detection Agnostic

2020-02-27 Thread Jonathan Wakely
On Thu, 27 Feb 2020 at 06:50, Nicholas Krause wrote: > > Greetings Martin, > > This patch: > https://gcc.gnu.org/git/?p=gcc.git;a=blobdiff;f=gcc/lto-wrapper.c;h=353187c60434f43a445e708dcfbf53c857f8cdc1;hp=946897726d03716f7c93f955c438ee4f8190044c;hb=f12fbeb535f192f742025cc4f9b69a48136730f1;hpb=80c7