On Sat, Nov 09, 2013 at 05:30:00PM +0100, Tom de Vries wrote:
>Richard,
>
>Consider the test-case test.c:
>...
>int z;
>int x;
>
>void
>f (int c, int d)
>{
> if (c)
>z = 5;
> else
>{
> if (d)
> x = 4;
> z = 5;
>}
>}
>...
>
>Atm, we don't tail-merge the 'z = 5' blocks,
Hi,
this is the issue with -Waddress caused by the fix for c++/56930. I'm
handling it as already described, that is by adding a bool parameter to
c_common_truthvalue_conversion.
Tested x86_64-linux.
Thanks!
Paolo.
//
/c-family
2013-11-11 Paolo Carlini
* c-com
Hi,
this is another issue, with -Wzero-as-null-pointer-constant, caused (or,
better, exposed) by the fix for c++/54170: the problem is that
build_ptrmemfunc, called by cp_convert_to_pointer, itself eventually
calls cp_convert_to_pointer.
The build_ptrmemfunc bit isn't essential: using the cp
On 2013-11-10 6:10, Jason Merrill wrote:
Hmm, actually I think messing with the non-pack's decl is dangerous,
and we should come up with a new decl for the pack instead. I think
you can use reduce_template_parm_level with a "levels" argument of 0
to build a new decl and parm index.
I actually
Andrew MacLeod writes:
> 2 - I really believe gimple needs a type system different from front end
> trees, that is my primary motivation. I'm tired of jumping through
> hoops to do anything slightly different, and I got fed up with it. With
> a separate type system for gimple, we can rid ours
On Fri, Nov 8, 2013 at 10:32 PM, Joseph S. Myers
wrote:
>> >> Can we introduce a target-dependant source here, in the same way as
>> >
>> > Sure, that seems a reasonable thing to do. I think putting a file fenv.c
>> > in an appropriate subdirectory of libatomic/config will result in it being
>>
> > +
> > + profile_status_for_function (fn)
> > + = (flag_guess_branch_prob ? PROFILE_GUESSED : PROFILE_ABSENT);
> > + node->frequency
> > + = hot ? NODE_FREQUENCY_HOT : NODE_FREQUENCY_NORMAL;
>
> In GCC code style the = goes at the end of the line:
>
> profile_status_for_function
> > But I think that you cannot transform
> >
> > foo ()
> > {
> >
> > *0 = 1;
> >
> > }
> >
> > to __builtin_trap as you can catch the trap via an exception handler
> > in a caller of foo, no?
>
> That is true. OK, I can see an argument that when using
> -fnon-call-exceptions that kind of
Hi,
this patch adds exeext to -rdynamic and exported symbol-check within
configure.ac. By this this test will be preformed as intended on
targets with executable-extensions (nevertheless -rdynamics will still
fail for pe-coff targets).
ChangeLog
2013-11-10 Karlson2k
Kai Tietz
On Fri, Nov 8, 2013 at 10:26 PM, Joseph S. Myers
wrote:
>> > I've applied this patch to update libgcc's copy of soft-fp from
>> > glibc. There are lots of coding standards fixes, but also various bug
>> > fixes; I've added testcases for various of the fixed bugs illustrating
>> > them for __floa
Richard Sandiford wrote:
>Andrew MacLeod writes:
>> 2 - I really believe gimple needs a type system different from front
>end
>> trees, that is my primary motivation. I'm tired of jumping through
>> hoops to do anything slightly different, and I got fed up with it.
>With
>> a separate type s
On 2013-11-10 10:38, Adam Butcher wrote:
On 2013-11-10 6:10, Jason Merrill wrote:
Hmm, actually I think messing with the non-pack's decl is dangerous,
and we should come up with a new decl for the pack instead. I think
you can use reduce_template_parm_level with a "levels" argument of 0
to bui
Steven Bosscher wrote:
>Hello,
>
>This patch is necessary to make ARM pass the test suite with LRA
>enabled. The symptom is recog failing to recognize a store_minmaxsi
>insn, see:
> http://gcc.gnu.org/ml/gcc-patches/2013-11/msg00725.html
>
>But I am not sure if that's also the root cause of the pr
Hello,
I am posting this patch to get some feedback on the approach. The goal is
to replace malloc+free with a stack allocation (a decl actually) when the
size is a small constant.
For testing, I highjacked the "leaf" attribute, but it isn't right, I'll
remove it from the list (not sure what
Currently on trunk the option -mpreferred-stack-boundary does not work
together with #pragma GCC target("sse") or __attribute__((target("sse"))).
There is already a test case that detects this:
gcc.target/i386/fastcall-sseregparm.c
The attached patch fixes this test case under i686-pc-linux-gnu.
Hi,
This patch documents __builtin_setjmp and __builtin_longjmp. OK to
install?
Thanks.
H.J.
---
2013-11-10 H.J. Lu
PR other/59039
* doc/extend.texi: Document __builtin_setjmp and
__builtin_longjmp.
diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
index 0d7281
On Sun, Nov 10, 2013 at 1:08 PM, Eric Botcazou wrote:
>> > +
>> > + profile_status_for_function (fn)
>> > + = (flag_guess_branch_prob ? PROFILE_GUESSED : PROFILE_ABSENT);
>> > + node->frequency
>> > + = hot ? NODE_FREQUENCY_HOT : NODE_FREQUENCY_NORMAL;
>>
>> In GCC code style the = goes
> Attached patch fixes an oversight in mode-switching. For blocks
> without ANY mode requirements, we have to consider instructions with
> MODE_AFTER mode changes. If the exiting mode from the block is
> different that no_mode (the mode we start), we have to mark the block
> as nontransparent.
>
>
> This is the first patch of what I think will be four to fix those few
> places.
>
> Bootstrapped&tested on powerpc64-unknown-linux-gnu. Also built SH to be
> sure.
>
> OK for trunk?
The generic part is OK (modulo the additional space after ! in the 3rd hunk of
the haifa-sched.c patch).
--
E
Janne Blomqvist wrote:
the attached patch sets the close-on-exec flag when opening files, as
is usually considered good practice these days. See e.g.
http://www.python.org/dev/peps/pep-0446/ and links therein for more
information.
+ int flags = O_RDWR|O_CREAT|O_EXCL;
I'd add spaces around "
For PR libstdc++/55963 I said I'd improve the diagnostics when you try
to use std::vector, which is invalid because the C++
Allocator requirements require the value_type to be a non-const object
type.
I tried adding static assertions to the primary std::allocator
template, but the diagnostics are
On Sun, Nov 10, 2013 at 5:45 PM, Eric Botcazou wrote:
>> Attached patch fixes an oversight in mode-switching. For blocks
>> without ANY mode requirements, we have to consider instructions with
>> MODE_AFTER mode changes. If the exiting mode from the block is
>> different that no_mode (the mode we
On Sun, Nov 10, 2013 at 7:40 PM, Uros Bizjak wrote:
> On Sun, Nov 10, 2013 at 5:45 PM, Eric Botcazou wrote:
>>> Attached patch fixes an oversight in mode-switching. For blocks
>>> without ANY mode requirements, we have to consider instructions with
>>> MODE_AFTER mode changes. If the exiting mode
On 10 November 2013 13:49:37 Kai Tietz wrote:
Hi,
this patch adds exeext to -rdynamic and exported symbol-check within
configure.ac. By this this test will be preformed as intended on
targets with executable-extensions (nevertheless -rdynamics will still
fail for pe-coff targets).
ChangeLog
On 11/10/2013 08:10 AM, Adam Butcher wrote:
With the convert function in pt.c, PATCH 2/3 now looks as follows:
I like this direction.
+ /* Build up a tree vec of empty tree vecs up to the inner substitution
+args built above. */
I think we want to copy the enclosing args; see
OK.
Jason
On 2013-11-10 18:49, Jason Merrill wrote:
On 11/10/2013 08:10 AM, Adam Butcher wrote:
+ /* Build up a tree vec of empty tree vecs up to the inner
substitution
+args built above. */
I think we want to copy the enclosing args; see existing uses of
add_outermost_template_args.
O
John Carr writes:
> I am trying to build the trunk version of gcc to have Cilk on OpenBSD.
>
> I attach three changes to libcilkrts. Two changes treat OpenBSD like
> FreeBSD. One change is needed because OpenBSD defines PTHREAD_MUTEX_* as
> enumerations rather than preprocessor constants.
That
Tejas Belagod writes:
>> The problem is that one reg rtx can span several hard registers.
>> E.g. (reg:V4SI 32) might represent one 64-bit register (no. 32),
>> but it might instead represent two 32-bit registers (nos. 32 and 33).
>> Obviously the latter's not very likely for vectors this small,
>
On 11/10/2013 11:54 AM, Jonathan Wakely wrote:
On 10 November 2013 16:52, Jonathan Wakely wrote:
I thought I'd already made similar changes to gcc-4.9/changes.html for
the C++14 changes but I never committed it. The only comment I have
is that "chrono" isn't a type, but the change is fine as f
On Sun, Nov 10, 2013 at 7:16 PM, Tobias Burnus wrote:
> Janne Blomqvist wrote:
>>
>> the attached patch sets the close-on-exec flag when opening files, as
>> is usually considered good practice these days. See e.g.
>> http://www.python.org/dev/peps/pep-0446/ and links therein for more
>> informat
On 10 November 2013 20:28, Ed Smith-Rowland wrote:
>
> OK, I folded our versions together. Thank you. I also fixed up my code
> examples.
>
> OK?
That looks good, thanks very much for updating it.
On Sun, 10 Nov 2013, Marc Glisse wrote:
I am posting this patch to get some feedback on the approach. The goal is to
replace malloc+free with a stack allocation (a decl actually) when the size
is a small constant.
A slightly updated version that handles abort and if(VAR==0) where VAR is
the
On 11/10/2013 02:39 PM, Adam Butcher wrote:
I assumed that tsubst simply doesn't do anything with a
null tree substitution (i.e. it is an identity op).
Substituting NULL_TREE for a template parameter gives a template
parameter with a reduced level; this happens during partial instantiation.
On Fri, Nov 08, 2013 at 10:37:00AM +0100, Richard Biener wrote:
> On Thu, Nov 7, 2013 at 5:00 PM, wrote:
> > From: Trevor Saunders
> >
> > Hi,
> >
> > This is the result of seeing what it would take to get rid of the has_gate
> > and
> > has_execute flags on pass_data. It turns out not much,
Can someone please review this patch?
http://gcc.gnu.org/ml/gcc-patches/2013-10/msg02637.html
I would like to commit the already-approved -fstrict-volatile-bitfields
patch once we also have an approved fix for the infinite recursion
problem I discovered while testing a backport of the patch se
On 11/10/2013 03:59 PM, Jonathan Wakely wrote:
On 10 November 2013 20:28, Ed Smith-Rowland wrote:
OK, I folded our versions together. Thank you. I also fixed up my code
examples.
OK?
That looks good, thanks very much for updating it.
OK, I got gcc-4.9/changes.html checked in (after some tr
If we have a jump threading request through a joiner block which has two
successors S1 and S2. If the threading request goes through S1 and
ultimately reaches S2, then we have to ensure that any PHI nodes in S2
have the same arguments for edges J->S2 and J->S1...->S2.
The SSA/CFG updating c
On 11/10/13 05:34, Eric Botcazou wrote:
But I think that you cannot transform
foo ()
{
*0 = 1;
}
to __builtin_trap as you can catch the trap via an exception handler
in a caller of foo, no?
That is true. OK, I can see an argument that when using
-fnon-call-exceptions that kind of code s
On Fri, Nov 8, 2013 at 10:06 PM, Richard Biener
wrote:
> On Fri, Nov 8, 2013 at 2:41 PM, bin.cheng wrote:
>> Hi,
>> This patch refactors force_expr_to_var_cost and handles type conversion
>> along with other tree nodes. It is split from the patch posted at
>> http://gcc.gnu.org/ml/gcc-patches/20
40 matches
Mail list logo
