On Sat, Jun 13, 2015 at 12:46:18PM +0200, Andreas Schwab wrote:
/usr/bin/mkdir -p .; files=`echo ../../../../libgo/go/errors/errors.go | sed
-e 's/[^ ]*\.gox//g'`; /bin/sh ./libtool --tag GO --mode=compile
/daten/gcc/gcc-20150613/Build/./gcc/gccgo
-B/daten/gcc/gcc-20150613/Build/./gcc/
On Mon, Jun 15, 2015 at 2:09 AM, Martin Liška mli...@suse.cz wrote:
On 06/11/2015 08:19 PM, Richard Biener wrote:
On June 11, 2015 7:50:36 PM GMT+02:00, Jakub Jelinek ja...@redhat.com
wrote:
On Fri, Jun 12, 2015 at 12:58:12AM +0800, pins...@gmail.com wrote:
This is just a bug in the older
On 14/06/15 23:45 +0800, Fan You wrote:
This is the revised patch.
Bootstrapped and Tested on Darwin 10.9.4. with testsuite 20_util/*
Great, it's *very* important that you can run the tests, so we know
your changes haven't broken the existing code.
You will also need to write new tests
Hi.
The attached patch adds new match-and-simplify patterns, which fold
~((~a) b) into (a b) for arithmetic shifts (i.e. when A is signed)
and perform similar folds for rotations. It also fixes PR
tree-optimization/54579 (because we already fold (-a - 1) into ~a).
A couple of questions:
1.
Hi Thomas, hi all,
I got no objections so far, therefore commited as r224477. Thanks for the
review.
Regards,
Andre
On Thu, 11 Jun 2015 23:59:48 +0200
Thomas Koenig tkoe...@netcologne.de wrote:
Hi Andre,
please find attached an updated version of the patch. This patch simplifies
On 12 June 2015 at 21:43, Jim Wilson jim.wil...@linaro.org wrote:
We have 5 patterns that can emit the movi instruction. These patterns
map it to 4 different type attributes. The movmode_aarch64 pattern
uses mov_imm. The movdi_aarch64 pattern uses fmov. The movtf_aarch64
pattern uses
On 4 June 2015 at 01:35, Jim Wilson jim.wil...@linaro.org wrote:
I noticed that poor code is emitted for a long double 0.0. This testcase
long double sub (void) { return 0.0; }
void sub2 (long double *ld) { *ld = 0.0; }
currently generates
sub:
ldr q0, .LC0
ret
...
sub2:
ldr q0, .LC1
On Wed, Jun 10, 2015 at 08:57:37AM +0100, Christian Bruel wrote:
Hi,
Commited [4/6] as attached (r224314)
thanks
Christian
On 06/08/2015 11:26 AM, Ramana Radhakrishnan wrote:
On 08/06/15 09:45, Christian Bruel wrote:
do you have other feedbacks for the remaining parts ?
many
Mikhail Maltsev malts...@gmail.com writes:
On 10.06.2015 10:05, Richard Sandiford wrote:
+/* Structure which holds data, required for generating stub gen_*
function. */
No comma after data
+/* These instructions require default stub function. Stubs are never
called.
require a
Alan Modra amo...@gmail.com writes:
diff --git a/libgo/configure.ac b/libgo/configure.ac
index 7c403a5..2ddcdfd 100644
--- a/libgo/configure.ac
+++ b/libgo/configure.ac
@@ -366,6 +366,13 @@ esac
AC_SUBST(OSCFLAGS)
dnl Use -fsplit-stack when compiling C code if available.
+case
On 06/11/2015 08:19 PM, Richard Biener wrote:
On June 11, 2015 7:50:36 PM GMT+02:00, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Jun 12, 2015 at 12:58:12AM +0800, pins...@gmail.com wrote:
This is just a bug in the older compiler. There was a change to fix
in
placement new operator. I can't
Hello!
The attached patch revises the tests for the filds and fists
mnemonics to use the assembly...
filds mem(%rip); fists mem(%rip)
and the test for the fildq and fistq mnemonics to use the assembly...
fildq mem(%rip); fistpq mem(%rip)
which will assemble for both 64-bit and 32-bit
On 29 May 2015 at 09:32, Shiva Chen shiva0...@gmail.com wrote:
Hi, Andrew
I modify the patch as you suggestion and testing on 32/64 bit host.
Thanks your tips.
I really appreciate for your help.
Shiva
OK and committed with this ChangeLog:
2015-06-14 Shiva Chen shiva0...@gmail.com
*
On 06/15/2015 11:13 AM, Andrew Pinski wrote:
On Mon, Jun 15, 2015 at 2:09 AM, Martin Liška mli...@suse.cz wrote:
On 06/11/2015 08:19 PM, Richard Biener wrote:
On June 11, 2015 7:50:36 PM GMT+02:00, Jakub Jelinek ja...@redhat.com
wrote:
On Fri, Jun 12, 2015 at 12:58:12AM +0800,
On Sat, Jun 06, 2015 at 12:10:00AM +0200, Tom de Vries wrote:
this patch removes superfluous -fopenmp settings.
In the case of target-8.c, we remove the whole dg-options line which did not
have an -On setting, which means the optimization level at which the
testcase compiles is changed from
On 8 May 2015 at 12:42, Richard Biener richard.guent...@gmail.com wrote:
On Tue, Nov 4, 2014 at 11:44 AM, Marcus Shawcroft
marcus.shawcr...@gmail.com wrote:
On 25 September 2014 04:45, Michael Collison
michael.colli...@linaro.org wrote:
On certain patterns in atomics.md the constraint 'n' is
On Mon, Jun 15, 2015 at 03:20:37PM +0300, Ilya Verbin wrote:
This patch introduces new versions of GOMP_target{,_data,_update} for OpenMP
4.1
with unsigned short for map kinds, but without new async arguments yet.
I think I'd prefer (for now) to suffix the functions with _41 instead of 1
(and
Hi,
This patch introduces new versions of GOMP_target{,_data,_update} for OpenMP 4.1
with unsigned short for map kinds, but without new async arguments yet.
make check-target-libgomp and bootstrap passed, ok for gomp-4_1-branch?
gcc/
* builtin-types.def
On 08/06/15 16:12 +0100, Jonathan Wakely wrote:
The linker script assumes that std::mbstate_t has the name __mbstate_t
for linkage purposes, but that's not necessarily true. For mingw32
it's just a typedef for int, so the patterns don't match.
This adds a new mingw32-specific pattern for
On Sat, Jun 06, 2015 at 12:05:44AM +0200, Tom de Vries wrote:
this patch removes a superfluous -std=c99 or -std=gnu99 setting in the
libgomp/testsuite/libgomp.c testcases (and a superfluous -fopenmp setting in
some cases as well).
The setting is superfluous because -std=gnu11 is the new
Hi!
As Tom has reported, the for-2.c testcase ICEs at -O2 -fopenmp,
because it has a noreturn function in the body and so while in omplower
we decide to use omp simd array arrays, in ompexp there is no loop
to attach the simd stuff to and I forgot to set the has_simduid_loops flag
in that case
Simon,
As discussed privately, your patch is interesting but isn't complete enough
so cannot be integrated as is since we also want to avoid not only the
generation of the initialization/finalization exception handlers, but also
to eliminate the creation of the various variables that keep track
We (Imagination) would like to change the layout of the mips-mti-linux-gnu
and mips-img-linux-gnu cross compiler toolchains. This patch, which affects
nothing other than those targets, implements that change.
Prior to this patch the mti and img cross compilers used a set of nested
directories
OK, thanks.
Jason
On 15/06/15 15:30, Kyrill Tkachov wrote:
On 29/05/15 11:15, Kyrill Tkachov wrote:
On 29/05/15 10:08, Kyrill Tkachov wrote:
Hi Mike,
On 28/05/15 22:15, Mike Stump wrote:
So, the arm memcpy code of aligned data isn’t as good as it can be.
void *memcpy(void *dest, const void *src, unsigned
On 29/05/15 11:15, Kyrill Tkachov wrote:
On 29/05/15 10:08, Kyrill Tkachov wrote:
Hi Mike,
On 28/05/15 22:15, Mike Stump wrote:
So, the arm memcpy code of aligned data isn’t as good as it can be.
void *memcpy(void *dest, const void *src, unsigned int n);
void foo(char *dst, int i) {
Hi, thanks for your answer. I was trying with warning_at this way
instead that I was doing before:
+ else if (TREE_CODE(init) == CALL_EXPR)
+{
+ tree fn = TREE_OPERAND(CALL_EXPR_FN(init), 0);
+ if (DECL_DECLARED_CONSTEXPR_P(fn) warn_constexpr)
+ warning_at
On 05/20/2015 11:28 AM, Jason Merrill wrote:
On 05/02/2015 04:16 PM, Ed Smith-Rowland wrote:
This extends' static assert to not require a message string.
I elected to make this work also for C++11 and C++14 and warn only with
-pedantic.
I think many people just write
static_assert(thing, );
Hi Matthew,
/* LRA will allocate an FPR for an integer mode pseudo instead of spilling
to memory if an FPR is present in the allocno class. It is rare that
we actually need to place an integer mode value in an FPR so where
possible limit the allocation to GR_REGS. This will
On June 15, 2015 9:58:33 PM GMT+02:00, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
I just was reading the gcc mailing list and realized that changes to
4.8
now require release manager approval. Adding Richard to the CC list
for
consideration. Thanks!
OK.
Richard.
Bill
On Mon, 2015-06-15
On Mon, Jun 15, 2015 at 01:37:48PM -0500, Segher Boessenkool wrote:
sub-rtx's in rtx_cost are
summed in order rather than reverse order so that the mode from an
earlier operand can be used for a later operand lacking a mode. This
is for ZERO_EXTEND and similar codes where the sub-rtx mode
On Mon, Jun 15, 2015 at 06:31:01PM +0200, Tom de Vries wrote:
Hi,
this patch removes superfluous dg-option -O2 settings in
testsuite/libgomp.c. The setting is superfluous, because DEFAULT_CFLAGS is
already set to -O2 in c.exp.
Tested on x86_64.
OK for trunk?
Ok, thanks.
On June 15, 2015 5:46:46 PM GMT+02:00, Jakub Jelinek ja...@redhat.com wrote:
Hi!
As Tom has reported, the for-2.c testcase ICEs at -O2 -fopenmp,
because it has a noreturn function in the body and so while in omplower
we decide to use omp simd array arrays, in ompexp there is no loop
to attach the
Alan Modra amo...@gmail.com writes:
This untested patch ought to fix the problem, I think.
There is no -fsplit-stack in the Makefile, and the configure script
has already determined the correct settings.
$ grep -e -fsplit-stack libgo/Makefile 32/libgo/Makefile
libgo/Makefile:SPLIT_STACK =
The master gofrontend repository has changed to git, so I am using a
different system for keeping the files up to date in GCC. I've added
a MERGE file to the GCC repository to track the most recent change
that has been merged over.
Ian
Index: gcc/go/gofrontend/MERGE
On 06/11/2015 09:25 PM, Patrick Palka wrote:
+ parameter_vec = make_tree_vec
+ (TREE_VEC_LENGTH (TREE_VALUE (current_template_parms)) + 1);
+
+ for (int i = 0; i TREE_VEC_LENGTH (parameter_vec) - 1; i++)
+ TREE_VEC_ELT (parameter_vec, i)
+ = TREE_VEC_ELT
OK.
Jason
Hi Alan,
On Mon, Jun 15, 2015 at 12:03:47PM +0930, Alan Modra wrote:
This patch changes the targetm.rtx_costs interface to pass a mode
parameter, and removes a redundant parameter. The reason for the
change is that powerpc and other backends need the mode that a
const_int is used in to
* go-lang.c (go_langhook_init_options_struct): Don't set
x_flag_split_stack.
(go_langhook_post_options): Set it here instead.
---
gcc/go/go-lang.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/gcc/go/go-lang.c b/gcc/go/go-lang.c
index
On Mon, Jun 15, 2015 at 2:05 PM, Jason Merrill ja...@redhat.com wrote:
On 06/11/2015 09:25 PM, Patrick Palka wrote:
+ parameter_vec = make_tree_vec
+ (TREE_VEC_LENGTH (TREE_VALUE (current_template_parms)) + 1);
+
+ for (int i = 0; i TREE_VEC_LENGTH (parameter_vec) - 1; i++)
Hi Steve,
Having worked on the new layout I of course am happy with it. I think it
makes the cross compiled sysroots much easier to use for installing
on a target as well as making the library paths match for cross compiled
and native. A couple of minor things...
diff --git
If the RTS in use is configurable (I believe this is the same in this context
as restricted) and includes finalization, gnatbind generates binder code that
won't compile.
This situation arises, for example, with an embedded RTS that incorporates the
Ada 2012 generalized container iterators.
On Mon, 15 Jun 2015, Martin Liška wrote:
Ah, I overlooked that it's not a placement new, but just static casting.
Anyway, if I added:
cselib_val () {}
to struct cselib_val and changed the cast to placement new:
char *ptr = (char *) header;
return new (ptr) T ();
I got following compilation
Hi,
we are getting bug reports (3 so far) about this issue: in C++11 we
reject the below testcase and we say that the virtual function declared
in A is never defined. Without considering more subtle details, the
error appears meaningless because the function is in fact *pure*
virtual. In any
Hi,
this patch removes superfluous dg-option -O2 settings in
testsuite/libgomp.c. The setting is superfluous, because DEFAULT_CFLAGS
is already set to -O2 in c.exp.
Tested on x86_64.
OK for trunk?
Thanks,
- Tom
Remove dg-options -O2 in libgomp.c
2015-06-15 Tom de Vries
Hi,
this patch:
- sets DEFAULT_CFLAGS to -O2, if not set otherwise
(similar to what is done in c.exp)
- removes superfluous dg-options -O2 settings.
- removes superfluous dg-options -fopenmp settings.
- uses dg-additional-options for -std=standard settings
Tested on x86_64.
OK for trunk?
On Mon, Jun 15, 2015 at 15:06:09 +0200, Jakub Jelinek wrote:
On Mon, Jun 15, 2015 at 03:20:37PM +0300, Ilya Verbin wrote:
This patch introduces new versions of GOMP_target{,_data,_update} for
OpenMP 4.1
with unsigned short for map kinds, but without new async arguments yet.
I think I'd
On Mon, Jun 15, 2015 at 07:18:27PM +0300, Ilya Verbin wrote:
On Mon, Jun 15, 2015 at 15:06:09 +0200, Jakub Jelinek wrote:
On Mon, Jun 15, 2015 at 03:20:37PM +0300, Ilya Verbin wrote:
This patch introduces new versions of GOMP_target{,_data,_update} for
OpenMP 4.1
with unsigned short
The problem here is that the cached DIE does not have a parent because
we purposely removed it, hoping that decls_for_scope will fill it in:
/* If we're a nested function, initially use a parent of NULL; if
we're
a plain function, this will be fixed up in decls_for_scope. If
The bug is of course that like DEFAULT_ABI, rs6000_isa_flags hasn't been
determined yet.
Andreas.
--
Andreas Schwab, sch...@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5
And now for something completely different.
This patch from Chris Manghane fixes the Go frontend so that when
using the -fgo-dump-ast option it does not crash when dumping an empty
block. This fixes http://golang.org/issue/10420 . Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
Index:
Hi,
When I backported support for unaligned vector load/store operations on
POWER8 to GCC 4.8, I fumbled the change for gcc.dg/vect/vect-33.c. One
of the original tests was:
/* { dg-final { scan-tree-dump-times Vectorizing an unaligned access 0 vect
} } */
which I modified to
/* { dg-final
On Thu, 11 Jun 2015, Joseph Myers wrote:
loongson and r8000 have the most changes, they no longer generate msub
instructions with -mfused-madd because that instruction does not generate
the correct NAN in some cases (the sign may be wrong). If HONOR_NANS
is not set then they will
On Fri, 2015-06-12 at 17:36 +0100, Vidya Praveen wrote:
On Thu, Apr 30, 2015 at 01:34:18PM +0100, Bill Schmidt wrote:
On Thu, 2015-04-30 at 18:26 +0800, Bin.Cheng wrote:
On Mon, Apr 27, 2015 at 9:26 PM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
On Mon, 2015-04-27 at 14:23 +0800,
On Mon, Jun 15, 2015 at 18:25:28 +0200, Jakub Jelinek wrote:
On Mon, Jun 15, 2015 at 07:18:27PM +0300, Ilya Verbin wrote:
On Mon, Jun 15, 2015 at 15:06:09 +0200, Jakub Jelinek wrote:
On Mon, Jun 15, 2015 at 03:20:37PM +0300, Ilya Verbin wrote:
This patch introduces new versions of
This patch adds preliminary support for worker state propagation inside
acc loops. Besides for the same lack of precise data flow information
shared with vector broadcasting, this patch does not attempt to reserve
a sufficient amount of .shared memory to spill-and-fill all of the
broadcasted
I just was reading the gcc mailing list and realized that changes to 4.8
now require release manager approval. Adding Richard to the CC list for
consideration. Thanks!
Bill
On Mon, 2015-06-15 at 14:54 -0500, Bill Schmidt wrote:
Hi,
When I backported support for unaligned vector load/store
On Mon, Jun 15, 2015 at 10:48:50PM +0300, Ilya Verbin wrote:
Here is the new patch. OK to commit?
gcc/
* builtin-types.def (BT_FN_VOID_INT_OMPFN_SIZE_PTR_PTR_PTR): New.
(BT_FN_VOID_INT_OMPFN_PTR_SIZE_PTR_PTR_PTR): Remove.
* omp-builtins.def (BUILT_IN_GOMP_TARGET):
2015-06-12 18:35 GMT-03:00 Joseph Myers jos...@codesourcery.com:
On Fri, 12 Jun 2015, Andres Tiraboschi wrote:
Hi, this patch is for adding a warning when a constexpr cannot be evaluated
at compile time.
This is a single case:
type var = fun(args...), with fun declared as a constexpr.
All
This is the first in a series of patches to make a build with an in-tree
GNU libiconv work as designed.
This patch fixes dependencies for parallel make, and avoids failures
with make targets not supported by GNU libiconv.
This is OK. Thanks!
Apart from wanting to understand the need/lack of ! in various places, this is
otherwise OK.
Matthew
Ok, I split the long lines and added the missing '!' characters to all
the MULTILIB_OSDIRNAMES except for the default ones. Using the '!' on
the default MULTILIB_OSDIRNAMES like:
On Mon, 15 Jun 2015, Maciej W. Rozycki wrote:
operands negated. That negation, implemented with the IEEE Std 754-2008
`negate' operation that you referred to, by definition is required to
operate on the sign of its operand in a specific way even if the operand
is a qNaN.
So for
POWER8 added a multiply instruction that makes mulv4si more efficient.
And vmladduhm can be used for mulv8hi3. This patch also changes
vmladduhm from a black box UNSPEC to descriptive RTL.
Bootstrapped on powerpc64le-linux.
* altivec.md: Delete UNSPEC_VMLADDUHM.
(mulv4si3_p8):
On Mon, 15 Jun 2015, Joseph Myers wrote:
operands negated. That negation, implemented with the IEEE Std 754-2008
`negate' operation that you referred to, by definition is required to
operate on the sign of its operand in a specific way even if the operand
is a qNaN.
So for
On Mon, 15 Jun 2015, Maciej W. Rozycki wrote:
It is only required (well, recommended) to pass the *payload*. The sign
bit is not part of the payload. For all other operations, this standard
does not specify the sign bit of a NaN result, even when there is only one
input NaN, or when
Ping?
On 27 May 2015 at 22:15, Christophe Lyon christophe.l...@linaro.org wrote:
This patch series is a follow-up to the tests I already contributed,
converted from my original testsuite.
This series consists in 20 new patches, which can be committed
independently. For vrecpe, I added the
On 06/15/2015 12:05 PM, Jason Merrill wrote:
On 05/20/2015 11:28 AM, Jason Merrill wrote:
On 05/02/2015 04:16 PM, Ed Smith-Rowland wrote:
This extends' static assert to not require a message string.
I elected to make this work also for C++11 and C++14 and warn only with
-pedantic.
I think many
This patch from Chris Manghane analyzes binary expressions in escape
analysis. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
Index: gcc/go/gofrontend/MERGE
===
--- gcc/go/gofrontend/MERGE
On Jun 15, 2015, at 12:55 PM, Andres Tiraboschi
andres.tirabos...@tallertechnologies.com wrote:
Hi, do you know where is the .exp file for the tests in
.../gcc/testsuite/g++.dg/warn?
I can't find it.
find srcdir -name \*.exp -print will show you all of them. You’ll discover
that a .exp
This patch teaches the c and c++ front ends to use a common function to
split clauses in combined acc parallel loops and acc kernel loops.
There's still a little bit of duplicate code inside c_parser_oacc_loop
and c_parser_oacc_loop and cp_parser_oacc_loop with their respective
calls to
70 matches
Mail list logo
