Patch: Remove non-mapped location support

Richard Guenther richard.guenther@gmail.com
Mon Feb 25 18:51:00 GMT 2008


On Mon, Feb 25, 2008 at 5:40 PM, Tom Tromey <tromey@redhat.com> wrote:
> In 4.3 we changed the default location representation to "mapped
>  locations".  The old representation was deprecated at that point.
>
>  This patch removes the old, deprecated location code.  By and large
>  this is a mechanical patch.  There are a few details worth mentioning:
>
>  * I poisoned 'USE_MAPPED_LOCATION' to prevent regressions due to
>   merges.  I think there's no point in poisoning the other deleted
>   input.h things; a merge would run into other errors if those are
>   used.
>
>  * java/java-tree.h still mentions USE_MAPPED_LOCATION in a comment.
>   This is a reminder to me to look into this -- the comment is for a
>   field which is apparently never set.  I will fix this separately.
>
>  * input.h used to claim that location_t was a deprecated typedef.  I
>   think that removing this is not very important and I "un-deprecated"
>   it.
>
>  * The Makefile.in change is needed to make gengtype work properly now
>   that I've deleted the input.h-related special case.
>
>  * There are a few places that had odd comments related to mapped
>   locations, e.g. look at cp/decl2.c.  I left the "???" comments
>   (slightly modified) for language maintainers to look at.  They don't
>   seem to cause any problems with the test suite -- and anyway 4.3 is
>   already running this branch of the conditionals...
>
>  * tree-vectorizer.h defines shorthand for various input.h-defined
>   things.  In general I think this is not the best thing to do; it is
>   better, in my view, for all the code in GCC to use a single API,
>   rather than having modules defining idiosyncractic variants.  I did
>   not change this.
>
>  Bootstrapped and regtested C, C++, ObjC, ObjC++, Ada, Fortran, Java,
>  and Treelang on x86 FC-6.  Ok?

The middle-end parts are ok.

Thanks,
Richard.



More information about the Fortran mailing list