diff mbox series

[FORTRAN] Remove OS dependency in definition of ASYNC_IO

Message ID CAGWvny=SxyZKUabA76UDKuNcD9RvMBSQM_baQA2swswV_RU__A@mail.gmail.com
State New
Headers show
Series [FORTRAN] Remove OS dependency in definition of ASYNC_IO | expand

Commit Message

David Edelsohn Aug. 23, 2018, 7:03 p.m. UTC 
The recent fix for Async I/O on systems without Gthread cond support added
an explicit test for _AIX.  This test is unnecessary because
__GTHREAD_HAS_COND is not defined on AIX.  Protecting the use of
__gthread_cond_t inside in the ASYNC_IO macro for the declaration of struct
adv_cond is sufficient.  This patch simplifies the macro definition.

Bootstrapped on powerpc-ibm-aix7.2.0.0

Okay?

Thanks, David

* async.h (ASYNC_IO): Revert _AIX test.

 #define ASYNC_IO 0

Comments

Thomas Koenig Aug. 23, 2018, 7:27 p.m. UTC | #1 
David,

> The recent fix for Async I/O on systems without Gthread cond support added
> an explicit test for _AIX.  This test is unnecessary because
> __GTHREAD_HAS_COND is not defined on AIX.  Protecting the use of
> __gthread_cond_t inside in the ASYNC_IO macro for the declaration of struct
> adv_cond is sufficient.  This patch simplifies the macro definition.
> 
> Bootstrapped on powerpc-ibm-aix7.2.0.0
> 
> Okay?

OK.

Thanks for the patch!

	Thomas
diff mbox series

Patch

Index: async.h
===================================================================
--- async.h     (revision 263818)
+++ async.h     (working copy)
@@ -29,7 +29,7 @@ 
    __gthread_cond_t and __gthread_equal / __gthread_self.  Check
    this.  */

-#if defined(__GTHREAD_HAS_COND) && defined(__GTHREADS_CXX0X) &&
!defined(_AIX)
+#if defined(__GTHREAD_HAS_COND) && defined(__GTHREADS_CXX0X)
 #define ASYNC_IO 1
 #else