[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #12 from kirr at landau dot phys dot spbu dot ru 2010-08-07 10:49 --- Could someone please suggest me what I'm maybe doing wrong? Because I posted backported patches twice (first time on the mailing list) and there is always silence on gcc side... And as it is now, -4.4 and -4.5 are left in regressed state compared to -4.3, and the fix is only applied to -4.6 . Thanks, Kirill P.S. and as to the workaround described in comment5 - it does not fully qualify, becase for example calling cpp directly does not define predefined __GFORTRAN__, and I'm sure there are other corner-case when cpp'ing by hand differs from preprocessing from-under gfortran. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #11 from kirr at landau dot phys dot spbu dot ru 2010-06-28 18:06 --- Created an attachment (id=21032) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21032&action=view) gfortran-4.5-MD.patch -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #10 from kirr at landau dot phys dot spbu dot ru 2010-06-28 18:05 --- Created an attachment (id=21031) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21031&action=view) gfortran-4.4-MD.patch -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #9 from kirr at landau dot phys dot spbu dot ru 2010-06-28 18:04 --- REOPEN Daniel, thanks for working on this. Because dropping support for -MD was initially spot as 4.3 -> 4.4 regression, and you committed the fix to 4.6 only, I think it would be good to apply your patch to 4.4 and 4.5 branches as well. I've backported it on top of today's gcc-4_4-branch and gcc-4_5-branch. Tested on i686-pc-linux-gnu OK for 4.4 and "OK for check-fortran" for 4.5 (*). Two patches for 4.4 & 4.5 will be next attached. Please apply and thanks, Kirill (*) as of today, gcc-4_5-branch does not pass all tests on i686-pc-linux-gnu. Look e.g. here http://gcc.gnu.org/ml/gcc-testresults/2010-06/ -- kirr at landau dot phys dot spbu dot ru changed: What|Removed |Added Status|RESOLVED|UNCONFIRMED Resolution|FIXED | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #8 from dfranke at gcc dot gnu dot org 2010-06-13 16:09 --- Makefile dependency generation is now available in trunk (-cpp -MD). See PR44526 for a follow-up request to libcpp. Closing. -- dfranke at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #7 from dfranke at gcc dot gnu dot org 2010-06-13 16:05 --- Subject: Bug 43954 Author: dfranke Date: Sun Jun 13 16:05:01 2010 New Revision: 160684 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=160684 Log: 2010-06-13 Daniel Franke PR fortran/31588 PR fortran/43954 * gfortranspec.c (lang_specific_driver): Removed deprecation warning for -M. * lang.opt: Add options -M, -MM, -MD, -MMD, -MF, -MG, -MP, -MT, -MQ. * lang-specs.h (CPP_FORWARD_OPTIONS): Add -M* options. * cpp.h (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. * cpp.c (gfc_cpp_option): Add deps* members. (gfc_cpp_makedep): New. (gfc_cpp_add_dep): New. (gfc_cpp_add_target): New. (gfc_cpp_init_options): Initialize new options. (gfc_cpp_handle_option): Handle new options. (gfc_cpp_post_options): Map new options to libcpp-options. (gfc_cpp_init): Handle deferred -MQ and -MT options. (gfc_cpp_done): If requested, write dependencies to file. * module.c (gfc_dump_module): Add a module filename as target. * scanner.c (open_included_file): New parameter system; add the included file as dependency. (gfc_open_included_file): Add the included file as dependency. (gfc_open_intrinsic_module): Likewise. * invoke.texi: Removed deprecation warning for -M. * gfortran.texi: Removed Makefile-dependencies project. Modified: trunk/gcc/fortran/ChangeLog trunk/gcc/fortran/cpp.c trunk/gcc/fortran/cpp.h trunk/gcc/fortran/gfortran.texi trunk/gcc/fortran/gfortranspec.c trunk/gcc/fortran/invoke.texi trunk/gcc/fortran/lang-specs.h trunk/gcc/fortran/lang.opt trunk/gcc/fortran/module.c trunk/gcc/fortran/scanner.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #6 from kirr at landau dot phys dot spbu dot ru 2010-05-05 16:24 --- > So, if this is so dear to your heart, pull up your sleeves and start trying > to fix the problem. I see. And thanks for the workaround. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #5 from kargl at gcc dot gnu dot org 2010-05-04 16:46 --- (In reply to comment #4) > On Mon, May 03, 2010 at 07:45:20PM -, dfranke at gcc dot gnu dot org > wrote: > > (In reply to comment #2) > > On Monday 03 May 2010 21:23:26 you wrote: > > > And when I say "it used to work" I don't mean generating dependencies for > > > f90/f95 sources (PR31588 mentions e.g. USE), but only generating > > > dependencies from plain preprocessor. > > > > What you ask for is "just" a subset of PR31588. If that gets fixed, your > > request will be fixed as well. > > > > Feel free to re-open if you believe that it would be worth to fix the subset > > first. > > Yes, it's a subset of PR31588. > > But as you wrote PR31588 is going to be (maybe) fixed for 4.6, while I > believe regressions in general and this one in particular should be > better handled with higher priority and fixes should go to maintanance > branches. What you apparently fail to understand is that gfortran is written and maintained by a small group of volunteers. If none of these volunteers use this feature and if none of volunteers have the time to work on it, it won't get fixed anytime soon. So, if this is so dear to your heart, pull up your sleeves and start trying to fix the problem. In the meantime, just call cpp directly. troutmask:sgk[214] ~/work/4x/bin/cpp -Wp,-MD,test.d test.F | sed s/#/\!/ ! 1 "test.F" ! 1 "" ! 1 "" ! 1 "test.F" ! 1 "test.h" 1 ! 2 "test.F" 2 program test print *,'Hello World!' print *,'Hello up there qqq!' end troutmask:sgk[215] cat test.d test.o: test.F test.h -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #4 from kirr at landau dot phys dot spbu dot ru 2010-05-04 07:44 --- On Mon, May 03, 2010 at 07:45:20PM -, dfranke at gcc dot gnu dot org wrote: > (In reply to comment #2) > On Monday 03 May 2010 21:23:26 you wrote: > > And when I say "it used to work" I don't mean generating dependencies for > > f90/f95 sources (PR31588 mentions e.g. USE), but only generating > > dependencies from plain preprocessor. > > What you ask for is "just" a subset of PR31588. If that gets fixed, your > request will be fixed as well. > > Feel free to re-open if you believe that it would be worth to fix the subset > first. Yes, it's a subset of PR31588. But as you wrote PR31588 is going to be (maybe) fixed for 4.6, while I believe regressions in general and this one in particular should be better handled with higher priority and fixes should go to maintanance branches. Also, my impression is that PR31588 is more work than fixing "just" this subset. That's why I'm reopening it. Sorry if I ask for too much, especially when I can't help with code... > > If that would help I'm able and will be glad to try patches and do > > other testing as well. > > I'll keep that in mind :) Yes Thanks, Kirill -- kirr at landau dot phys dot spbu dot ru changed: What|Removed |Added Status|RESOLVED|UNCONFIRMED Resolution|DUPLICATE | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #3 from dfranke at gcc dot gnu dot org 2010-05-03 19:45 --- (In reply to comment #2) On Monday 03 May 2010 21:23:26 you wrote: > And when I say "it used to work" I don't mean generating dependencies for > f90/f95 sources (PR31588 mentions e.g. USE), but only generating > dependencies from plain preprocessor. What you ask for is "just" a subset of PR31588. If that gets fixed, your request will be fixed as well. Feel free to re-open if you believe that it would be worth to fix the subset first. > If that would help I'm able and will be glad to try patches and do > other testing as well. I'll keep that in mind :) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #2 from kirr at landau dot phys dot spbu dot ru 2010-05-03 19:23 --- Daniel, thanks for reply. As I see it, yes, PR31588 would be handy enhancement, but this bug is different -- it's a _regression_ -- it used to work in 4.3 and stopped in 4.4. And when I say "it used to work" I don't mean generating dependencies for f90/f95 sources (PR31588 mentions e.g. USE), but only generating dependencies from plain preprocessor. Please, let's not break what used to work. Is it ok reopen? P.S. I'd like to help as time permits, but hacking on GCC codebase is out of my skills at present. If that would help I'm able and will be glad to try patches and do other testing as well. Sorry and thanks, Kirill -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
[Bug fortran/43954] gfortran-4.4 does not support -Wp, -MD for *.F (4.3 -> 4.4 regression, needed for auto-dependencies)
--- Comment #1 from dfranke at gcc dot gnu dot org 2010-05-03 18:09 --- If real life allows, I'll give it another try for 4.6 - started twice already but there are quite a few options to handle. However, if you feel like it, dig in. Help is always welcome :) Closing as dupe of PR31588. *** This bug has been marked as a duplicate of 31588 *** -- dfranke at gcc dot gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution||DUPLICATE http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43954
