mismatch in configure.ac for libgomp

2015-04-29 Thread Pedro Giffuni

Hello;

I was checking out some old gcc revisions and I found this commit:

https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=140497

It has an obvious error that is still carried today:

Instead of adding adding the check for the  header
a redundant check for  was added.  The mismatch
is still in trunk (and all recent gcc versions).
The attached patch fixes it.

Regards,

Pedro.
--- libgomp/configure.ac.orig   2015-04-29 20:33:18.322158239 -0500
+++ libgomp/configure.ac2015-04-29 20:34:01.202743439 -0500
@@ -170,7 +170,7 @@
 AC_STDC_HEADERS
 AC_HEADER_TIME
 ACX_HEADER_STRING
-AC_CHECK_HEADERS(unistd.h semaphore.h sys/loadavg.h sys/time.h sys/time.h)
+AC_CHECK_HEADERS(unistd.h semaphore.h sys/loadavg.h sys/sysctl.h sys/time.h)
 
 GCC_HEADER_STDINT(gstdint.h)
 


Warning flags for compatibility with clang.

2014-05-21 Thread Pedro Giffuni

Hello;

FWIW, I have been looking at some changes from Apple's GCC and these two 
small changes should be interesting to have in the official gcc.


Implement -Wmost for compatibility with clang.
http://svnweb.freebsd.org/base?view=revision&revision=259920

Add Apple compatible -Wnewline-eof
http://svnweb.freebsd.org/base?view=revision&revision=259555

I don't own them but I doubt any of them is copyrightable. Perhaps they 
are available in the old Apple branches that the FSF has control of.


Regards,

Pedro.