On 8/6/19 6:04 AM, Martin Liška wrote:
Hi.

The patch is about proper checking of file descriptors on Windows.

Patch can bootstrap on x86_64-linux-gnu and survives regression tests.

Is there a way to share the definition of the new function so
it doesn't have to be duplicated?

Other than that, I'm wondering if the guard for F_GETFD is
necessary and when (the original code didn't guard its use).

Martin



@Pekka: Can you please test it on Windows?

Ready to be installed?
Thanks,
Martin

gcc/ChangeLog:

2019-08-06  Martin Liska  <mli...@suse.cz>

        PR bootstrap/91352
        * gcc.c (fd_exists): New.
        (driver::detect_jobserver): Use fd_exists.
        * lto-wrapper.c (fd_exists): New.
        (jobserver_active_p): Use fd_exists.
---
  gcc/gcc.c         | 19 +++++++++++++++++--
  gcc/lto-wrapper.c | 19 +++++++++++++++++--
  2 files changed, 34 insertions(+), 4 deletions(-)



Reply via email to