On Sat, Mar 28, 2020 at 08:10:38AM +0100, Tobias Burnus wrote: > Two remarks: > > * As the file trigd_lib.inc is shared between libgfortran > and gcc/fortran, I wonder whether it should be placed > under include/ (under the GCC toplevel directroy)
The original name and location were chosen to match intrinsics/erfc_scaled_inc.c, which is included in intrinsics/erfc_scaled.c. I think Fritz's re-use in simplification makes since given the ugly nested if-elseif-else constructs in the file. As this is Fortran specific, I do not believe it should be moved up to a toplevel diretory. > > * All included files need dependency; I do not quickly > see whether that' the case. They do? See intrinsics/erfc_scaled*.c. % grep erfc_ Makefile.am intrinsics/erfc_scaled.c \ -- steve