On Tue, Nov 11, 2014 at 4:09 PM, Marc Glisse wrote:
> Hello,
>
> here is the combined patch+ChangeLog. I'll run a last regtest just before
> committing. Ok for trunk?
>
> 2014-11-12 Marc Glisse
>
> gcc/
> * config/i386/xmmintrin.h (_mm_add_ps, _mm_sub_ps, _mm_mul_ps,
> _mm_div_p
On Tue, 11 Nov 2014, Jonathan Wakely wrote:
+ template>
Is there a particular reason not to write "void" directly?
--
Marc Glisse
Hi,
this proposed patch adds an option "-Warray-bounds=" in addition to
"-Warray-bound". "-Warray-bounds=1" corresponds to "-Warray-bound".
For higher warning levels more warnings about optional accesses
outside of arrays are emitted. For example, warnings for
arrays accessed through pointers are
> -Original Message-
> From: Ulrich Weigand [mailto:uweig...@de.ibm.com]
> Sent: Friday, November 07, 2014 12:11 AM
> To: Richard Henderson
> Cc: Zhenqiang Chen; 'Jan-Benedict Glaw'; Hartmut Penner; Andreas Krebbel;
> gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH, ifcvt] Allow CC mode if
> On Tue, Nov 11, 2014 at 3:22 PM, Martin Liška wrote:
> > Hello.
> >
> > Following patch adds checking for aliasing support. Patch can bootstrap on
> > x86_64-apple-darwin1 and is part of patches needed for bootstrap restory on
> > the target. I plan to introduce additional patch that will cover
On Tue, Nov 11, 2014 at 8:48 AM, Richard Biener
wrote:
> On Tue, Nov 11, 2014 at 1:10 PM, Patrick Palka wrote:
>> This patch is a replacement for the 2nd VRP refactoring patch. It
>> simply teaches VRP to look through widening type conversions when
>> finding suitable edge assertions, e.g.
>>
>>
This patch to libgo updates the libtool support to that currently
found in the gcc trunk. This includes a patch for support of OS X
10.10. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
This is Go issue 9089.
Ian
diff -r 1c308e945873 libgo/config/libtool.m
> On Wed, Oct 22, 2014 at 10:49 PM, Michael Collison
>
> wrote:
> >
> > Patch that removes extraneous comment attached.
> >
> > The CLZ_DEFINED_VALUE_AT_ZERO macro is hard coded to return 32. For
> > the vector intrinsic vclz this is incorrect and should return the
> > value
>
> vclz_{s,u}8 ...
On Tue, Nov 4, 2014 at 7:54 AM, Jiong Wang wrote:
>
> On 04/11/14 15:51, Paolo Carlini wrote:
>>
>> Hi,
>>
>> On 11/04/2014 04:37 PM, Martin Jambor wrote:
>>>
>>> Hi,
>>>
>>> since revision 216728, testsuite/g++.dg/ipa/devirt-40.C is failing
>>> because although the tested-for devirtualization doe
Hi,
This patch updates c-c++-common/asan/strlen-overflow-1.c to avoid tail
call on x32. Tested on Linux/x86-64/ia32/x32. OK to install?
Thanks.
H.J.
---
2014-11-11 H.J. Lu
PR testsuite/63830
* c-c++-common/asan/strlen-overflow-1.c (main): Avoid tail call.
diff --git a/gcc/
On Tue, Nov 11, 2014 at 01:10:01AM -0600, Segher Boessenkool wrote:
> 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 a
This patch adds 2 tests to the testsuite to make sure the -mupper-regs-df and
-mupper-regs-sf options work, and you can generate add, subtract, multiply,
divide, and compare instructions on scalars living in the Altivec registers. I
also fixed the p8vector-ldst.c test, which has been broken for so
The attached patch adds a news entry about the JIT merge to the
frontpage of the website.
The page seems to have some pre-existing validation errors; I don't
think this adds any new ones.
OK?
Thanks
Dave
Index: htdocs/index.html
===
This patch documents the previously undocumented -mupper-regs-df and
-mupper-regs-sf switches. It also provides feature test macros that users can
use to determine if the upper register support is enabled.
Once the prevous patches have gone in, is this patch ok to install?
2014-11-11 Michael Me
This is the big patch that enables the upper regs support. It reorganizes the
secondary reload handler to try and make it easier to understand, by having a
variable that says it is done, rather than using cascading if statements. The
secondary reload inner function (which is called from the reloa
On Sun, Oct 5, 2014 at 9:34 AM, Jan Hubicka wrote:
> Hi,
> this patch makes ipa_polymorphic_call_context::get_dynamic_type to
> track if vptr has changed in between function entry and polymorphic
> call.
>
> The patch also add logic skipping if (a==0) checks in multiple
> inheritance so I can cons
This patch sets up some of the support that will be needed in the next patch,
and updates the debug functions. It also adds checks to make sure the upper
regs support has the other options enabled. Is this patch acceptable to be
checked in once the PowerPC bootstraps?
2014-11-11 Michael Meissne
Last year, in preparation for the upper regs patch, I went through the
instructions, and changed the iterators, so the vector support was in vsx.md,
while the scalar support was in rs6000.md. I missed the int<->fp conversion
operators. This patch moves the scalar support into rs6000.md. Note, th
When I did the original power7 work, I put the reload handlers into vector.md,
since they were only used for vector types. Since they are now more general, I
am moving these insns from vector.md to rs6000.md. Is this patch acceptable to
be checked in once the PowerPC boostraps again.
2014-11-11
At one stage in working on the patches, I was editing the easy_fp_constant
predicate, and I noticed there was a test for the constant 0.0 at the top of
the function, as well as redundant tests in SFmode/DFmode. I deleted these
redundant tests. Is the patch acceptable to check in once the PowerPC
On 12/11/14 00:16 +, Jonathan Wakely wrote:
The _Has_addressof trait only detected an overloaded operator&() as a
member function, this also checks for a non-member.
Also clean up the tests by removing some unused headers.
Tested x86_64-linux, committed to trunk.
commit df1baf34aaf4b6e9aaf
This set of 7 patches finally adds the support to allow scalar floating point
values to live in the traditional Altivec registers on machines that have the
appropriate support.
ISA 2.06 (i.e. power7): double precision can go in Altivec registers
ISA 2.07 (i.e. power8): single preci
The _Has_addressof trait only detected an overloaded operator&() as a
member function, this also checks for a non-member.
Tested x86_64-linux, committed to trunk.
commit 32b294e3eddf7789c25f3b751ae6b2a2c5c43275
Author: Jonathan Wakely
Date: Mon Nov 10 19:02:02 2014 +
* include/experi
This implements LWG 2048. We've been shipping a SFINAE-friendly
iterator_traits in C++11 mode for years, but this adjusts it to match
the form that will be in the next standard i.e. require all the nested
members, not only iterator_category.
To implement this I added __void_t, which is identical
> > 150 and 200 make Silvermont performance better on 173.applu (+8%) and
> > 183.equake (+3%); Haswell spec2006 performance stays almost unchanged.
> > Higher value of 300 leave the performance of mentioned tests
> > unchanged, but add some regressions on other benchmarks.
> >
> > So I like 200 a
On Tue, Oct 28, 2014 at 1:21 AM, Maxim Ostapenko
wrote:
>
>>> The code which identifies interceptors seems to be duplicated in
>>> expand_builtin. What about factoring this out to some
>>> is_asan_intercepted()
>>> in asan.h?
>>
>> I agree.
>
>
> Fixed this.
>>
>>
>> BTW, when you handle a built
Hi,
The problem here is that aarch64-builtins.c contains gty markers but
does not include gt-aarch64-builtins.h and is not included in the
target_gtfiles list in config.gcc. So sometimes the builtins get
garbage collected when they should not be.
Committed as obvious after a build and test on a
My recent patch to add ref-qualifier support to std::mem_fn didn't
cover C-style varargs functions (as opposed to C++ variadic templates,
which work fine). Rather than double the number of partial
specializations this generates them using a macro.
The arity-checking needs to be adjusted to allow
On Tue, Nov 11, 2014 at 04:05:57PM +0100, Mircea Namolaru wrote:
> > I'm not sure if Tobi or Albert have told you, but the separation_class
> > option
> > is going to be phased out since its design is fundamentally flawed.
> > If you can't wait until isl-0.15, then I guess you have no choice but
>
On 11/08/14 03:27, Kai Tietz wrote:
Hi Jeff,
2014-11-07 21:03 GMT+01:00 Jeff Law :
On 11/06/14 12:37, Kai Tietz wrote:
Hi,
This fixes recent fallout of debug-tests on Windows target for sdbout
(coff) caused by an ICE.
ChangeLog
2014-11-06 Kai Tietz
* sdbout.c (sdbout_symbol): Eli
On 11 November 2014 17:10, Jason Merrill wrote:
> On 10/31/2014 10:01 PM, Manuel López-Ibáñez wrote:
>>
>> +sense. So let's live the location intact in that case. */
>
>
> s/live/leave/
>
>> + /* If MAP is not the last line map of its set, then the new location
>> + (loc + offset) s
In https://gcc.gnu.org/ml/gcc-patches/2012-01/msg00363.html, Tom reported an
ICE in the DWARF CFI pass because a frame-related insn was speculated by the
reorg pass and, therefore, disabled the optimization. It turns out that the
same code (when condition == const_true_rtx) can also duplicate f
On 11/11/14 11:38 +, Jonathan Wakely wrote:
On 10/11/14 21:49 +, Jonathan Wakely wrote:
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
This is the RTL checking failure triggered by gcc.dg/vect/vect-singleton_1.c
on SPARC 64-bit: the back-end gets confused passing 1-element float vectors.
As documented in the code, we pass all FP vectors like aggregate types.
Tested on SPARC64/Solaris, applied on the mainline.
2014-11-11 Eric
On Tue, 11 Nov 2014, H.J. Lu wrote:
> Currently mpx-runtime uses
>
> dlsym(RTLD_NEXT, "sigaction");
>
> to get the original sigaction in liibc.so. Glibc does provides __sigaction
> for the original sigaction. But dlsym(RTLD_NEXT, is more portable.
I guess there's a question of what the portab
On Tue, Nov 11, 2014 at 1:33 PM, Joseph Myers wrote:
> On Tue, 11 Nov 2014, Andi Kleen wrote:
>
>> On Tue, Nov 11, 2014 at 01:04:42PM -0800, H.J. Lu wrote:
>> > On Tue, Nov 11, 2014 at 1:01 PM, Andi Kleen wrote:
>> > >> It is similar to libsanitizer. Put it in glibc isn't going to work well
>> >
On Tue, 11 Nov 2014 17:51:01 +0100
Jakub Jelinek wrote:
> On Tue, Nov 11, 2014 at 02:52:20PM +, Julian Brown wrote:
> > On Tue, 11 Nov 2014 08:10:29 +0100
> > Jakub Jelinek wrote:
> >
> > We've been preparing new patches against trunk for the libgomp and
> > middle-end bits: I've now posted
On 11/10/14 09:59, Renlin Li wrote:
On 06/11/14 18:07, Renlin Li wrote:
On 06/11/14 17:59, Teresa Johnson wrote:
Thanks for fixing the test case. Can you also add the comment I
suggested to the source change?
Please add a comment that this is needed due to insane incoming
frequencies.
Sorr
Ping
On Tue, 11 Nov 2014, Andi Kleen wrote:
> On Tue, Nov 11, 2014 at 01:04:42PM -0800, H.J. Lu wrote:
> > On Tue, Nov 11, 2014 at 1:01 PM, Andi Kleen wrote:
> > >> It is similar to libsanitizer. Put it in glibc isn't going to work well
> > >> for MPX.
> > >
> > > Can you explain it more please?
> >
On Wed, Nov 12, 2014 at 02:08:03AM +0530, Prathamesh Kulkarni wrote:
> Is this version okay ?
I have no further comments on this patch, so deferring to Joseph.
Thanks,
Marek
Hi!
On Thu, 5 Jun 2014 16:00:16 +0200, I wrote:
> --- gcc/omp-low.c
> +++ gcc/omp-low.c
> @@ -1738,6 +1754,10 @@ scan_sharing_clauses (tree clauses, omp_context *ctx)
> {
> tree base = get_base_address (decl);
> tree nc = OMP_CLAUSE_CHAIN (c);
> + gcc_as
On Tue, Nov 11, 2014 at 01:04:42PM -0800, H.J. Lu wrote:
> On Tue, Nov 11, 2014 at 1:01 PM, Andi Kleen wrote:
> >> It is similar to libsanitizer. Put it in glibc isn't going to work well
> >> for MPX.
> >
> > Can you explain it more please?
> >
>
> Are you suggesting putting MPX run-time in glib
Hi!
On Wed, 5 Nov 2014 17:57:10 +, Julian Brown wrote:
> This patch moves plugin-nvptx.c [...]
> into a new "plugin" subdirectory, [...]
The "move" aspect got lost in the commit; now made up for in r217371 on
gomp-4_0-branch:
commit 5296af1a8fa4a53505a74260397cd71ef4cfa45b
Author: tschwinge
On Nov 11, 2014, at 11:48 AM, Jack Howarth wrote:
> The only problem with this fix for the broken libtool.m4 is that
> it will require constant tending as other patches regenerate these
> various configure files. For example,
> https://gcc.gnu.org/ml/gcc-cvs/2014-11/msg00379.html which was jus
On Tue, Nov 11, 2014 at 1:01 PM, Andi Kleen wrote:
>> It is similar to libsanitizer. Put it in glibc isn't going to work well
>> for MPX.
>
> Can you explain it more please?
>
Are you suggesting putting MPX run-time in glibc? Will we have
2 glibc, one with MPX run-time and one without MPX run-t
> It is similar to libsanitizer. Put it in glibc isn't going to work well
> for MPX.
Can you explain it more please?
-Andi
On 11/11/2014 03:06 PM, Markus Trippelsdorf wrote:
On 2014.11.11 at 13:39 -0500, Jason Merrill wrote:
On 11/11/2014 10:37 AM, Markus Trippelsdorf wrote:
On 2014.11.11 at 10:11 -0500, Jason Merrill wrote:
On 11/08/2014 06:57 AM, Markus Trippelsdorf wrote:
+++ b/gcc/testsuite/g++.old-deja/g++.p
On Wed, Nov 12, 2014 at 1:28 AM, Marek Polacek wrote:
> On Tue, Nov 11, 2014 at 11:27:21PM +0530, Prathamesh Kulkarni wrote:
>> I tried the following:
>> struct A { struct B {}; int x; } /* { dg-warning "empty struct has
>> size 0 in C" } */
>>/* { dg-bo
This patch adds a predefined macro __NO_MATH_ERRNO__ for when
-fno-math-errno is passed or implied. This allows math.h to provide a
more accurate definition of the C99 math_errhandling macro that takes
this option into account, and allows for choice of libm functions to
be optimized at compile tim
> Totally agree with Mike here. If you look at the patch it's clear it's just
> hitting Darwin and it's absolutely safe.
Thanks everyone for the comments and review.
Committed as r217366
FX
On Tue, 11 Nov 2014, Andi Kleen wrote:
> Joseph Myers writes:
>
> > On Tue, 11 Nov 2014, Ilya Enkovich wrote:
> >
> >> Hi,
> >>
> >> This patch integrates MPX runtime library into GCC source tree. MPX
> >> runtime is responsible for initialization of MPX feature in HW, signal
> >> handling,
2014-11-11 21:09 GMT+03:00 Joseph Myers :
> On Tue, 11 Nov 2014, Joseph Myers wrote:
>
>> You have lots of static writable variables. Are you sure all those
>> variables are handled in a thread-safe way (e.g. only modified before any
>> threads start)?
>
> Another general multi-thread issue for li
2014-11-11 23:02 GMT+03:00 Jeff Law :
> On 11/11/14 08:34, Ilya Enkovich wrote:
>>
>> Hi,
>>
>> This patch integrates MPX runtime library into GCC source tree. MPX
>> runtime is responsible for initialization of MPX feature in HW, signal
>> handling, reporting etc. Library is linked to codes comp
On Tue, Nov 11, 2014 at 10:49 AM, Andi Kleen wrote:
> Joseph Myers writes:
>
>> On Tue, 11 Nov 2014, Ilya Enkovich wrote:
>>
>>> Hi,
>>>
>>> This patch integrates MPX runtime library into GCC source tree. MPX
>>> runtime is responsible for initialization of MPX feature in HW, signal
>>> handling
On 11/11/14 13:10, Ilya Enkovich wrote:
tests are compiled ignoring CPUID, mpx.exp only checks tests may be
compiled (i.e. binutils support MPX, runtime wasn't disabled etc.).
All MPX binaries are supposed to work correctly on hardware without
MPX, we just shouldn't expect any bound violations de
2014-11-11 22:55 GMT+03:00 Jeff Law :
> On 11/11/14 11:42, Mike Stump wrote:
>>
>> On Nov 11, 2014, at 8:02 AM, Ilya Enkovich wrote:
>>>
>>> This patch adds simple runtime MPX tests.
>>>
>>> Make check shows no fails for mpx.exp suite for
>>> x86_64-unknown-linux-gnu{-m32,-mx32,}. OK for trunk?
>
On 2014.11.11 at 13:39 -0500, Jason Merrill wrote:
> On 11/11/2014 10:37 AM, Markus Trippelsdorf wrote:
> > On 2014.11.11 at 10:11 -0500, Jason Merrill wrote:
> >> On 11/08/2014 06:57 AM, Markus Trippelsdorf wrote:
> >>> +++ b/gcc/testsuite/g++.old-deja/g++.pt/explicit73.C
> >>> @@ -7,9 +7,9 @@
> >
On 11/11/14 08:34, Ilya Enkovich wrote:
Hi,
This patch integrates MPX runtime library into GCC source tree. MPX runtime is
responsible for initialization of MPX feature in HW, signal handling, reporting
etc. Library is linked to codes compiled with -mmpx.
Bootstrap is OK for x86_64-unknown-
On Tue, Nov 11, 2014 at 11:27:21PM +0530, Prathamesh Kulkarni wrote:
> I tried the following:
> struct A { struct B {}; int x; } /* { dg-warning "empty struct has
> size 0 in C" } */
>/* { dg-bogus "delcaration
> does not declare anything" } */
> but it f
On 11/11/14 11:42, Mike Stump wrote:
On Nov 11, 2014, at 8:02 AM, Ilya Enkovich wrote:
This patch adds simple runtime MPX tests.
Make check shows no fails for mpx.exp suite for
x86_64-unknown-linux-gnu{-m32,-mx32,}. OK for trunk?
Ok with any changes necessary for:
I’m assuming that this w
Hi Jack,
> The only problem with this fix for the broken libtool.m4 is that
> it will require constant tending as other patches regenerate these
> various configure files.
As I explained in an earlier email today, you are mistaken about the patch. It
does fix libtool.m4, which means that all fur
Joseph Myers writes:
[...]
>> Neither Per nor Tom are active in GCC anymore. If the FE maintainers
>> do not feel comfortable reviewing line-map changes, could you nominate
>> Dodji as line-map maintainer if he is willing to accept it? I think he
>> is currently the person that understands that
Mike,
The only problem with this fix for the broken libtool.m4 is that
it will require constant tending as other patches regenerate these
various configure files. For example,
https://gcc.gnu.org/ml/gcc-cvs/2014-11/msg00379.html which was just
committed would have removed the proposed change h
On 11/11/14 12:38, Mike Stump wrote:
2014-11-11 Francois-Xavier Coudert
PR target/63610 * configure:
Missing Regenerate.
OK to commit?
Ok.
This touches so many area it probably needs a build maintainer or
global maintainer to approve it.
Only darwin is affected, and unlikely anyone e
On Nov 11, 2014, at 11:06 AM, FX wrote:
> Here’s v2 of the patch
> OK to commit?
Ok with comments:
libvtv:
libvtv/
2014-11-11 Francois-Xavier Coudert
PR target/63610
* configure:
Missing Regenerate.
> OK to commit?
Ok.
> This touches so many area it probably needs a bu
On 11/11/2014 08:51 AM, Jakub Jelinek wrote:
>> We've been preparing new patches against trunk for the libgomp and
>> middle-end bits: I've now posted the former, and the latter are on
>> their way soon, I believe. The middle-end bits are also present on the
>> gomp-4_0-branch SVN branch (likewise
Here’s v2 of the patch, including libjava/configure and
libjava/classpath/configure, as well as zlib/configure (since it has some
adaptations from upstream, documented in zlib/ChangeLog.gcj).
OK to commit? Bootstrapped with all supported languages on
x86_64-apple-darwin14.
FX
--
libtool.
On Nov 11, 2014, at 8:51 AM, Jiong Wang wrote:
> currently, if a testcase use dg-output, then absolute file path will occur
> in the test summary reported by dejagnu which is bad.
> ok for trunk?
Ok.
Joseph Myers writes:
> On Tue, 11 Nov 2014, Ilya Enkovich wrote:
>
>> Hi,
>>
>> This patch integrates MPX runtime library into GCC source tree. MPX
>> runtime is responsible for initialization of MPX feature in HW, signal
>> handling, reporting etc. Library is linked to codes compiled with
On November 11, 2014 6:49:34 PM CET, Jakub Jelinek wrote:
>On Tue, Nov 11, 2014 at 06:40:25PM +0100, Marek Polacek wrote:
>> --- gcc/fold-const.c
>> +++ gcc/fold-const.c
>> @@ -7862,9 +7862,15 @@ fold_unary_loc (location_t loc, enum tree_code
>code, tree type, tree op0)
>>return fold_view_
On Nov 11, 2014, at 8:02 AM, Ilya Enkovich wrote:
> This patch adds simple runtime MPX tests.
>
> Make check shows no fails for mpx.exp suite for
> x86_64-unknown-linux-gnu{-m32,-mx32,}. OK for trunk?
Ok with any changes necessary for:
I’m assuming that this won’t break i386 (targets without
On 11/11/2014 10:37 AM, Markus Trippelsdorf wrote:
On 2014.11.11 at 10:11 -0500, Jason Merrill wrote:
On 11/08/2014 06:57 AM, Markus Trippelsdorf wrote:
+++ b/gcc/testsuite/g++.old-deja/g++.pt/explicit73.C
@@ -7,9 +7,9 @@
// the template
namespace N {
- template class foo; // { dg-error
On 11/11/2014 12:37 PM, Paolo Carlini wrote:
Committed. What about 4.9? Technically the issue is a regression, but in
my opinion the fix isn't completely trivial for a release branch..
I think it's safe enough for a regression fix.
Jason
On Nov 11, 2014, at 7:37 AM, Markus Trippelsdorf wrote:
> Both EGG and clang currently accept it.
EDG… I think you mean.
On Nov 11, 2014, at 6:22 AM, Martin Liška wrote:
>
> Following patch adds checking for aliasing support. Patch can bootstrap on
> x86_64-apple-darwin1
I’m impressed, must have been a lot of work to port darwin1 to x86_64. :-)
On 2014.11.10 at 10:55 -0500, Ed Smith-Rowland wrote:
> On 11/09/2014 11:45 PM, Jason Merrill wrote:
> > On 11/09/2014 08:33 PM, Ed Smith-Rowland wrote:
> >> + //cpp_hashnode *node = 0;
> >> + //node = token->val.node.node;
> >> + //if (node)
> >> + // pfile->mi_ind_cmacro = node;
> >
> >
On Tue, 11 Nov 2014, Joseph Myers wrote:
> You have lots of static writable variables. Are you sure all those
> variables are handled in a thread-safe way (e.g. only modified before any
> threads start)?
Another general multi-thread issue for libraries: files should be opened
with O_CLOEXEC (
On Tue, 11 Nov 2014, Ilya Enkovich wrote:
> Hi,
>
> This patch integrates MPX runtime library into GCC source tree. MPX
> runtime is responsible for initialization of MPX feature in HW, signal
> handling, reporting etc. Library is linked to codes compiled with
> -mmpx.
>
> Bootstrap is OK f
On Tue, Nov 11, 2014 at 4:30 PM, Marek Polacek wrote:
> On Tue, Nov 11, 2014 at 04:45:46AM +0530, Prathamesh Kulkarni wrote:
>> Index: gcc/c/c-decl.c
>> ===
>> --- gcc/c/c-decl.c(revision 217287)
>> +++ gcc/c/c-decl.c(working
On Tue, Nov 11, 2014 at 06:40:25PM +0100, Marek Polacek wrote:
> --- gcc/fold-const.c
> +++ gcc/fold-const.c
> @@ -7862,9 +7862,15 @@ fold_unary_loc (location_t loc, enum tree_code code,
> tree type, tree op0)
>return fold_view_convert_expr (type, op0);
>
> case NEGATE_EXPR:
> -
On Wed, Nov 05, 2014 at 11:50:20AM +0100, Jakub Jelinek wrote:
> On Wed, Nov 05, 2014 at 11:29:19AM +0100, Marek Polacek wrote:
> > On Wed, Nov 05, 2014 at 12:54:37PM +0300, Yury Gribov wrote:
> > > Are you going to work on ASan soon? I could rebase my patches on top of
> > > Marek's infrastructur
While match.pd has been changed to not transform -(-A) to A
when the overflow does not wrap or the signed integer ovreflow
checking is enabled, fold_negate_expr still happily does such
a transformation. That's bad because then we can't detect
an overflow, as on the following testcase. But I allow
Hi,
On 11/11/2014 04:03 PM, Jason Merrill wrote:
OK.
Committed. What about 4.9? Technically the issue is a regression, but in
my opinion the fix isn't completely trivial for a release branch..
Paolo.
Hello!
2014-11-11 Uros Bizjak
* config/alpha/alpha.c (alpha_emit_conditional_branch): Replace
open-coded swap with std::swap to swap values.
(alpha_emit_setcc): Ditto.
(alpha_emit_conditional_move): Ditto.
(alpha_split_tmode_pair): Ditto.
Bootstrapped and regression tested
I'm reading over the concepts code now with a view toward merging in the
next few weeks. We don't need to address all of my comments before
then, but it would be good to get started.
+ // The constraint info maintains information about constraints
+ // associated with the declaration.
+ tr
Hi,
The bswap pass iterate over the statements in a basic block and replace them by
a bswap, load or load+bswap when equivalent. However, to keep the code
equivalent the load need to be perform at the same place as where one of the
original load was made so that they have the same VUSE. Conside
On Tue, Nov 11, 2014 at 02:52:20PM +, Julian Brown wrote:
> On Tue, 11 Nov 2014 08:10:29 +0100
> Jakub Jelinek wrote:
>
> > 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 thi
currently, if a testcase use dg-output, then absolute file path will occur
in the test summary reported by dejagnu which is bad. because the summary
should only contain the test name and it's better to exclude all local test
machine info from the report.
dg-output eventually calls dejagnu generic
On 11/10/2014 01:49 PM, Joseph Myers wrote:
Then, do you mean that line-map falls under the reach of the
diagnostics maintainer? I agree, but Dodji himself does not seem to be
sure about this:
I think we should consider it to be covered by diagnostics maintainers.
That makes sense.
Jason
On 11 November 2014 11:27, James Greenhalgh wrote:
> 2014-11-11 James Greenhalgh
>
> * config/aarch64/aarch64-simd.md
> (aarch64_simd_bsl_internal): Remove float cases, canonicalize.
> (aarch64_simd_bsl): Add gen_lowpart expressions where we
> are punning betwee
> 150 and 200 make Silvermont performance better on 173.applu (+8%) and
> 183.equake (+3%); Haswell spec2006 performance stays almost unchanged.
> Higher value of 300 leave the performance of mentioned tests
> unchanged, but add some regressions on other benchmarks.
>
> So I like 200 as well as 12
On 11/11/14 08:44, Richard Sandiford wrote:
There are no more callers to for_each_rtx or for_each_rtx_in_insn,
so this patch removes the functions.
Tested on x86_64-linux-gnu. OK to install?
Thanks,
Richard
gcc/
* rtl.h (rtx_function, for_each_rtx, for_each_rtx_in_insn): Delete.
On 11/06/2014 02:05 PM, Richard Biener wrote:
On Wed, Nov 5, 2014 at 5:06 PM, Martin Jambor wrote:
In any case, if we want abstract origins to have this field set
correctly in abstract origins, we need to have the possibility to
modify it after IPA.
My point is we don't need the abstract orig
On 10/31/2014 10:01 PM, Manuel López-Ibáñez wrote:
+sense. So let's live the location intact in that case. */
s/live/leave/
+ /* If MAP is not the last line map of its set, then the new location
+ (loc + offset) should be less than the first location encoded by
+ the next l
On Mon, Nov 10, 2014 at 11:42 AM, H.J. Lu wrote:
> On Mon, Nov 10, 2014 at 5:44 AM, Richard Biener
> wrote:
>> On Mon, Nov 10, 2014 at 2:43 PM, Jakub Jelinek wrote:
>>> On Mon, Nov 10, 2014 at 05:32:32AM -0800, H.J. Lu wrote:
On Mon, Nov 10, 2014 at 4:05 AM, Jakub Jelinek wrote:
> On
Ping?
>
> > -Original Message-
> > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> > ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
> > Sent: Wednesday, August 20, 2014 9:28 AM
> > To: gcc-patches@gcc.gnu.org
> > Subject: [PATCH][ARM] Fix -fcall-saved-rX for X > 7
> >
> > T
There are no more callers to for_each_rtx or for_each_rtx_in_insn,
so this patch removes the functions.
Tested on x86_64-linux-gnu. OK to install?
Thanks,
Richard
gcc/
* rtl.h (rtx_function, for_each_rtx, for_each_rtx_in_insn): Delete.
* rtlanal.c (non_rtx_starting_operands, fo
On 11/11/2014 04:02 PM, David Edelsohn wrote:
> Michael,
>
> Why does the configure change match with p*-*-aix... instead of power*
> or powerpc*? Yes, it's unique and will match, but why make it as
> short as possible, which doesn't match other uses?
Actually I did have powerpc* initially, bu
Fixed the subject.
Best regards,
Thomas
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Thomas Preud'homme
> Sent: Tuesday, November 11, 2014 3:31 PM
> To: gcc-patches@gcc.gnu.org; Ramana Radhakrishnan; Richard Earnshaw
>
1 - 100 of 205 matches
Mail list logo
