David Malcolm writes:
> This patch moves the body of the RTL operand-parsing loop (about 180 lines of
> code) from read_rtx_code out into a new function (read_rtx_operand).
>
> Hopefully this is a reasonable cleanup in itself, but it's also useful
> for the RTL frontend I'm working on.
>
> Success
On 2016.09.23 at 15:29 +0200, Richard Biener wrote:
> >
> > So 5 looks too big to me.
>
> I think the issue is that the default number of partitions is too high
> (32) which pessimizes 4-core machines if the units are too small.
The more partitions are used the less memory is required at LTRA
On Fri, Sep 23, 2016 at 10:12:00PM +, Joseph Myers wrote:
> On Fri, 23 Sep 2016, Marek Polacek wrote:
>
> > On Fri, Sep 23, 2016 at 10:22:15AM -0400, Jason Merrill wrote:
> > > On Thu, Sep 22, 2016 at 9:59 AM, Marek Polacek wrote:
> > > >> This is very close, thanks. Let's give a more helpfu
Hi Andre,
It seems that the committed test gfortran.dg/coarray_lib_comm_1.f90 is not
right. I have to apply the patch
--- ../_clean/gcc/testsuite/gfortran.dg/coarray_lib_comm_1.f90 2016-09-23
12:45:56.0 +0200
+++ gcc/testsuite/gfortran.dg/coarray_lib_comm_1.f902016-09-24
12:15:02.
This patch broke bootstrap on AIX.
The ChangeLog entry was not committed to the gcc/ChangeLog file.
TARGET_LITTLE_ENDIAN only is defined for PPC64 Linux. This patch
should be using !BYTES_BIG_ENDIAN.
Thanks, David
Hi Dominique,
Yes of course. It makes sense to have the param a 4byte entity on 32 bit
systems. Please commit. I am on a mobile only currently and can't do so before
tomorrow afternoon.
Thanks for that correction.
Regards,
Andre
Am 24. September 2016 13:58:45 MESZ, schrieb "Dominique d'Humièr
Done as revision r240463.
> Le 24 sept. 2016 à 16:32, Andre Vehreschild a écrit :
>
> Hi Dominique,
>
> Yes of course. It makes sense to have the param a 4byte entity on 32 bit
> systems. Please commit. I am on a mobile only currently and can't do so
> before tomorrow afternoon.
>
> Thanks f
On Sat, 2016-09-24 at 09:32 -0400, David Edelsohn wrote:
> This patch broke bootstrap on AIX.
>
> The ChangeLog entry was not committed to the gcc/ChangeLog file.
Changelog entry added. Forgot to write that emacs buffer before commit.
> TARGET_LITTLE_ENDIAN only is defined for PPC64 Linux. This
/tree-ssa/builtin-sprintf-warn-1.c (test for warnings, line 1383)
FAIL: gcc.dg/tree-ssa/builtin-sprintf-warn-1.c (test for excess errors)
Excess errors:
/daten/aranym/gcc/gcc-20160924/gcc/testsuite/gcc.dg/tree-ssa/builtin-sprintf-warn-1.c:1408:3:
warning: specified size 4294967295 exceeds the size
On 21 September 2016 at 13:33, Jonathan Wakely wrote:
> This patch predates my testsuite changes, so this should now be
> { dg-do compile { target c++11 } }
> Otherwise OK for trunk, thanks.
So, this has been applied to trunk, and also applies cleanly to the
gcc-6 branch, and we need
to backport
FAIL: gfortran.dg/coarray_lib_comm_1.f90 -O0 scan-tree-dump-times original
"_gfortran_caf_get \\(caf_token.0, \\(integer\\(kind=8\\)\\) parm.[0-9]+.data -
\\(integer\\(kind=8\\)\\) a, 1, &parm.[0-9]+, 0B, &p, 4, 4, 1, 0B\\);" 1
FAIL: gfortran.dg/coarray_lib_comm_1.f90 -O0 scan-tree-dump-t
Really fixed by revision r240467. Sorry for the noise.
Dominique
> Le 24 sept. 2016 à 17:14, Dominique d'Humières a écrit :
>
> Done as revision r240463.
>
>> Le 24 sept. 2016 à 16:32, Andre Vehreschild a écrit :
>>
>> Hi Dominique,
>>
>> Yes of course. It makes sense to have the param a 4b
Hi,
In make_new_ssa_for_def (tree-reassoc.c) we should use gimple_get_lhs to
get lhs instead of gimple_assign_lhs as stmt can be builtins too.
Attached patch fixes this.
Testcase from PR (attached) seems to fail. I dont any fortran so I didnt
try fixing it. Any help here is appreciate. The l
Hi,
Two possibilities:
(a) pass -w in the dg-options
(b) replace INTEGER, PARAMETER :: dp=8 with INTEGER, PARAMETER :: dp=8,
spdim=42 (or any number you like!-).
Dominique
14 matches
Mail list logo