> if you want, you could fix the ChangeLog entry in place, but don't add a
> new one for that change.
Done,
Dominique
Mike,
In http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48094#c21 you wrote
> Thanks for the report and the fix guys. I'd be fine with back port, if
> someone wants to develop and test it.
The patch is identical to r202593 and has been tested on x86_64-apple-darwin13.
Is it still OK?
Dominique
The patch is r202593 adjusted for 4.7 and has been tested on
x86_64-apple-darwin13.
OK?
Dominique
ChangeLog
CL_48094a
Description: Binary data
Patch
patch-48094b
Description: Binary data
On Fri, 4 Apr 2014, Jan Hubicka wrote:
> Hi,
> here is an updated version of my earlier ipa.c change. It turns out that the
> problem was that I did not drop always_inline.
> In this version I just drop always_inline attribute on all functions whose
> body
> is removed. The patch will affect no
2014-04-02 22:39 GMT+02:00 Jason Merrill :
> On 04/02/2014 04:21 PM, Fabien Chêne wrote:
>>
>> * cp/decl.c (duplicate_decls): Check for the return of
>> permerror before emitting a note.
>
>
> You don't need "cp/" within cp/ChangeLog. OK with that change.
I was a bit too optimistic, old
Hi!
On Mon, 31 Mar 2014 13:50:25 +0200 (CEST), Richard Biener
wrote:
> PR bootstrap/60719
> * Makefile.in (install-driver): Guard extra installs with special
> names properly.
>
> Index: gcc/Makefile.in
> ===
> **
On Thu, Apr 3, 2014 at 9:22 PM, Jeff Law wrote:
>
>
> As noted in the PR, there are a few insns in the ARM backend which use
> const_int_operand as a predicate, but which have constraints like "I" or
> "M".
>
> With the predicate accepting all constants, it's possible for a pass such as
> combine
Code generation without 'cfun' is tricky (and should be generally
avoided). The following patch fixes one case that clearly has
zero test coverage until now as the various uncovered issues show ;)
Bootstrap & regtest running on x86_64-unknown-linux-gnu, will apply
to trunk and the tree-ssanames.
On 04/04/2014 07:55 AM, Thomas Schwinge wrote:
Hi!
On Thu, 3 Apr 2014 18:13:08 +0200, Bernd Schmidt
wrote:
The patch below should be a better fix, making the references to
__OPENMP_TARGET__ weak. Does this work for you?
Yes, it does, thanks! Please revert my patch when committing yours.
On 03/21/2014 04:20 PM, Jakub Jelinek wrote:
On Fri, Mar 21, 2014 at 04:13:45PM +0100, Bernd Schmidt wrote:
On 03/20/2014 07:56 PM, Jakub Jelinek wrote:
When we were discussing the design last year, my strong preference was that
either this lives in some other crt object that mkoffload/linker p
On 03/20/2014 06:08 PM, Bernd Schmidt wrote:
This is based on Michael Zolotukhin's patch 3/3 from a while ago. It
enables lto-wrapper to build target images using the offload compilers
(identifying them through an env variable passed in by the gcc driver).
All the target-specific code is gone, ho
*ping*
--Alexander
2014-03-27 13:43 GMT+04:00 Alexander Ivchenko :
> Adding Balaji.
>
> --Alexander
>
> 2014-03-26 18:56 GMT+04:00 Alexander Ivchenko :
>> Hi,
>>
>> In gcc/config/linux-android.h we have "builtin_define ("__ANDROID__");"
>> So "ANDROID" as in libcilkrts now is not the correct macr
On Fri, Apr 4, 2014 at 12:03 PM, Alexander Ivchenko wrote:
> *ping*
I wonder whether this is consistend between compilers (note GCC is not
upstream here?). So eventually all places should be ANDROID || __ANDROID__?
Richard.
> --Alexander
>
> 2014-03-27 13:43 GMT+04:00 Alexander Ivchenko :
>> A
On Thu, Mar 27, 2014 at 11:25 AM, Ramana Radhakrishnan wrote:
> Hi,
>
>This is a partial fix for PR60655 where dwarf2out.c rejects NOT of a
> value in const_ok_for_output_1. There is still a problem with the testcase
> on armhf where we get operations of the form, const (minus (const_int)
> (s
Hi all,
Here is a patch, that fixes infinite backtraces in __cxa_end_cleanup().
The Bugzilla entry for this:http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60758
The __cxa_end_cleanup() does not save/restore LR in function header/footer and
does not provide any unwind info,
which causes problems wi
Respin of a long standing forgotten patch
(http://gcc.gnu.org/ml/gcc-patches/2006-12/msg01562.html).
This patch extends the mode-switching pass to toggle on/off a control
register status bit instead of setting the value. e.g on the SH4a to
support the FPCHG instruction used to switch FPU precision
On Fri, Apr 4, 2014 at 12:27 PM, Ramana Radhakrishnan
wrote:
> On Thu, Mar 27, 2014 at 11:25 AM, Ramana Radhakrishnan
> wrote:
>> Hi,
>>
>>This is a partial fix for PR60655 where dwarf2out.c rejects NOT of a
>> value in const_ok_for_output_1. There is still a problem with the testcase
>> on
2014-04-04 14:19 GMT+04:00 Richard Biener :
> On Fri, Apr 4, 2014 at 12:03 PM, Alexander Ivchenko
> wrote:
>> *ping*
>
> I wonder whether this is consistend between compilers (note GCC is not
> upstream here?). So eventually all places should be ANDROID || __ANDROID__?
>
I checked that gcc-4.[67
This fixes PR60750 by removing the premature (and bogus) optimization
of omitting VDEFs for noreturn calls (in this case the call is still
returning via EH and you also have to consider it returning abnormally).
Bootstrap & regtest running on x86_64-unknown-linux-gnu.
Richard.
2014-04-04 Richa
Hi,
Following the previous patchlet about conditional notes after
'permerror', same cleanup with 'warning'. Tested x86_64 linux without
regressions. OK to commit ?
cp/ChangeLog
2014-04-04 Fabien Chêne
* class.c (find_abi_tags_r): Check for the return of warning
before emitting a note
ping for maintainer.
Could this be considered for 4.8.3 please?
Thanks,
Daniel.
On Tue, Apr 1, 2014 at 2:46 PM, Daniel Gutson
wrote:
>
> I just realized I posted the patch in the wrong list.
>
>
> -- Forwarded message --
> From: Daniel Gutson
> Date: Tue, Apr 1, 2014 at 10
Hi all,
In PR 60743 it is noted that the genautomata computation has increased a lot in
both size and time due to my recently added a53 scheduling additions.
This patch attempts to mitigate that by reducing the large reservation duration
of the dividers that is causing a state-space explosion i
Sorry to be pedantic again, but 'wierd' should be spelt 'weird'. Otherwise,
looks good to me and much neater than before. (Seems you'd rather keep the
re-enabling, here and in the testsuite, for another patch?)
--Alan
Tejas Belagod wrote:
Richard Henderson wrote:
On 02/21/2014 08:30 AM, Tejas
On 03/18/14 07:16, Kai Tietz wrote:
Hi,
this patch skips anon2.C and anon3.C test for mingw target. Issue
here is that weak under pe-coff is different to ELF-targets and
therefore test doesn't apply for
FAIL: g++.dg/abi/anon2.C -std=c++11 scan-assembler
.weak(_definition)?[ \t]_?_ZN2N11D1C3fn
On 04/04/14 05:54, Richard Biener wrote:
This fixes PR60750 by removing the premature (and bogus) optimization
of omitting VDEFs for noreturn calls (in this case the call is still
returning via EH and you also have to consider it returning abnormally).
Bootstrap & regtest running on x86_64-unkn
On 04/03/14 11:02, Ramana Radhakrishnan wrote:
On Thu, Apr 3, 2014 at 2:27 PM, Charles Baylis
wrote:
Hi
This bug causes the compiler to create a Thumb-2 TBB instruction with
a jump table containing an out of range value in a .byte field:
whatever.s:148: Error: value of 256 too large for field
On Thu, Apr 3, 2014 at 1:55 PM, Michael Meissner
wrote:
> On Thu, Apr 03, 2014 at 01:24:25PM -0400, David Edelsohn wrote:
>> On Tue, Apr 1, 2014 at 7:55 PM, Michael Meissner
>> wrote:
>> > In backporting the power8 changes to the 4.8 branch, one of the testers of
>> > these patches noticed that l
Hi,
On Thu, Apr 03, 2014 at 11:19:10PM +0200, Jan Hubicka wrote:
> > > > +/* If E does not lead to a thunk, simply redirect it to N. Otherwise
> > > > create
> > > > + one or more equivalent thunks for N and redirect E to the first in
> > > > the
> > > > + chain. */
> > > > +
> > > > +void
OK.
Jason
>>
>> My first reaction is to wonder why this is this not a bug in the
>> "shorten" phase.
>
> I don't think that code ever expected an alignment directive to be emitted
> by ASM_OUTPUT_CASE_END :(
Fair point and it looks like this support came in when the support for
Thumb2 was added eons ago.
>
On 04/03/14 10:56, Marek Polacek wrote:
Under certain circumstances the sanitizer builtins are not initialized
properly and ubsan_instrument_return must make sure they are
initialized. Otherwise builtin_decl_explicit returns NULL and
we'll ICE in build_call_expr_loc_array. I'm not sure which ot
On 03/13/14 17:06, John David Anglin wrote:
This patch fixes PR rtl-optimization/60155.
The PA backend has a number of INSN patterns which trap on signed
overflow. These are
implemented as parallels using the trap_if code. Currently, single_set
does not consider
a parallel with a trap_if rtx t
Hi,
on the 4.8 branch, I'd like to fix the bug by simply never propagating
through thunks. There is very little pre-IPA devirtualization going
on and so this should have minimal impact.
Bootstrapped and tested on x86_64. OK for the branch?
Thanks,
Martin
2014-04-01 Martin Jambor
On 04/04/14 02:44, Ramana Radhakrishnan wrote:
On Thu, Apr 3, 2014 at 9:22 PM, Jeff Law wrote:
As noted in the PR, there are a few insns in the ARM backend which use
const_int_operand as a predicate, but which have constraints like "I" or
"M".
With the predicate accepting all constants, it's
Hi,
on the 4.7 branch just like on the newer one, I'd like to fix the bug
by simply never propagating through thunks. There even less pre-IPA
devirtualization going on and so this should have minimal impact.
Bootstrapped and tested on x86_64. OK for the branch?
Thanks,
Martin
2014-04-01 Ma
On 03/28/14 04:16, K_s, Vishnu wrote:
Hi all,
The tests added in gcc.dg/tree-ssa/isolate-*.c is failing for AVR target,
Because the isolate erroneous path pass needs -fdelete-null-pointer-checks
option to be enabled. For AVR target that option is disabled, this cause
the tests to fail. Following
Thanks to everyone who helped with development, testing, and review of
the patch set! I've committed the changes to 4.8 this morning. Note
that patch 15/26 was rejected as not really germane to this series and
has been submitted separately by Peter Bergner.
209087 1/26 diff-p8
209088 2/26
On Fri, Apr 4, 2014 at 1:35 PM, Kyrill Tkachov wrote:
> Hi all,
>
> In PR 60743 it is noted that the genautomata computation has increased a lot
> in both size and time due to my recently added a53 scheduling additions.
> This patch attempts to mitigate that by reducing the large reservation
> dur
On 04/01/14 03:40, Eric Botcazou wrote:
Hi,
the gnattools are now linked with the C++ compiler so, for the native case, it
has a dependency on libstdc++-v3. Adding such a dependency for a host tool on
a target library is debatable, but there is already a precedent with libada.
Tested on x86_64
On 03/26/14 22:18, Alan Modra wrote:
On Wed, Mar 26, 2014 at 09:43:08PM +, Maciej W. Rozycki wrote:
Alan,
On Tue, 17 Dec 2013, Alan Modra wrote:
On Tue, Dec 17, 2013 at 01:14:23PM +0100, Bernd Edlinger wrote:
the reason for this is overwriting GMPINC for the auto-build generation, becaus
On 04/04/14 09:17 -0600, Jeff Law wrote:
So what are those circumstances? ISTM this deserves some kind of
comment at the least.
I found the ICE using ubsan on the libstdc++ testsuite. Including the
bits/stdc++.h PCH'd header (i.e. the entire library) seemed to trigger
it. So either related to
On 03/28/14 19:26, Cong Hou wrote:
This patch is fixing PR60656. Elements in a vector with
vect_used_by_reduction property cannot be reordered if the use chain
with this property does not have the same operation.
Bootstrapped and tested on a x86-64 machine.
OK for trunk?
thanks,
Cong
diff -
Hi,
the attached patch modifies the builtin-bswap-6/7 testcases in order
to prevent GCC from using math instead of a compare. Only with a
compare the folding in combine actually takes place.
Whether the return value is produce with a compare or not depends
again on the value of branch cost.
Ide
Hi,
when combine deals with (xor (x) (const_int -1)) it turns it into a
(not (x)) first. This seems to be a disadvantage for targets not
providing (not ..) patterns since it prevents intermediate steps from
matching.
If noticed this for the builtin-bswap-8/9 testcases which are fixed
with the at
On 4/4/2014 11:27 AM, Jeff Law wrote:
On 03/13/14 17:06, John David Anglin wrote:
This patch fixes PR rtl-optimization/60155.
The PA backend has a number of INSN patterns which trap on signed
overflow. These are
implemented as parallels using the trap_if code. Currently, single_set
does not c
On 04/04/14 10:35, John David Anglin wrote:
On 4/4/2014 11:27 AM, Jeff Law wrote:
On 03/13/14 17:06, John David Anglin wrote:
This patch fixes PR rtl-optimization/60155.
The PA backend has a number of INSN patterns which trap on signed
overflow. These are
implemented as parallels using the tr
On 2014.03.31 at 14:03 +0100, Nathan Sidwell wrote:
> This patch fixes some unexpected behaviour of the Weffc++ and
> Wnon-virtual-dtor
> flags. The documentation for the latter says it's also enabled by Weffc++,
> but
> that's untrue.
>
> The current behaviour of Weffc++ is to warn about any
On 04/04/14 17:38, Markus Trippelsdorf wrote:
I'm not sure that this is a good idea. This changes the existing
behavior of -Wnon-virtual-dtor and causes hundreds of new warnings when
building LLVM. Wouldn't it make more sense to move the 3rd ed Weffc++
behavior to the -Weffc++ flag alone?
IIUC
richi asked for a testcase for 60731, and since we didn't already have
support for tests using dlopen, I had to add it. Does this approach
make sense?
commit c0a8675658fcf59ae8f08c7e4588c865346c
Author: Jason Merrill
Date: Fri Apr 4 06:15:02 2014 -0400
PR c++/60731
* lib/gcc-dg
On 2014.04.04 at 17:48 +0100, Nathan Sidwell wrote:
> On 04/04/14 17:38, Markus Trippelsdorf wrote:
>
> > I'm not sure that this is a good idea. This changes the existing
> > behavior of -Wnon-virtual-dtor and causes hundreds of new warnings when
> > building LLVM. Wouldn't it make more sense to m
On 04/04/14 17:54, Markus Trippelsdorf wrote:
markus@x4 tmp % g++ -Wnon-virtual-dtor -std=c++11 -c test.ii
test.ii: In instantiation of ‘class A<>’:
test.ii:12:16: required from here
test.ii:9:26: warning: base class ‘class opt_storage<0>’ has accessible
non-virtual destructor [-Wnon-virtual-
> Hi,
>
> on the 4.7 branch just like on the newer one, I'd like to fix the bug
> by simply never propagating through thunks. There even less pre-IPA
> devirtualization going on and so this should have minimal impact.
>
> Bootstrapped and tested on x86_64. OK for the branch?
OK for both branch
On Fri, Apr 04, 2014 at 04:48:48PM +0100, Jonathan Wakely wrote:
> On 04/04/14 09:17 -0600, Jeff Law wrote:
> >So what are those circumstances? ISTM this deserves some kind of
> >comment at the least.
>
> I found the ICE using ubsan on the libstdc++ testsuite. Including the
> bits/stdc++.h PCH'd
On 04/04/2014 01:04 PM, Nathan Sidwell wrote:
On 04/04/14 17:54, Markus Trippelsdorf wrote:
markus@x4 tmp % g++ -Wnon-virtual-dtor -std=c++11 -c test.ii
test.ii: In instantiation of ‘class A<>’:
test.ii:12:16: required from here
test.ii:9:26: warning: base class ‘class opt_storage<0>’ has
acc
On 2014.04.04 at 18:04 +0100, Nathan Sidwell wrote:
> On 04/04/14 17:54, Markus Trippelsdorf wrote:
>
> > markus@x4 tmp % g++ -Wnon-virtual-dtor -std=c++11 -c test.ii
> > test.ii: In instantiation of ‘class A<>’:
> > test.ii:12:16: required from here
> > test.ii:9:26: warning: base class ‘class
On 04/04/14 18:19, Jason Merrill wrote:
I also notice that the opt_storage destructor is implicitly declared and
trivial; it seems excessive to warn in that case.
I disagree, I think that's exactly a case that should be warned about -- because
it's easy to overlook.
I'm also somewhat skep
On 04/04/14 10:18, Andreas Krebbel wrote:
Hi,
the attached patch modifies the builtin-bswap-6/7 testcases in order
to prevent GCC from using math instead of a compare. Only with a
compare the folding in combine actually takes place.
Whether the return value is produce with a compare or not dep
On 04/04/14 11:14, Marek Polacek wrote:
So what happens here is that normally we initialize the builtins via
c_common_nodes_and_builtins, which has:
5711 if (!flag_preprocess_only)
5712 c_define_builtins (va_list_ref_type_node, va_list_arg_type_node);
c_define_builtins defines builti
Hi,
next victim of the cleanup: warning_at.
Tested x86_64 linux without regressions. OK to commit ?
cp/ChangeLog
2014-04-04 Fabien Chêne
* decl.c (duplicate_decls): Check for the return of warning_at
before emitting a note.
(warn_misplaced_attr_for_class_type): Likewise.
(che
Hi,
this regression is an ICE happening during error recovery: the loop in
sort_constexpr_mem_initializers tries to access the vector beyond its
end. Robustifying the whole thing, using in particular a standard
pattern using vec_safe_iterate to go through the vector, works. In case,
I'm not s
OK.
Jason
OK.
Jason
On Fri, Apr 04, 2014 at 11:50:04AM -0600, Jeff Law wrote:
> On 04/04/14 11:14, Marek Polacek wrote:
> >
> >So what happens here is that normally we initialize the builtins via
> >c_common_nodes_and_builtins, which has:
> >
> > 5711 if (!flag_preprocess_only)
> > 5712 c_define_builtins (va_l
This fixes a simple typo in my patch from a couple of days ago. I'm
checking it in.
2014-04-04 Anthony Green
* config/moxie/moxie.md (zero_extendqisi2, zero_extendhisi2): Fix
typos.
PR ipa/59626
Index: gcc/config/moxie/moxie.md
=
> On Fri, 4 Apr 2014, Jan Hubicka wrote:
>
> > Hi,
> > here is an updated version of my earlier ipa.c change. It turns out that
> > the
> > problem was that I did not drop always_inline.
> > In this version I just drop always_inline attribute on all functions whose
> > body
> > is removed. The
Hello,
Thomas Schwinge, le Wed 26 Jun 2013 23:30:03 +0200, a écrit :
> On Sat, 22 Jun 2013 08:15:46 -0700, Ian Lance Taylor wrote:
> > Go can work without split stack. In that case libgo will use much
> > larger stacks for goroutines, to reduce the chance of running out of
> > stack space (see S
> The third testcase uses anonymous namespaces and ipa-devirt correctly
> reports that its list of possible targets is final, but even though
> the list has only two items and one of them is __cxa_pure_virtual, it
> still only devirtualizes speculatively.
Ah, yes, it is what I was discussing this
On 04/04/2014 04:05 PM, Jan Hubicka wrote:
Ah, yes, it is what I was discussing this with Jason, but apparently the
discussion died out. According to his comment __cxa_pure_virtual is a
synonym for undefined behaviour so it may be correct to unconditionally
devirtualize here (changing runtime be
On Thu, 2014-03-20 at 20:38 -0500, Bill Schmidt wrote:
> The original workaround for vector permute on a little endian platform
> includes subtracting each element of the permute control vector from 31.
> Because the upper 3 bits of each element are unimportant, this was
> implemented as subtractin
> On 04/04/2014 04:05 PM, Jan Hubicka wrote:
> >Ah, yes, it is what I was discussing this with Jason, but apparently the
> >discussion died out. According to his comment __cxa_pure_virtual is a
> >synonym for undefined behaviour so it may be correct to unconditionally
> >devirtualize here (changin
On 04/04/14 13:45, Marek Polacek wrote:
I'm sorry, my guess was wrong, it's not about flag_preprocess_only at
all. But still it's about PCH ;). In short: we define the builtins,
but PCH then removes the definitions.
Ah.
The thing is, we define the
builtins just fine (via c_common_nodes_and_
Applied, thanks.
Jason
Applied, thanks.
Jason
We also enabled ZCX in the Ada compiler for the ARM port of VxWorks (but,
unlike the Linux port, it works because it uses DWARF2 EH instead of EHABI)
and this made the ACATS stack checking test cb1010c fail at -O2, because the
scheduler moves an update of the stack pointer ahead of a stack probe
On 04/04/2014 01:45 PM, Bill Schmidt wrote:
> Per Richard Henderson's previous comment, I have changed the
> patch slightly to avoid the use of emit_move_insn.
Thanks.
r~
This patch ensures that the finalization expression is generated and
that use-associated finalizers are properly accessed.
Build and regtested on x86-64-gnu-linux.
OK for the trunk?
Tobias
2014-04-04 Tobias Burnus
PR fortran/58880
PR fortran/60495
* resolve.c (gfc_resolve_finalizers): En
Committed as obvious, Rev. 209133
Tobias
Index: ChangeLog
===
--- ChangeLog (Revision 209132)
+++ ChangeLog (Arbeitskopie)
@@ -1,3 +1,7 @@
+2014-04-04 Tobias Burnus
+
+ * check.c (gfc_check_cmplx): Fix typo.
+
2014-03-28 Mikael
Hi All,
This is a request to apply the attached patch to GCC project. I am adding
support for TI-RTOS
to newlib and require the attached change in configure.ac file.
Thanks,
Ashish
Change log:
From: Ashish Kapania
Date: Fri, 4 Apr 2014 16:34:20 -0700
Subject: [PATCH] configure.ac: Add support
Forgot to attach the patch file :)
-Original Message-
From: Kapania, Ashish
Sent: Friday, April 04, 2014 4:50 PM
To: 'gcc-patches@gcc.gnu.org'
Subject: [PATCH] (configure.ac) Add support for TI-RTOS
Hi All,
This is a request to apply the attached patch to GCC project. I am adding
suppo
79 matches
Mail list logo
