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: Alan Modra <amodra at gmail dot com>
- To: "Maciej W. Rozycki" <macro at codesourcery dot com>
- Cc: Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 10 Sep 2014 22:39:22 +0930
- 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>
On Wed, Sep 10, 2014 at 01:43:22PM +0100, Maciej W. Rozycki wrote:
> Thanks for your work on this issue, I have tested your change with my
> usual powerpc-gnu-linux multilibs with the old and new result for
> gcc.c-torture/compile/pr60655-2.c noted on the right:
>
> -mcpu=603e FAIL -> PASS
> -mcpu=603e -msoft-float FAIL -> PASS
> -mcpu=8540 -mfloat-gprs=single -mspe=yes -mabi=spe FAIL -> PASS
> -mcpu=8548 -mfloat-gprs=double -mspe=yes -mabi=spe FAIL -> PASS
> -mcpu=7400 -maltivec -mabi=altivec FAIL -> PASS
> -mcpu=e6500 -maltivec -mabi=altivec FAIL -> PASS
> -mcpu=e5500 -m64 PASS -> PASS
> -mcpu=e6500 -m64 -maltivec -mabi=altivec PASS -> PASS
>
> -- please note that the test case used to pass for 64-bit multilibs even
> before your fix so unless your powerpc64-linux configuration includes
> 32-bit multilibs as well, it does not really provide suitable coverage.
I always build powerpc64-linux with --enable-targets=powerpc-linux and
regression test with RUNTESTFLAGS=--target_board=unix/'{-m32,-m64}',
so my "bootstrapped and regression tested powerpc64-linux" claim
includes a -m32 regression test too. Not quite as comprehensive a
test as you've done (thanks!), but I did see the FAIL->PASS for -m32.
--
Alan Modra
Australia Development Lab, IBM