Any news about this one ?
Here is another version with additional random tests on algos just to
challenge other combinations of tests.
PR libstdc++/61107
* include/bits/stl_algo.h (__inplace_stable_partition): Delete.
(__stable_partition_adaptive): Return __first is range length is
Hi!
This patch implements what I understood from Joseph's
https://gcc.gnu.org/ml/gcc/2013-10/msg00280.html
and also adds clang compatible builtins (which implement
small subset of the typegeneric ones).
Besides the clang compatibility builtins, there are 3 new
type-generic builtins, __builtin_{ad
On November 10, 2014 9:45:27 PM CET, Jakub Jelinek wrote:
>Hi!
>
>As the following two testcases shows, there are lots of issues in
>ICF compare_gimple_call, in particular, it doesn't handle indirect
>calls
>properly (see the ipa-icf-31.c testcase), doesn't handle internal calls
>properly (see ubs
Hi all,
here a second chunk which uses arm_eabi instead of arm*-*-*eabi* and
arm*-*-symbianelf*.
As I was told, arm*-*-symbianelf* should be EABI so we can use arm_eabi
for all instead of listing each OS.
Ok for trunk?
TIA,
Andreas
2014-11-10 Andreas Tobler
* gcc.target/arm/ea
On November 10, 2014 9:13:29 PM CET, Uros Bizjak wrote:
>Hello!
>
>std::swap was recently mentioned in gcc-patches@ mailing list, so I
>gave it a try. As can be seen below, a lot of code in config/i386
>benefits from this conversion.
>
>Surprisingly, I didn't have to include any header on F20 linu
On Mon, Nov 10, 2014 at 10:13 PM, Richard Biener
wrote:
> On November 10, 2014 9:13:29 PM CET, Uros Bizjak wrote:
>>Hello!
>>
>>std::swap was recently mentioned in gcc-patches@ mailing list, so I
>>gave it a try. As can be seen below, a lot of code in config/i386
>>benefits from this conversion.
Uros reported that my latest patch to fix PR63620 does not fix actually
the problem and H.J. reported that the patch resulted in 2 java test
failures (PR63799).
The following patch fixes PR63620 and PR63799.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63620
https://gcc.gnu.org/bugzilla/show
Ping. I'm getting more reports of this bug internally, and it would be
nice to have the fix upstream.
-cary
On Mon, Oct 13, 2014 at 11:43 AM, Cary Coutant wrote:
> Ping. Jason, do you still think the special-case for conversion ops is
> inappropriate?
>
> -cary
>
>
> On Fri, Jul 25, 2014 at 2:1
On 10/11/14 21:50 +0100, François Dumont wrote:
Any news about this one ?
Here is another version with additional random tests on algos just to
challenge other combinations of tests.
PR libstdc++/61107
* include/bits/stl_algo.h (__inplace_stable_partition): Delete.
(__stable_partitio
Tobias Grosser wrote:
On 10.11.2014 20:14, Roman Gareev wrote:
Sure. We should drop the flag in these test cases.
This seems to make sense, as they now test something different and
the flag removal would reflect this.
I personally would include this in the same patch. Would this be
difficult
On 09/11/14 16:00 -0800, Siva Chandra wrote:
Hello,
Attached is a patch which adds xmethods for the associative containers
(set, map, multiset and multimap) and their unordered versions. I
think the GDB Python API is not rich enough to implement xmethods for
the more interesting methods like fin
On Mon, 10 Nov 2014, Richard Biener wrote:
No extra includes required?
is already included in wide-int.h and rtl.h, should probably
move those.
--
Marc Glisse
On Nov 10, 2014, at 1:12 PM, Andreas Tobler wrote:
> As I was told, arm*-*-symbianelf* should be EABI so we can use arm_eabi for
> all instead of listing each OS.
>
> Ok for trunk?
Ok.
Hi,
For the following test-case:
struct A {};
clang -fsyntax-only -Wc++-compat gives following warning and gcc does not:
empty-struct.c:1:1: warning: empty struct has size 0 in C, size 1 in
C++ [-Wc++-compat]
struct F {};
This patch adds the above warning to the C FE.
Bootstrapped on x86_64-unkn
[ sorry for dup, if any ]
On Nov 10, 2014, at 1:12 PM, Andreas Tobler wrote:
> As I was told, arm*-*-symbianelf* should be EABI so we can use arm_eabi for
> all instead of listing each OS.
>
> Ok for trunk?
Ok.
> Hi!
>
> As the following two testcases shows, there are lots of issues in
> ICF compare_gimple_call, in particular, it doesn't handle indirect calls
> properly (see the ipa-icf-31.c testcase), doesn't handle internal calls
> properly (see ubsan/ipa-icf-1.c), didn't check gimple_call flags at all
On Tue, Nov 11, 2014 at 03:24:48AM +0530, Prathamesh Kulkarni wrote:
> * gcc/c/c-decl.c
> (warn_cxx_compat_finish_struct): Add new parameter of type location_t.
> Warn for empty struct.
> (finish_struct): Pass loc to warn_cxx_compat_finish_struct.
>
> * gcc/testsuite/gcc.dg/Wcxx-compat-22.
Hi all,
another one. Here I'm not really sure if there are EABI variants which
do _not_ support these test cases.
My target succeeds with this patch.
Comments?
If no (comments), ok for trunk?
TIA,
Andreas
2014-11-10 Andreas Tobler
* g++.old-deja/g++.jason/enum6.C: Use arm_eabi
Cesar Philippidis wrote:
This patch adds support for OpenACC 2.0a, with some omissions, to the
fortran front end. It only contains the fortran changes from
gomp-4_0-branch, therefore the middle end and runtime changes are a
necessary prerequisite for this patch.
I'd assume that one could commit
I introduced the random tests after Christopher Jefferson request
to have more intensive tests on those algos. Is it the whole stuff of
tests using random numbers that you don't like or just the usage of
mt19937 ? If second is this new version using the usual random_device I
used so far bet
On 10/11/14 23:14 +0100, François Dumont wrote:
I introduced the random tests after Christopher Jefferson request
to have more intensive tests on those algos. Is it the whole stuff of
tests using random numbers that you don't like or just the usage of
mt19937 ?
The use of random number in
On Sat, 2014-11-08 at 14:56 +0100, Jakub Jelinek wrote:
> On Sat, Nov 08, 2014 at 01:07:28PM +0100, Richard Biener wrote:
> > To be constructive here - the above case is from within a
> > GIMPLE_ASSIGN case label
> > and thus I'd have expected
> >
> > case GIMPLE_ASSIGN:
> > {
> >
On Fri, Oct 24, 2014 at 01:06:41PM +0100, Alan Lawrence wrote:
> This migrates the reduction patterns in altivec.md and vector.md to
> the new names. I've not touched paired.md as I wasn't really sure
> how to fix that (how do I vec_extractv2sf ?), moreover the testing I
> did didn't seem to exerci
No the random tests didn't show any problem. I had demonstrated the
problems with the modifications on the existing tests simulating
constraint memory context.
So unless specified otherwise I will commit tomorrow without the tests
using random numbers.
François
On 10/11/2014 23:20, Jonatha
On 11/10/2014 02:08 PM, Tobias Burnus wrote:
> Cesar Philippidis wrote:
>> This patch adds support for OpenACC 2.0a, with some omissions, to the
>> fortran front end. It only contains the fortran changes from
>> gomp-4_0-branch, therefore the middle end and runtime changes are a
>> necessary prereq
On 10/11/14 23:39 +0100, François Dumont wrote:
No the random tests didn't show any problem. I had demonstrated the
problems with the modifications on the existing tests simulating
constraint memory context.
So unless specified otherwise I will commit tomorrow without the tests
using random n
On 05/11/14 17:49 +, Jonathan Wakely wrote:
On 5 November 2014 14:14, David Edelsohn wrote:
Jonathan,
I still am seeing new failures in the libstdc++ deque testsuite as of
last night. I don't know if you still are working through the fallout
from the earlier patches, but I wanted to make y
On Mon, Nov 10, 2014 at 10:08:54PM +0100, Richard Biener wrote:
> >@@ -662,9 +662,49 @@ func_checker::compare_gimple_call (gimpl
> > t1 = gimple_call_fndecl (s1);
> > t2 = gimple_call_fndecl (s2);
>
> Just drop these and compare gimple_call_fn only.
>
> >+ tree chain1 = gimple_call_chain (s1
On Sun, Nov 02, 2014 at 08:31:43PM +0100, Uros Bizjak wrote:
> Attached (mechanical) patch adds i?86-*-* target to many tests, where
> only x86_64-*-* is listed. Please note that x86_64-*-* already
> included && lp64 for 64bit specific tests due to -m32 multilib
> testing.
>
> 2014-11-02 Uros Biz
On Tue, Nov 11, 2014 at 3:35 AM, Marek Polacek wrote:
> On Tue, Nov 11, 2014 at 03:24:48AM +0530, Prathamesh Kulkarni wrote:
>> * gcc/c/c-decl.c
>> (warn_cxx_compat_finish_struct): Add new parameter of type location_t.
>> Warn for empty struct.
>> (finish_struct): Pass loc to warn_cxx_comp
Ping.
On Mon, Oct 6, 2014 at 1:43 PM, Sriraman Tallam wrote:
> Ping.
>
> On Mon, Sep 29, 2014 at 10:57 AM, Sriraman Tallam wrote:
>> Ping.
>>
>> On Fri, Sep 19, 2014 at 2:11 PM, Sriraman Tallam wrote:
>>> Hi Richard,
>>>
>>> I also ran the gcc testsuite with
>>> RUNTESTFLAGS="--tool_opts=-mcopy
Hello!
Now that Vlad fixed the real problem of PR 63620 [1], we can remove
the temporary workaround. The patch also adds the testcase from PR.
2014-11-11 Uros Bizjak
Revert:
2014-10-31 Uros Bizjak
PR target/63620
* config/i386/i386-protos.h (ix86_use_pseudo_pic_reg): Decla
On Mon, Nov 10, 2014 at 2:27 PM, David Malcolm wrote:
> On Sat, 2014-11-08 at 14:56 +0100, Jakub Jelinek wrote:
>> On Sat, Nov 08, 2014 at 01:07:28PM +0100, Richard Biener wrote:
>> > To be constructive here - the above case is from within a
>> > GIMPLE_ASSIGN case label
>> > and thus I'd have exp
On Nov 10, 2014, at 2:06 PM, Andreas Tobler wrote:
> another one. Here I'm not really sure if there are EABI variants which do
> _not_ support these test cases.
I think the patch is fine, just watch for any follow-on comments from an
eabi/arm expert. Usually they are pretty responsive.
Backport pending upstream patch to fix demangler crash.
https://gcc.gnu.org/ml/gcc-patches/2014-05/msg02279.html
This patch is for the google/gcc-4_9 branch.
Google ref: 17891596
-cary
2014-05-27 Pedro Alves
include/
* demangle.h (enum demangle_component_type)
: New value.
On Mon, Nov 10, 2014 at 3:56 PM, Cary Coutant wrote:
> Backport pending upstream patch to fix demangler crash.
>
> https://gcc.gnu.org/ml/gcc-patches/2014-05/msg02279.html
>
> This patch is for the google/gcc-4_9 branch.
>
> Google ref: 17891596
>
> -cary
OK for Google branches.
Hi Jeff,
I have adapted the code generation part from James' patch to current trunk, and
the resulting patch gets the 30% speedup on coremark and passes bootstrap of
GCC.
Ok for trunk?
Thanks,
Sebastian
Sebastian Pop wrote:
> Sebastian Pop wrote:
> > Jeff Law wrote:
> > > On 08/21/14 04:30, Ri
On x86_64-apple-darwin14, the attached patch allows gcc trunk to
build against isl 0.14. I assume if we want to retain the...
#if defined(__cplusplus)
extern "C" {
#endif
#if defined(__cplusplus)
}
#endif
wrappers around the include of isl/val_gmp.h, to continue to support
isl 0.12.2, isl.m
Hi,
This patch tweaks the VRP code to simply inspect the need_assert_for
bitmap when determining whether any asserts need to be inserted.
Consequently we no longer have to manually keep track of whether a call
to register_new_assert_for() was made.
This patch is an updated version of a patch that
This patch refactors the VRP edge-assertion code to make it always
traverse SSA-name definitions in order to find suitable edge assertions
to insert. Currently SSA-name definitions get traversed only when the
LHS of the original conditional is a bitwise AND or OR operation which
seems like a stran
On 11/10/2014 12:10 PM, Jason Merrill wrote:
On 11/10/2014 10:55 AM, Ed Smith-Rowland wrote:
Would a 4.9 version be accepted?
Sure.
What do you think about defining the macros for unsupported features
to 0 rather than leaving them undefined? The document doesn't seem to
specify.
Jason
On Fri, Oct 24, 2014 at 1:55 AM, Richard Biener
wrote:
> On Fri, Oct 24, 2014 at 8:28 AM, wrote:
>> From: Trevor Saunders
>>
>> Hi,
>>
>> do $subject, and cleanup for always 64 bit hwi.
>>
>>
>> bootstrapped + regtested x86_64-unknown-linux-gnu, ok?
>
> Ok. Can you please replace remaining HOS
Cesar Philippidis wrote:
This patch contains compile-time tests for OpenACC in gfortran. Is this
patch OK for mainline trunk after the OpenACC fortran front end changes
make their way in?
I browsed the test cases – and they look okay. I didn't try hard to see
whether every single one is valid
On Mon, Nov 10, 2014 at 05:30:38PM +0300, Ilya Verbin wrote:
> On 06 Nov 19:25, Jakub Jelinek wrote:
> > Oh, one more point, if mic_lib_path is NULL, what is the point
> > to do the alloca/malloc and string copying? Can't you just
> > setenv (MIC_LD_LIBRARY_PATH_ENV, ld_lib_path, 1);
> > in that
On Mon, Nov 10, 2014 at 05:34:30PM +0300, Ilya Verbin wrote:
> Done, I put them into env vars.
>
> > > +lappend ALWAYS_CFLAGS
> > > "additional_flags=${offload_additional_options}"
> > > }
> >
> > Perhaps add this only if offload_additional_options is non-empty?
>
> Done.
Ok (with appropr
On Mon, Nov 10, 2014 at 05:36:24PM -0500, Michael Meissner wrote:
> However, the double pattern is completely broken. This cannot go in.
[snip]
> It is unacceptable to have to do the inner loop doing a load, vector add, and
> store in the loop.
Before the patch, the final reduction used *vsx_re
On Mon, Nov 10, 2014 at 02:43:38PM -0800, Cesar Philippidis wrote:
> >> I'll post a separate patch with the fortran tests later. If anyone
> >> wants to test this patch, please use gomp-4_0-branch instead. You
> >> don't need a CUDA accelerator to use
> >> OpenACC, and some of the runtime tests wil
On Mon, Nov 10, 2014 at 05:27:50PM -0500, David Malcolm wrote:
> On Sat, 2014-11-08 at 14:56 +0100, Jakub Jelinek wrote:
> > On Sat, Nov 08, 2014 at 01:07:28PM +0100, Richard Biener wrote:
> > > To be constructive here - the above case is from within a
> > > GIMPLE_ASSIGN case label
> > > and thus
PING! I am worried if it goes in stage-1.
-Original Message-
From: Gopalasubramanian, Ganesh
Sent: Thursday, October 30, 2014 2:24 PM
To: gcc-patches@gcc.gnu.org
Subject: [PATCH, aarch64] Add prefetch support
Hi,
Below is the patch that implements prefetching support.
This patch has be
Hi,
Dose anybody have time to review this? Thanks.
>
> Hello,
>
> Ping for https://gcc.gnu.org/ml/gcc-patches/2014-10/msg02933.html
>
> Thanks
Hello!
>>> do $subject, and cleanup for always 64 bit hwi.
>>>
>>>
>>> bootstrapped + regtested x86_64-unknown-linux-gnu, ok?
>>
>> Ok. Can you please replace remaining HOST_WIDE_INT
>> vestiges in there with [u]int64_t please?
>
>
> This patch breaks the build on debian 6.0:
>
> ../../gcc/sreal.
101 - 151 of 151 matches
Mail list logo
