Hello!
> The attached patch replaces manual swaps (i.e. "tmp = a; a = b; b =
> tmp;") with std::swap. It also removes a couple of static functions
> which were used only for implementing such swaps.
>
> Bootstrapped/regtested on x86_64-linux; full target list build in
> progress. OK for trunk?
Ac
On June 15, 2015 9:58:33 PM GMT+02:00, Bill Schmidt
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 at 14:54 -0500, Bil
> 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!
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-r
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 c_finish_om
On Jun 15, 2015, at 12:55 PM, Andres Tiraboschi
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 file can run the entire tree under it.
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
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): Ne
Ping?
On 27 May 2015 at 22:15, Christophe Lyon 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 setting of the "Flush-t
> 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:
MULTIL
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,
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.
> >
> > S
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 e
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
2015-06-12 18:35 GMT-03:00 Joseph Myers :
> 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 options nee
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/stor
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):
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-fin
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
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 variab
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
> > > wrote:
> > > > On Mon, 2015-04-27 at 14:23 +0800, Bin.Cheng wrot
* 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 ce4dd9b.
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 a/gcc/config/mips/mt
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 pr
On Mon, Jun 15, 2015 at 2:05 PM, Jason Merrill 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++)
>> +
OK.
Jason
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 (TREE_VALU
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
==
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: gcc/go/gofr
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
Alan Modra 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 = -fsplit-stack
$ gre
On June 15, 2015 5:46:46 PM GMT+02:00, Jakub Jelinek 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 simd stu
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= settings
Tested on x86_64.
OK for trunk?
Thanks,
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
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.
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
* testsuit
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, 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 unsig
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
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 th
OK, thanks.
Jason
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, "");
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 of
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 (
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 for
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, co
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 (DECL_SOURCE_L
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 slig
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) {
me
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
(a
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
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 f
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 codecvt
On 06/15/2015 11:13 AM, Andrew Pinski wrote:
> On Mon, Jun 15, 2015 at 2:09 AM, Martin Liška wrote:
>> On 06/11/2015 08:19 PM, Richard Biener wrote:
>>> On June 11, 2015 7:50:36 PM GMT+02:00, Jakub Jelinek
>>> wrote:
On Fri, Jun 12, 2015 at 12:58:12AM +0800, pins...@gmail.com wrote:
> T
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 (BT_FN_VOID_INT_PTR_SIZE_PTR_PTR_
On 8 May 2015 at 12:42, Richard Biener wrote:
> On Tue, Nov 4, 2014 at 11:44 AM, Marcus Shawcroft
> wrote:
>> On 25 September 2014 04:45, Michael Collison
>> wrote:
>>> On certain patterns in atomics.md the constraint 'n' is used in combination
>>> with the predicate atomic_op_operand. The const
On 12 June 2015 at 21:43, Jim Wilson wrote:
> We have 5 patterns that can emit the movi instruction. These patterns
> map it to 4 different type attributes. The mov_aarch64 pattern
> uses mov_imm. The movdi_aarch64 pattern uses fmov. The movtf_aarch64
> pattern uses fconstd. And the two aarch
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 wrote:
> Hi Andre,
>
> > please find attached an updated version of the patch. This patch simplifies
> > some cases and
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.
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
On 29 May 2015 at 09:32, Shiva Chen 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
* aarch64.c (aarch64_simd_lane_bo
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 an
Alan Modra 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 "$target" in
>
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 (under
Mikhail Maltsev 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
On 4 June 2015 at 01:35, Jim Wilson 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
> str q0, [x0]
On Mon, Jun 15, 2015 at 2:09 AM, Martin Liška wrote:
> On 06/11/2015 08:19 PM, Richard Biener wrote:
>> On June 11, 2015 7:50:36 PM GMT+02:00, Jakub Jelinek
>> 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
On 06/11/2015 08:19 PM, Richard Biener wrote:
> On June 11, 2015 7:50:36 PM GMT+02:00, Jakub Jelinek 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 find
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/ -
71 matches
Mail list logo
