This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RFA] make missing header a fatal error with -MMD
ping?
(this might have gotten classified as spam for some ppl last time.)
On Fri, Sep 11, 2009 at 10:01, Chris Demetriou <cgd@google.com> wrote:
>
> This fixes a the long-open bug http://gcc.gnu.org/PR28435, in which
> -MMD allows compilation to succeed even though a header was missing.
> (.o file produced, and everything. ?yuck.)
>
> bootstrapped on x86_64-linux (Ubuntu Hardy) system @ rev 151627.
>
> Tested:
> Dropped the new tests into an existing source tree at the same rev.
> bootstrapped, tested with make -j3 check -k
> RUNTESTFLAGS="--target_board 'unix{-m32,-m64}'" from top-level.
> Applied patch to a new tree, bootstrapped, ran the tests.
>
> In the unmodified source tree, the new test
> gcc.dg/cpp/missing-sysheader-MMD.c fails. ?(The other new tests pass,
> as expected).
> In the modified source tree, all of the new tests pass. ?All of the
> other test results were identical.
>
>
>
> chris
> ----
> [libcpp/ChangeLog]
> 2009-09-10 ?Chris Demetriou ?<cgd@google.com>
>
> ? ? ? ?PR preprocessor/28435:
> ? ? ? ?* include/cpplib.h (struct cpp_options): Add new member
> ? ? ? ?deps.need_preprocessor_output.
> ? ? ? ?* files.c (open_file_failed): If preprocessor output is needed
> ? ? ? ?always report an error.
>
> [gcc/ChangeLog]
> 2009-09-10 ?Chris Demetriou ?<cgd@google.com>
>
> ? ? ? ?PR preprocessor/28435:
> ? ? ? ?* c-opts.c (c_common_handle_option): For -MD and -MMD, indicate
> ? ? ? ?to cpplib that the preprocessor output is needed.
>
> [gcc/testsuite/ChangeLog]
> 2009-09-10 ?Chris Demetriou ?<cgd@google.com>
>
> ? ? ? ?PR preprocessor/28435:
> ? ? ? ?* gcc.dg/cpp/missing-header-MD.c: New test.
> ? ? ? ?* gcc.dg/cpp/missing-header-MMD.c: New test.
> ? ? ? ?* gcc.dg/cpp/missing-sysheader-MD.c: New test.
> ? ? ? ?* gcc.dg/cpp/missing-sysheader-MMD.c: New test.