[libgfortran, patch] Remove unused headers

FX fxcoudert@gmail.com
Wed Dec 21 11:18:00 GMT 2016


> Ok. Watch out for regressions on other targets though. I have more
> than once caused regressions by using some function that happened to
> be included in the existing headers on glibc, but needed some other
> header on other targets.

Committed: https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=243844
I’ve tried to choose based on standards and not glibc, but yes, I will watch out for regressions.


> Yes, sounds reasonable to include stdlib.h, yes.

I have committed a patch handling stdlib.h: https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=243846

I have also committed a patch removing some unused limits.h in minloc/maxloc, limits.h and errno.h from runtime/minimal.c, and some includes of headers already pulled in from libgfortran.h (math.h from norm and parity, sys/types.h from random.c). Commit message: https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=243847

I have not touched caf/* files, as duplication of includes there seems intentional.

FX


More information about the Gcc-patches mailing list