public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH][DOCS] Document make jobserver related changes for GCC 13.
@ 2022-08-18 11:33 Martin Liška
  2022-08-18 20:53 ` Gerald Pfeifer
  0 siblings, 1 reply; 3+ messages in thread
From: Martin Liška @ 2022-08-18 11:33 UTC (permalink / raw)
  To: gcc-patches

Ready for master?

---
 htdocs/gcc-13/changes.html | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/htdocs/gcc-13/changes.html b/htdocs/gcc-13/changes.html
index 57bd8724..95fee90d 100644
--- a/htdocs/gcc-13/changes.html
+++ b/htdocs/gcc-13/changes.html
@@ -52,6 +52,15 @@ a work-in-progress.</p>
     AddressSanitizer defaults to <code>detect_stack_use_after_return=1</code> on GNU/Linux targets.
     For compatibility, it can be disabled with <code>env ASAN_OPTIONS=detect_stack_use_after_return=0</code>.
   </li>
+  <li>
+    Link-time optimization improvements:
+    <ul>
+        <li>LTO supports the newly added GNU make's jobserver that uses named pipes (<code>--jobserver-style=fifo</code>)
+        by default.</li>
+        <li>If make's jobserver is active, parallel LTO WPA streaming communicates with it and so the streaming
+        does not lead to the system overcommitting.</li>
+    </ul>
+  </li>
 </ul>
 
 
-- 
2.37.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH][DOCS] Document make jobserver related changes for GCC 13.
  2022-08-18 11:33 [PATCH][DOCS] Document make jobserver related changes for GCC 13 Martin Liška
@ 2022-08-18 20:53 ` Gerald Pfeifer
  2022-08-22  9:24   ` Martin Liška
  0 siblings, 1 reply; 3+ messages in thread
From: Gerald Pfeifer @ 2022-08-18 20:53 UTC (permalink / raw)
  To: Martin Liška; +Cc: gcc-patches

On Thu, 18 Aug 2022, Martin Liška wrote:
> Ready for master?

Nearly. :)

> +    Link-time optimization improvements:
> +    <ul>
> +        <li>LTO supports the newly added GNU make's jobserver that uses named pipes (<code>--jobserver-style=fifo</code>)

I believe that's just "GNU make" (instead of "GNU make's"), or maybe 
"...the newly added jobserver of GNU make...".

Not 100% sure myself.

> +        <li>If make's jobserver is active, parallel LTO WPA streaming 
>    communicates with it and so the streaming
> +        does not lead to the system overcommitting.</li>

Might we be able to simplify this to something like

  "...parallel LTO WPA streaming communicates with it and thus avoids
  system overcommitting" ?


Just two minor points; the patch as such looks fine.

Thank you,
Gerald

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH][DOCS] Document make jobserver related changes for GCC 13.
  2022-08-18 20:53 ` Gerald Pfeifer
@ 2022-08-22  9:24   ` Martin Liška
  0 siblings, 0 replies; 3+ messages in thread
From: Martin Liška @ 2022-08-22  9:24 UTC (permalink / raw)
  To: Gerald Pfeifer; +Cc: gcc-patches

On 8/18/22 22:53, Gerald Pfeifer wrote:
> On Thu, 18 Aug 2022, Martin Liška wrote:
>> Ready for master?
> 
> Nearly. :)
> 
>> +    Link-time optimization improvements:
>> +    <ul>
>> +        <li>LTO supports the newly added GNU make's jobserver that uses named pipes (<code>--jobserver-style=fifo</code>)
> 
> I believe that's just "GNU make" (instead of "GNU make's"), or maybe 
> "...the newly added jobserver of GNU make...".
> 
> Not 100% sure myself.
> 
>> +        <li>If make's jobserver is active, parallel LTO WPA streaming 
>>    communicates with it and so the streaming
>> +        does not lead to the system overcommitting.</li>
> 
> Might we be able to simplify this to something like
> 
>   "...parallel LTO WPA streaming communicates with it and thus avoids
>   system overcommitting" ?
> 
> 
> Just two minor points; the patch as such looks fine.

Thanks for the review, installed with your suggestions.

Martin

> 
> Thank you,
> Gerald


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-08-22  9:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-18 11:33 [PATCH][DOCS] Document make jobserver related changes for GCC 13 Martin Liška
2022-08-18 20:53 ` Gerald Pfeifer
2022-08-22  9:24   ` Martin Liška

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).