On Wed, Jul 31, 2013 at 10:16 PM, Trevor Saunders wrote:
> On Wed, Jul 31, 2013 at 10:02:29PM -0500, Gabriel Dos Reis wrote:
>> On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders
>> wrote:
>> > bootstrapped and same test results on x86_64-linux-gnu against r201084
>> >
>> > gcc/cp/
>> > *
On Wed, Jul 31, 2013 at 10:02:29PM -0500, Gabriel Dos Reis wrote:
> On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders
> wrote:
> > bootstrapped and same test results on x86_64-linux-gnu against r201084
> >
> > gcc/cp/
> > * cxx-pretty-print.h (cxx_pretty_printer): inherit from
> > c_prett
On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders wrote:
> Hi!
>
> its my first time sending patches here so please be gentle :)
it is very much appreciated. Thanks for tackling it.
-- Gaby
On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders wrote:
> bootstrapped and same test results on x86_64-linux-gnu against r201084
>
> gcc/c-family/
> * c-pretty-print.h (PP_C_BASE): remove macro.
> * c-pretty-print.c adjust accordingly.
>
> gcc/cp/
> * cxx-pretty-pri
On Wed, Jul 31, 2013 at 8:14 PM, Trevor Saunders wrote:
> bootstrapped and same test results on x86_64-linux-gnu against r201084
>
> gcc/cp/
> * cxx-pretty-print.h (cxx_pretty_printer): inherit from
> c_pretty_printer
> instead of include it.
> * (cxx_pretty_print.c): adju
Thank you all for the comments. The patch is updated as:
1) Revert it to the original one.
2) For the testcase, replace the dg-options with
/* { dg-do compile } */
/* { dg-require-effective-target arm_neon } */
/* { dg-add-options arm_neon } */
/* { dg-options "-O3" } */
Bootstrap on Chromebook an
bootstrapped and same test results on x86_64-linux-gnu against r201084
gcc/c-family/
* c-pretty-print.h (PP_C_BASE): remove macro.
* c-pretty-print.c adjust accordingly.
gcc/cp/
* cxx-pretty-print.c adjust accordingly.
* cxx-pretty-print.h adjust accordingl
bootstrapped and same test results on x86_64-linux-gnu against r201084
gcc/cp/
* cxx-pretty-print.h (cxx_pretty_printer): inherit from c_pretty_printer
instead of include it.
* (cxx_pretty_print.c): adjust accordingly.
---
gcc/cp/cxx-pretty-print.c | 40 ++
Hi!
its my first time sending patches here so please be gentle :)
I'm working on further patches to use c++ inheritance instead of macros here,
but I figured I might as well send these out now.
Trev
I have a new patch that supersedes this. The new patch also fixes PR
tree-optimization/57393 and PR tree-optimization/58011. Bootstraps and
no test regression on x86_64/linux. Ok for trunk?
2013-07-31 Easwaran Raman
PR middle-end/57370
* tree-ssa-reassoc.c (build_and_add_sum): Fix UID
On Wed, Jul 31, 2013 at 8:18 PM, Paolo Carlini wrote:
> in order to bootstrap successfully the patch I needed these additional
> changes. Please double check at your end, run the testsuite, and if
> everything goes well, please commit (again ;) the whole thing.
Fully tested:
cd gcc-bulid
rm -r *
On 07/31/2013 09:58 PM, François Dumont wrote:
Here is another proposal using std::common_type to find out how to
instantiate std::less and std::greater_equal. Much easier with C++11
features indeed.
Tested under linux x86_64 debug mode.
Ok to commit ?
Ok, thanks!
Paolo.
Honza,
After this patch, I see new testsuite failures on PowerPC related to profiling.
FAIL: gcc.dg/tree-prof/ic-misattribution-1.c scan-ipa-dump profile
"hist->count 1 hist->all 1"
FAIL: gcc.dg/tree-prof/indir-call-prof.c scan-ipa-dump profile
"Indirect call -> direct call.* a1 transformation on
Jonathan Wakely writes:
> On 31 July 2013 20:44, Matthias Klose wrote:
>> if you mention distribution specific packages, please add the ones needed for
>> some distributions. For Debian/Ubuntu this would be g++-multilib if the
>> architecture is multilib'ed, g++ otherwise.
> That's not the packa
On 07/31/2013 07:08 AM, DJ Delorie wrote:
>>> The first doesn't have a clobber, though...
>>
>> Hmm. Missed that. So the only reason you'd care to include the bic
>> in the regular and pattern is if it were a smaller encoding than the
>> regular r/0/i alternative.
>
> It is, but the separate pat
On 31 July 2013 20:44, Matthias Klose wrote:
> if you mention distribution specific packages, please add the ones needed for
> some distributions. For Debian/Ubuntu this would be g++-multilib if the
> architecture is multilib'ed, g++ otherwise.
That's not the package that provides gnu/stubs-32.h,
Hi Fabien,
Can you tell me what the status of the following patch that
removes the pow() overload from tr1 is? Specifically:
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01166.html
It seemed to have been approved and you were going to do some
additional testing before committing the patch l
Here is another proposal using std::common_type to find out how to
instantiate std::less and std::greater_equal. Much easier with C++11
features indeed.
Tested under linux x86_64 debug mode.
Ok to commit ?
François
On 07/25/2013 12:10 AM, Paolo Carlini wrote:
On 07/24/2013 10:41 PM, Fran
Am 29.07.2013 15:06, schrieb FX:
>> As a consensual first step toward addressing this issue, I suggest the
>> following patch to the doc. I hope it is clear enough, but suggestions are
>> obviously welcome. (I haven't even compiled the docs with it, as I'm on my
>> laptop with little battery.)
>
On Wed, Jul 31, 2013 at 3:04 PM, Brooks Moses wrote:
> Ok to commit to google/gcc-integration and then merge to
> google/gcc-main and google/gcc-4_8?
OK.
Diego.
On Wed, Jul 31, 2013 at 8:58 AM, Matthias Klose wrote:
>
> there are now a lot of empty directories. please remove.
Done on both mainline and 4.8 branch. Sorry about that.
Ian
Diego -
The attached patch corrects the fact that we were not picking up the
GRTE-specific spec files in our non-x86 GCC builds, and incidentially
simplifies the logic for these spec files since they do not appear to
be changing from one GRTE version to another.
* gcc/configs/linux-grtev2.h: Remo
On 07/31/2013 01:33 PM, Marek Polacek wrote:
There are still at least two issues though, which is why
bootstrap with -fsanitize=undefined fails:
http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01480.html
This looks like a serious bug, properly caught by -Wuninitialized.
When sanitizing,
in .uni
OK.
Jason
On Jul 30, 2013, at 8:24 PM, DJ Delorie wrote:
> [nickc added for comments about the bits he wrote]
>
>> ... define these as
>>
>> (define_predicate "msp_general_operand"
>> (match_code "mem,reg,subreg,const_int,const,symbol_ref,label_ref"
>> {
>> int save_volatile_ok = volatile_ok;
>> volati
On Thu, Jul 25, 2013 at 04:13:29PM -0400, Jason Merrill wrote:
> Otherwise, looks fine. If nobody else has comments, go ahead and
> check it in next week.
I had to make a few changes since, particularly:
- [ubsan] Add -static-libubsan
http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01467.html
Another round of documentation and naming (not for coding style, but
for clearer semantics) related comments:
David
>
> namespace autofdo {
>
> /* Represent a source location: (function_decl, lineno). */
> typedef std::pair decl_lineno;
> /* Represent an inline stack. vector[0] is the leaf node
> > The first doesn't have a clobber, though...
>
> Hmm. Missed that. So the only reason you'd care to include the bic
> in the regular and pattern is if it were a smaller encoding than the
> regular r/0/i alternative.
It is, but the separate pattern picks that out just fine.
> You need to im
Hi,
Here is the patch, Tested by profiledbootstrap. Ok for google gcc-4.8?
thanks, Dinar.
On Wed, Jul 31, 2013 at 12:01 AM, Rong Xu wrote:
> Will do.
>
> The patch was in gcc-4_7 by Dehao.
>
> r194713 | dehao | 2012-12-24 16:49:06 -0800 (Mon, 24 Dec 2012) | 5 lines
>
> Fix the pro
Missing ubsan tests in g++.dg/dg.exp were causing mischief. So add it.
Tested x86_64-pc-linux-gnu, applying to ubsan branch.
diff --git a/gcc/testsuite/ChangeLog.ubsan b/gcc/testsuite/ChangeLog.ubsan
index f3f18f9..02a6ca2 100644
--- a/gcc/testsuite/ChangeLog.ubsan
+++ b/gcc/testsuite/ChangeLog.
On Wed, 2013-07-31 12:18:42 -0400, David Malcolm wrote:
> On Wed, 2013-07-31 at 18:00 +0200, Jan-Benedict Glaw wrote:
> > That's why I'm running the build robot :) Once I've got some
> > time[tm], I'll spend it a small web frontend to look into the basic
> > information (which builds worked/faile
On 07/30/2013 05:24 PM, DJ Delorie wrote:
> Fixed. Perhaps gen* could error on those? I know they're ignored, I
> just keep forgetting that splits are special.
>
> (alternately, why can't they be included in the matching logic?)
(Because pre-reload, pseudos don't have register classes, mostly)
Hi!
We've seen ICEs while outputting an operand (not even the excessive
CISC of a VAX could do that), which should be fixed by this patch:
2013-07-31 Jan-Benedict Glaw
* config/vax/constraints.md (T): Add missing CONSTANT_P check.
diff --git a/gcc/config/vax/constraints.md b/gcc/conf
On 07/31/2013 05:02 AM, Kirill Yukhin wrote:
>
> There's ICE (max. number of generated reload insns per insn is achieved (90)),
> when LRA tried to save mask register before call. This was caused by fact
> that split_reg function
> in lra-constraints.c allocates memory for saved reg in
> SECOND
On Wed, 2013-07-31 at 18:00 +0200, Jan-Benedict Glaw wrote:
> On Wed, 2013-07-31 11:31:35 -0400, David Malcolm wrote:
> > On Wed, 2013-07-31 at 11:36 +0200, Jan-Benedict Glaw wrote:
> > > On Wed, 2013-07-31 10:34:10 +0200, Jan-Benedict Glaw
> > > wrote:
> [breakage with mawk]
> > > > It seems th
On Mon, Jul 29, 2013 at 2:39 PM, Michael Meissner
wrote:
> This is the revised version of my patch #8 for power8 support. I have removed
> all of the incidental changes, and only added the support for load fusion. I
> have added support for fusion on 32-bit Linux. I have added a test to make
>
On Wed, 2013-07-31 11:31:35 -0400, David Malcolm wrote:
> On Wed, 2013-07-31 at 11:36 +0200, Jan-Benedict Glaw wrote:
> > On Wed, 2013-07-31 10:34:10 +0200, Jan-Benedict Glaw
> > wrote:
[breakage with mawk]
> > > It seems this does only happen on one of the three running build
> > > clients. Tha
Am 16.07.2013 08:55, schrieb Ian Lance Taylor:
> I have committed a large patch to update libgo to the library that was
> part of the Go 1.1.1 release. As usual, I'm not including the entire
> patch in this e-mail message, because it is too large. I'm only
> including the changes to the files tha
On Wed, 2013-07-31 at 11:36 +0200, Jan-Benedict Glaw wrote:
> On Wed, 2013-07-31 10:34:10 +0200, Jan-Benedict Glaw
> wrote:
> > On Tue, 2013-07-30 20:50:27 -0400, David Malcolm
> > wrote:
> > [...]
> > > Committed to trunk as r201359, having double-checked that it
> > > bootstrapped by itself o
Hello,
In this patch we add support for new mask register k0-k7. Changes are mostly
strightforward, but there are two problems. First we can't use k0 as mask in
vector instructions, so we have introduce two register classes. One for use in
vector instruction with "k" constraint - corresponding to k
Hi,
This patch is part of the ongoing work to unify instruction classification
between the ARM and AARCH64 backends.
This patch wires up the cortex-a53 pipeline description defined in the ARM
backend to be used in the AARCH46 backend.
OK for trunk?
Thanks
Sofiane
-
ChangeLog
* co
Hi,
if you don't have anything better to do, plenty of details in the audit
trail. Tested x86_64-linux, committed mainline and 4_8-branch.
Thanks,
Paolo.
2013-07-31 Paolo Carlini
PR libstdc++/56627
* include/bits/stl_bvector.h: Use friend struct hash i
On 31/07/13 14:48, Sofiane Naci wrote:
Hi,
This patch is part of the ongoing work of ARM/AARCH64 instruction
classification cleanup.
This patches moves the scheduling dependency routines from arm.c into a new
file aarch-common.c, preparing them to be used in AARCH64.
OK for trunk?
Thanks
Sofi
On 31/07/13 14:47, Sofiane Naci wrote:
Hi,
This patch is part of the ongoing work of ARM/AARCH64 instruction
classification cleanup.
This patches moves the "type" attribute, and the related "mul32" and "mul64"
from arm.md to a new file types.md, preparing it to be included in AARCH64.
OK for t
On 31/07/13 14:31, Kyrylo Tkachov wrote:
Hi all,
Resurrecting this thread...
On 01/07/13 12:05, Kyrylo Tkachov wrote:
Hi Richard,
This hurts code size.
Therefore I've disabled the new peephole2 for
optimize_insn_for_size_p
so that
the original peephole before r200197 is used when optimisi
Hi,
This patch is part of the ongoing work of ARM/AARCH64 instruction
classification cleanup.
This patches moves the scheduling dependency routines from arm.c into a new
file aarch-common.c, preparing them to be used in AARCH64.
OK for trunk?
Thanks
Sofiane
-
ChangeLog:
* config.
Hi,
This patch is part of the ongoing work of ARM/AARCH64 instruction
classification cleanup.
This patches moves the "type" attribute, and the related "mul32" and "mul64"
from arm.md to a new file types.md, preparing it to be included in AARCH64.
OK for trunk?
Thanks
Sofiane
-
ChangeLog:
Hi DJ, Hi Richard,
+;; The next two patterns are here to support a "feature" of how GCC implements
+;; varargs. When a function uses varargs and the *second* to last named
+;; argument is split between argument registers and the stack, gcc expects the
+;; callee to allocate space on the stack t
Hi all,
Resurrecting this thread...
> On 01/07/13 12:05, Kyrylo Tkachov wrote:
> > Hi Richard,
> >
> >>> This hurts code size.
> >>> Therefore I've disabled the new peephole2 for
> optimize_insn_for_size_p
> >> so that
> >>> the original peephole before r200197 is used when optimising for
> size.
This is now committed.
Please double check me, Sebastian.
--joel
On 7/31/2013 2:02 AM, Sebastian Huber wrote:
Ping.
On 2013-07-08 15:29, Sebastian Huber wrote:
The __cxa_atexit support is a reqirement for destructor registration of
thread-local objects.
For *-*-elf it is already enabled by
On Thu, Jul 25, 2013 at 10:43:30PM +, Joseph S. Myers wrote:
> On Thu, 25 Jul 2013, Marek Polacek wrote:
>
> > +@item -fsanitize=undefined
> > +Enable UndefinedBehaviorSanitizer, a fast undefined behavior detector
> > +Various computations will be instrumented to detect
> > +undefined behavior
Hi,
On 07/31/2013 05:43 AM, Tim Shen wrote:
I found some other wired problems in that patch after committing.
Probably need more time to work on it, so now I revert it :(
in order to bootstrap successfully the patch I needed these additional
changes. Please double check at your end, run the tes
This patch hopefully improves the documentation of the option in
question.
Applying to ubsan branch.
diff --git a/gcc/ChangeLog.ubsan b/gcc/ChangeLog.ubsan
index ac584ff..d7932c5 100644
--- a/gcc/ChangeLog.ubsan
+++ b/gcc/ChangeLog.ubsan
@@ -1,3 +1,7 @@
+2013-07-31 Marek Polacek
+
+ * do
On 07/30/2013 09:44 AM, Andrew MacLeod wrote:
On 07/30/2013 09:16 AM, Joseph S. Myers wrot
THis also means that for the 3 floating point operations all we need
are RTL
insn patterns, no buitin. And as with the other atomics, if the
pattern
I think something will also be needed to specify all
On 7/25/13 12:41 AM, Chung-Ju Wu wrote:
> On 7/10/13 8:04 AM, Joseph S. Myers wrote:
>> There are lots of files in this patch with no copyright or license
>> notices. GNU policy is that every file of more than ten lines needs
>> copyright and license notices (with the libgcc license exception in t
On Thu, Jul 25, 2013 at 10:40:22PM +, Joseph S. Myers wrote:
> What happens if you bootstrap with this enabled - do whatever failures
> appear look like genuine bugs? Running the testsuite with a compiler
> built with this option? Running the testsuite with this option used when
> compilin
On 30/07/13 14:28, Ian Bolton wrote:
Our movdi_aarch64 pattern allows moving a constant into an FP_REG,
but has the constraint Dd, which is stricter than the one for
moving a constant into a CORE_REG. This is due to restricted values
allowed for MOVI instructions.
Due to the predicate for the p
On 07/30/2013 09:36 PM, Jason Merrill wrote:
OK.
Thanks, installed.
Today I had a look to the error recovery in cp_parser_sizeof_pack and
noticed a few cases in which we could do better. A very simple issue is
the following: when cp_parser_identifier returns error_mark_node we
don't notice a
On Wed, 2013-07-31 10:34:10 +0200, Jan-Benedict Glaw wrote:
> On Tue, 2013-07-30 20:50:27 -0400, David Malcolm wrote:
> [...]
> > Committed to trunk as r201359, having double-checked that it
> > bootstrapped by itself on top of what had gone before, and that the
> > testsuite results were unaffec
On Wed, 2013-07-31 10:34:10 +0200, Jan-Benedict Glaw wrote:
> On Tue, 2013-07-30 20:50:27 -0400, David Malcolm wrote:
> [...]
> > Committed to trunk as r201359, having double-checked that it
> > bootstrapped by itself on top of what had gone before, and that the
> > testsuite results were unaffec
On Tue, 2013-07-30 20:50:27 -0400, David Malcolm wrote:
[...]
> Committed to trunk as r201359, having double-checked that it
> bootstrapped by itself on top of what had gone before, and that the
> testsuite results were unaffected by it (on x86_64-unknown-linux-gnu).
My build robot chokes on it.
Ping.
On 2013-07-08 15:29, Sebastian Huber wrote:
The __cxa_atexit support is a reqirement for destructor registration of
thread-local objects.
For *-*-elf it is already enabled by default. See comment line 810 in
"gcc/config.gcc".
Define TARGET_LIBGCC_SDATA_SECTION on PowerPC for RTEMS to ".
62 matches
Mail list logo
