This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 0/9] Remove macros that implicitly use input_location


On Fri, 2013-11-22 at 12:57 +0100, Richard Biener wrote:
> On Fri, Nov 22, 2013 at 4:31 AM, David Malcolm <dmalcolm@redhat.com> wrote:
> > The following patch series eliminates the following macros that
> > implicitly use input_location:
> >   input.h:
> >     #define input_line LOCATION_LINE (input_location)
> >     #define input_filename LOCATION_FILE (input_location)
> >     #define in_system_header (in_system_header_at (input_location))
> >     #define EXPR_LOC_OR_HERE(NODE) (EXPR_HAS_LOCATION (NODE) \
> >                                ? (NODE)->exp.locus : input_location)
> >
> > replacing the first three by their expansions inline, and the last by
> >   EXPR_LOC_OR_LOC(NODE, input_location)
> >
> > This is a revised version of the single patch I sent as:
> >   http://gcc.gnu.org/ml/gcc-patches/2013-07/msg00072.html
> > splitting it up into multiple patches by subdirectory, and trying to be
> > less clever about picking alternate locations than "input_location"
> > (removal of uses of "input_location" can wait for another patch, this
> > series merely makes them explicit).
> >
> > Successfully bootstrapped&regtested on x86_64-unknown-linux-gnu; my
> > bootstrap covered c, c++, ada, fortran, but managed to miss java (though
> > Tom already approved that part in a previous version).
> >
> > OK for trunk, assuming I can get it to build with Java, and targetting
> > bfin?
> 
> Ok.

Thanks; committed to trunk as r205262, having verified the java and bfin
builds.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]