Hello!
Attached patch synchronizes i386.o dependecies with i386.c.
2013-09-16 Uros Bizjak
* config/i386/t-i386 (i386.o): Update dependecies.
Bootstrap on x86_64-pc-linux-gnu is in progress.
Uros.
Index: config/i386/t-i386
=
Hi!
This patch fixes two issues I found on the pr58392.c testcase:
1) we weren't copying decl attributes, so e.g. inside #pragma omp parallel
"omp simd array" temporary arrays lost their attribute and weren't
adjusted because of that
2) DR_ALIGNED_TO wasn't reset after resetting DR_OFFSET on simd
I noticed there are a couple of dump_printf_loc instances in
coverage.c not ended with newline. They should be fixed.
David
On Tue, Sep 10, 2013 at 6:32 AM, Teresa Johnson wrote:
> On Mon, Sep 9, 2013 at 9:55 PM, Xinliang David Li wrote:
>> looks fine to me.
>>
>> In the long run, I wonder if t
>> Just notice another problem here:
>> processor_type only contains PROCESSOR_COREI7, so I cannot
>> differentiate Westmere and Sandybridge in x86-tune.def, which are
>> different for TARGET_FUSE_ALU_AND_BRANCH. So do I have to separate
>> m_SANDYBRIDGE out from m_COREI7?
>
> Yes, please.
>
> Than
On Mon, Sep 16, 2013 at 10:57 AM, Xinliang David Li wrote:
> I noticed there are a couple of dump_printf_loc instances in
> coverage.c not ended with newline. They should be fixed.
I committed this change this morning as r202628. I believe I fixed all
the dump_printf_loc calls (just double-checke
Il 16/09/2013 19:23, Tom Tromey ha scritto:
> Tom> I can try a test build using a setting for CC that rejects -c -o.
>
> I did this and the build died pretty early on:
>
> /home/tromey/Space/Trunk/Git/bin/my-cc -c -DHAVE_CONFIG_H -g -g -I.
> -I../../gcc/libiberty/../include -W -Wall -Wwrite-s
New version that passed testing.
2013-09-16 Marc Glisse
PR libstdc++/58338
* include/bits/stl_vector.h (vector::vector(),
vector::vector(const allocator_type&)): Merge.
(_Vector_impl::_Vector_impl(_Tp_alloc_type const&),
_Vector_impl::_Vector_impl(_Tp_a
The description of the macro REVERSE_CONDEXEC_PREDICATES_P has been
removed from (gccint)Cond Exec Macros in r188983, but the now
effectively empty node has been left behind.
Tested with make info and installed as obvious.
Andreas.
* doc/tm.texi.in (Cond Exec Macros): Remove node.
Looks like there is one missing spot:
@@ -349,7 +349,7 @@ get_coverage_counts (unsigned counter, u
(flag_guess_branch_prob
? "file %s not found, execution counts
estimated" <
: "file %s not foun
On Fri, Sep 13, 2013 at 08:01:36PM +0200, Marek Polacek wrote:
> 2013-09-13 Marek Polacek
>
> PR sanitizer/58413
> c-family/
> * c-ubsan.c (ubsan_instrument_shift): Don't instrument
> an expression if we can prove it is correct.
>
> testsuite/
> * c-c++-common/ubsan/shi
> On Sun, Sep 15, 2013 at 09:08:00PM +0200, Jan Hubicka wrote:
> > > 2013-09-13 Jakub Jelinek
> > >
> > > * ipa-prop.c (ipa_compute_jump_functions_for_edge): Return early
> > > for internal calls.
> >
> > That seems resonable. I wonder if we want to consider internal calls to
> > form
>
> "Uros" == Uros Bizjak writes:
Uros> Attached patch decleres dependecies for config/alpha/alpha.o.
Uros> 2013-09-16 Uros Bizjak
Uros> * config/alpha/t-alpha: New.
Uros> * config.gcc (alpha*-*-linux*): Add alpha/t-alpha to tmake_file.
Uros> (alpha*-*-freebsd*): Ditto.
Uros>
I incorporated all the comments and committed the change (also fixed a
test failure with --help=optimizers).
thanks,
David
On Mon, Sep 16, 2013 at 3:07 AM, Richard Biener
wrote:
> On Fri, Sep 13, 2013 at 6:56 PM, Xinliang David Li wrote:
>> Updated patch implementing the logic that more specif
On Sep 16, 2013, at 5:10 AM, Nick Clifton wrote:
> The new MSP430 target has a few limitations which show up as
> unexpected failures in the GCC testsuite. I would like to apply the
> patch below to update target-supports.exp with this information.
> Applying the patch results in a drop of 13
There were two related issues with the _vtab of CLASS(*): First, it was
crashing in the generation of the finalization wrapper. Secondly, one
needs to ensure that the _vtab contains a NULL pointer for _final.
Build and regtested on x86-64-gnu-linux.
OK for the trunk?
Tobias
I forgot, again, to attach the patch.
However, to my surprise the email made it through. Contrary to Janus'
second patch review of today which didn't went past GCC's mail server.
(The same problem I had yesterday: Three emails didn't make it.)
Tobias Burnus wrote:
There were two related issue
Yep, looked too quickly every time and thought the newline after "be
zero" was applying. Here is the patch with the fix. Ok for trunk
pending regression testing?
2013-09-16 Teresa Johnson
* coverage.c (get_coverage_counts): Add missing newline.
Index: coverage.c
==
On Mon, Sep 16, 2013 at 3:13 AM, Richard Biener
wrote:
> On Fri, Sep 13, 2013 at 5:16 PM, Xinliang David Li wrote:
>> On Fri, Sep 13, 2013 at 1:30 AM, Richard Biener
>> wrote:
>>> On Thu, Sep 12, 2013 at 10:31 PM, Xinliang David Li
>>> wrote:
Currently -ftree-vectorize turns on both loop
> Contrary to Janus' second
> patch review of today which didn't went past GCC's mail server.
Huh, indeed. I hadn't noticed ...
> (The same
> problem I had yesterday: Three emails didn't make it.)
Any idea what's the reason for this?
Cheers,
Janus
On Mon, Sep 16, 2013 at 4:50 PM, Michael Matz wrote:
>> > Why do you think so? The t-* frags are includes via
>> > ...
>> > tmake_file=...$(srcdir)/config/i386/t-i386
>> > ...
>> > ifneq ($(tmake_file),)
>> > include $(tmake_file)
>> > endif
>> > ...
>> >
>> > It's just that there's no t-alpha f
This patch adds support for the RL78/G10 variant, which doesn't have
register banks like the other RL78 chips. Committed.
* config/rl78/rl78.c (rl78_asm_file_start): Specify alternate
vregs location for RL78/G10.
(rl78_expand_prologue): Avoid SEL on G10.
(rl78_exp
Just for completeness. Installed.
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index f606965..7150063 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2013-09-16 Jeff Law
+
+ * gcc.c-torture/execute/pr58419.c: New test.
+ * gc
On 09/16/2013 05:28 AM, Richard Biener wrote:
which exhibits exactly the behavior you quote - return x is considered throwing
an exception. The C++ FE doesn't arrange for TREE_THIS_NOTRAP to be
set here (maybe due to this issue you quote?).
I haven't been aware of TREE_THIS_NOTRAP, but we coul
Janus Weil wrote:
(The same problem I had yesterday: Three emails didn't make it.)
Any idea what's the reason for this?
No idea. Usually, one gets a bounce message but this time one didn't.
Thus, I have no possibility to debug this from my side. #overseers
didn't see it as their business and
Ok -- abandon the patch.
On Mon, Sep 16, 2013 at 2:04 AM, Richard Biener
wrote:
> On Sat, Sep 14, 2013 at 8:10 AM, Xinliang David Li wrote:
>> tree if conversion is an enabler pass for vectorization, so by
>> default, it is only turned on when vectorization is on, but may also
>> depend on the o
Paolo> The series looks okay to me with that change.
Two last questions while I'm testing my rebase --
First, do you mind if I resend the whole series? Otherwise I can try to
pick out just the patches that have changed, plus the additional
patches.
Second, should I get a separate ok for the cha
Forgot this bit, committed.
* config/rl78/vregs.h: Add G10 register definitions.
Index: config/rl78/vregs.h
===
--- config/rl78/vregs.h (revision 202637)
+++ config/rl78/vregs.h (working copy)
@@ -8,12 +8,35 @@ r_2 =
Nobody has expressed concern for a week, so it may be worth doing an
official review ;-)
http://gcc.gnu.org/ml/gcc-patches/2013-09/msg00676.html
On Mon, 9 Sep 2013, Marc Glisse wrote:
I have now tested bootstrap+testsuite and there was no regression.
2013-09-07 Marc Glisse
PR c++
On Sep 16, 2013, at 2:09 AM, Richard Biener wrote:
>> Here is the patch to add precision to partial int modes. I included
>> switching over the rs6000 port to the new scheme so one can get a feel for
>> that it would look like.
> Can you instead of adding PARTIAL_INT_MODE_N change all existing
Hi,
On Sun, Sep 15, 2013 at 06:55:17PM +0200, Bernd Edlinger wrote:
> Hello Richard,
>
> attached is my second attempt at fixing PR 57748. This time the
> movmisalign path is completely removed and a similar bug in the read
> handling of misaligned structures with a non-BLKmode is fixed
> too. Th
On Thu, Sep 12, 2013 at 09:33:05AM +0100, Andrew Haley wrote:
> On 09/12/2013 03:11 AM, Alan Modra wrote:
> > We have precedent for compiling libffi based on gcc preprocessor
> > defines, eg. __NO_FPRS__, so here's a way of making upstream libffi
> > compatible with the various versions of gcc out
There are two separate root causes for the problem reported in PR
57393. This patch attempts to fix both.
First is due to newly created stmts that have the default UID of 0
which are compared with statements with valid UIDs leading to broken
dependences. As discussed in an earlier thread, I check
On 09/16/2013 07:32 PM, Marc Glisse wrote:
New version that passed testing.
Looks good to me, thanks!
Paolo.
On 16.09.2013 17:54, Eric Botcazou wrote:
Apologies if this was out of context. This was a change suggested
and
reviewed by Jason
(http://gcc.gnu.org/ml/gcc-patches/2013-09/msg01114.html).
Then don't repost it on gcc-patches, it is already available on
gcc-cvs:
http://gcc.gnu.org/ml/gcc-c
Hi,
for this kind of code:
template class F, typename T> struct apply
{ typedef F type; };
template using map = const T;
static_assert(std::is_same::type, const int>::value, "");
line #11542 of pt.c, in tsubst is reached:
r = lookup_template_class (arg,
argve
On Tue, Jun 11, 2013 at 07:43:05PM -0400, Michael Meissner wrote:
> Ah, you are correct. I misread the code, thinking it was returning a string,
> and not a pointer to the string in memory.
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00640.html
Patch reverted.
--
Alan Modra
Australia Developme
> -Original Message-
> From: Richard Earnshaw
> Sent: Thursday, September 12, 2013 11:24 PM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more comparison
> instructions
>
> On 18/04/13 06:34, Bin Cheng wrote:
>
> Sorry for the delay,
m32c's PSImode is 24-bits, why does it have "32" in the macro?
/* 24-bit pointers, in 32-bit units */
-PARTIAL_INT_MODE (SI);
+PARTIAL_INT_MODE_NAME (SI, 32, PSI);
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of bin.cheng
> Sent: Thursday, September 12, 2013 2:13 PM
> To: 'Bill Schmidt'; Yufeng Zhang; Yufeng Zhang
> Cc: Richard Biener; GCC Patches
> Subject: RE: [PATCH GCC]Catch more
The attached patch fixes the regression in gcc.dg/attr-ifunc-4.c (PR 58398).
The problem is that the resolver function just looks like an alias, but it
actually is
something completely different. So inlining the resolver function has to be
avoided.
The patch was bootstrapped and regression-test
101 - 140 of 140 matches
Mail list logo
