This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PR debug/60655, debug loc expressions
- From: "Maciej W. Rozycki" <macro at codesourcery dot com>
- To: Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>
- Cc: Alan Modra <amodra at gmail dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 10 Sep 2014 18:27:26 +0100
- Subject: Re: PR debug/60655, debug loc expressions
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot DEB dot 1 dot 10 dot 1409020249380 dot 2958 at tp dot orcam dot me dot uk> <CAGWvnyk61JeAUVBogq5uf5gDQzqkj+-VTuNag1FMb=h6ttNztg at mail dot gmail dot com> <20140903053117 dot GF17693 at bubble dot grove dot modra dot org> <20140904122150 dot GL17693 at bubble dot grove dot modra dot org> <alpine dot DEB dot 1 dot 10 dot 1409042020320 dot 27075 at tp dot orcam dot me dot uk> <20140905013004 dot GN17693 at bubble dot grove dot modra dot org> <20140909115027 dot GR17693 at bubble dot grove dot modra dot org> <alpine dot DEB dot 1 dot 10 dot 1409092255590 dot 27075 at tp dot orcam dot me dot uk> <54106540 dot 4080903 at arm dot com>
On Wed, 10 Sep 2014, Ramana Radhakrishnan wrote:
> > 2014-04-10 Ramana Radhakrishnan <ramana.radhakrishnan@arm.com>
> >
> > PR debug/60655
> > * config/arm/arm.c (TARGET_CONST_NOT_OK_FOR_DEBUG_P): Define
> > (arm_const_not_ok_for_debug_p): Reject MINUS with SYM_REF's
> > ameliorating the cases where it can be.
> >
> > can I suppose be reverted too.
>
> That was always something to help get 4.9 out of the door - I still mean to do
> the proper work for 5.0 where we look at all possible types of output
> generated but it won't happen this month either.
>
> If this fixes it properly it would be nice to revert the change - I can't take
> this on for a few weeks as I am travelling. If you want to take that up please
> verify that the original testcase is fixed and the usual caveats of testing
> apply.
This is supposed to properly fix the issue the test case covers I
believe. I'm not going to do anything for the ARM target here, I just
thought you might want to know that PR debug/60655 has now been further
addressed, as it may be easy to miss mailing list traffic (especially as
ARM has been only sparsely mentioned in the discussion). And what you do
with this knowledge is up to you. :)
Maciej