On Jun 20, 2012, at 12:36 PM, rbmj wrote:
> My issue is that I'm uncomfortable with this, as it seems *too* easy.
I'd just be comfortable with a stake in the ground and press forward. I do
think this covers most all the cases.
Hi!
During htab_delete (dropped_values), loc_exp_dep_pool
allocated objects might be accessed, so it is better to free the
pool afterwards.
Bootstrapped/regtested on i686-linux, ok for trunk?
2012-06-21 Jakub Jelinek
PR debug/53706
* var-tracking.c (vt_emit_notes): Free loc_e
OK.
Jason
On Jun 20, 2012, at 1:44 AM, Steven Bosscher wrote:
> Attached is a concept patch to move the C front end to its own
> sub-directory
I like the idea... but did you build and test objective c++?
On Wed, Jun 20, 2012 at 08:43:53PM -0300, Alexandre Oliva wrote:
> On Jun 20, 2012, Jakub Jelinek wrote:
>
> > On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote:
> >> When promote_debug_loc was first introduced, it would never be called
> >> with a NULL loc list. However, because o
On Jun 20, 2012, at 12:26 AM, Jason Merrill wrote:
> The recent regression in libgomp leads me to want to add libgomp tests to the
> check-c++ target.
I'm fine with the idea...
I modified the documentation and it now looks like this:
@item -ftemp-stack-reuse
@opindex ftemp_stack_reuse
This option enables stack space reuse for temporaries. The default is on.
The lifetime of a compiler generated temporary is well defined by the C++
standard. When a lifetime of a temporary
Hi,
This patch creates basic gimple_assign statements. It is a little raw
not considering all types of gimple_assign statements for which I have
already started working.
Here is the Changelog.
2012-06-09 Sandeep Soni
* parser.c (gimple_symtab_get): New.
(gimple_symtab_get
The documentation needs to explain more what the option controls, and
why you might want it on or off. Other than that it looks fine.
Jason
Lawrence,
you ask a number of awfully good questions. :-)
First of all you made me realize that we were missing a
cross-link from http://gcc.gnu.org/projects/web.html to
http://gcc.gnu.org/contribute.html#webchanges which the
first patch included below does now.
On Tue, 5 Jun 2012, Lawrence Cro
OK.
Jason
On 10 April 2012 10:11, Ramana Radhakrishnan
wrote:
>>> The patch with correct configure output is ok.
>>
>> Thanks - this is what I committed.
>
> Is this something that can be considered for backporting to release
> branches ? This patch technically doesn't fix a regression but brings
> in line
On Jun 20, 2012, Jakub Jelinek wrote:
> On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote:
>> When promote_debug_loc was first introduced, it would never be called
>> with a NULL loc list. However, because of the strategy of temporarily
>> resetting loc lists before recursion intro
Applied.
Gerald
Index: gcc-4.7/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v
retrieving revision 1.119
diff -u -3 -p -r1.119 changes.html
--- gcc-4.7/changes.html14 Jun 2012 17:57:18 - 1.1
One of the most common runtime errors we have seen in gcc-4_7 is
caused by dangling references to temporaries whole life time have
ended
e.g,
const A& a = foo();
or
foo (A());// where temp's address is saved and used after foo.
Of course this is user error according to the standard, triaging o
>> + /* If we're putting types in their own .debug_types sections,
>> + the .debug_pubtypes table will still point to the compile
>> + unit (not the type unit), so we want to use the offset of
>> + the skeleton DIE (if there is one). */
>> + if (pub
Hi,
the attached patch makes the FTELL intrinsic function work on offsets
larger than 2 GB on 32-bit systems that support large files. As this
is an ABI change the old library function is left untouched, to be
removed when/if the library ABI is bumped.
Regtested on x86_64-unknown-linux-gnu, Ok fo
This patch addresses the problem of building GCC for mips with the
'--with-synci' configure option. If you do that and then compile a
program with GCC and specify an architecture that does not support synci
(such as the -mips32 option), GCC will issue a warning that synci
is not supported. This r
On 06/12/2012 08:16 AM, rbmj wrote:
On 06/11/2012 08:01 AM, Paolo Bonzini wrote:
VxWorks should define TARGET_POSIX_IO if it has both access and
mkdir. Please add it to gcc/config/vxworks.h if this is the case.
I misspoke in my earlier email - sorry for my lack of attention to
detail. The is
On Wed, 2012-06-20 at 11:52 -0700, Richard Henderson wrote:
> On 06/20/2012 04:11 AM, Richard Guenther wrote:
> > I notice (with all of these functions)
> >
> > +unsigned
> > +negate_cost (enum machine_mode mode, bool speed)
> > +{
> > + static unsigned costs[NUM_MACHINE_MODES];
> > + rtx seq;
>
On 06/13/2012 01:59 PM, Joseph S. Myers wrote:
On Wed, 13 Jun 2012, rbmj wrote:
Since u?int.*_t are already defined, would this work? Or should I use the
non-typedef'd versions? Also, I'm not exactly sure how to run the regression
tests with a cross compiler. I'm still new to everything, bea
Hi,
A few weeks ago I submitted a fix for a garbage collection issue I ran
into involving -Wunused-local-typedefs [1]. The analysis for that patch
still stands, but unfortunately the patch is wrong.
The problem is that the allocation reuse can't be removed otherwise the
information about local t
Hello!
2012-06-20 Uros Bizjak
* config/i386/i386.md (SINCOS): New int iterator.
(sincos): New int attribute.
(*xf2_i387): Macroize insn from *{sin,cos}xf2_i387 using
SINCOS int iterator.
(*_extendxf2_i387): Macroize insn from
*{sin,cos}_extendxf2
On Wed, 20 Jun 2012, Richard Guenther wrote:
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse wrote:
Hello,
currently, tree-ssa-ifcombine handles pairs of imbricated "if"s that share
the same then branch, or the same else branch. There is no particular reason
why it couldn't also handle the case
On 06/20/2012 04:11 AM, Richard Guenther wrote:
> I notice (with all of these functions)
>
> +unsigned
> +negate_cost (enum machine_mode mode, bool speed)
> +{
> + static unsigned costs[NUM_MACHINE_MODES];
> + rtx seq;
> + unsigned cost;
> +
> + if (costs[mode])
> +return costs[mode];
> +
On 12-06-20 13:50 , Diego Novillo wrote:
OK. I suppose you do not have write to the repo, so I will commit it
for you.
Committed r188841.
Diego.
On 12-06-20 13:43 , Rüdiger Sonderfeld wrote:
The patch is extremely trivial and probably doesn't need copyright assignment.
However I have signed copyright assignment for Emacs and maybe that will work
too (not sure if this has to be signed for every project).
It does, unfortunately.
gcc/C
Hello,
I'm currently writing a plugin for GCC and I need to extract the column of a
declaration. There exists DECL_SOURCE_FILE/LINE already and for consistency
reasons and ease of use I've added a DECL_SOURCE_COLUMN macro.
The patch is extremely trivial and probably doesn't need copyright assign
Hello!
2012-06-20 Uros Bizjak
* config/i386/i386.md (2): Macroize expander
from {floor,ceil,btrunc}2 using FIST_ROUNDING int iterator.
(l2): Macroize expander
from l{floor,ceil}2 using FIST_ROUNDING
int iterator.
Bootstrapped and regression tested on x8
On 20/06/12 15:41, Sebastian Huber wrote:
> On 06/13/2012 02:51 PM, Richard Earnshaw wrote:
>> This patch is the first of a series to remove support for the now
>> obsolete FPA and Maverick co-processors.
>>
>> This patch removes those targets and configuration options that were
>> marked as deprec
On Mon, Jun 18, 2012 at 9:34 AM, Roland McGrath wrote:
> OK then. If you like the original patch, would you like to commit it for me?
ping?
On 06/20/2012 12:57 AM, Ville Voutilainen wrote:
If a single pipe is indeed to be used, perhaps we want to correct that
piece of documentation, lest fools follow its advice. :)
Done, thanks.
Jason
On Wed, 2012-06-20 at 13:11 +0200, Richard Guenther wrote:
> On Thu, Jun 14, 2012 at 3:21 PM, William J. Schmidt
> wrote:
> > Pro forma ping. :)
>
> ;)
>
> I notice (with all of these functions)
>
> +unsigned
> +negate_cost (enum machine_mode mode, bool speed)
> +{
> + static unsigned costs[NU
On Wed, Jun 20, 2012 at 5:09 PM, Michael Matz wrote:
> Hi,
>
> On Wed, 20 Jun 2012, Richard Guenther wrote:
>
>> > + exp = fold_build2 (MEM_REF,
>> > + build_array_type (char_type_node,
>> > + build_range_type (sizetype,
>> > +
Hi,
On Wed, 20 Jun 2012, Richard Guenther wrote:
> > + exp = fold_build2 (MEM_REF,
> > + build_array_type (char_type_node,
> > + build_range_type (sizetype,
> > + size_one_node,
> > le
On 12-06-20 10:10 , Steven Bosscher wrote:
-# By default, C is the only stage 1 language.
+# By default, C and C++ are the only stage 1 languages.
stage1_languages=,c,
So shouldn't you add c++ here?
That was a bad change on the comment. We only need C for stage1.
Thanks for spotting it.
On Wed, Jun 20, 2012 at 4:57 PM, Michael Matz wrote:
> Hi,
>
> On Tue, 19 Jun 2012, Richard Guenther wrote:
>
>> The MEM_REF is acceptable to the tree oracle and it can extract
>> points-to information from it.
>>
>> Thus for simplicity unconditionally building the above is the best.
>
> But it do
Hi,
On Tue, 19 Jun 2012, Richard Guenther wrote:
> The MEM_REF is acceptable to the tree oracle and it can extract
> points-to information from it.
>
> Thus for simplicity unconditionally building the above is the best.
But it doesn't work, as refs_may_alias_p_1 only accepts certain operands
On 06/13/2012 02:51 PM, Richard Earnshaw wrote:
This patch is the first of a series to remove support for the now
obsolete FPA and Maverick co-processors.
This patch removes those targets and configuration options that were
marked as deprecated in GCC-4.7 and removes the config fragments that
de
On Wed, Jun 20, 2012 at 4:10 PM, Steven Bosscher wrote:
> On Wed, Jun 20, 2012 at 1:08 AM, Diego Novillo wrote:
>> diff --git a/configure.ac b/configure.ac
>> index 071b5e2..2a2a0c6 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -1667,7 +1653,7 @@ ACX_ELF_TARGET_IFELSE([# ELF platforms
On Wed, Jun 20, 2012 at 1:08 AM, Diego Novillo wrote:
> diff --git a/configure.ac b/configure.ac
> index 071b5e2..2a2a0c6 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1667,7 +1653,7 @@ ACX_ELF_TARGET_IFELSE([# ELF platforms build the
> lto-plugin always.
> ])
>
>
> -# By default, C is
On Wed, 20 Jun 2012, Richard Guenther wrote:
> On Wed, 20 Jun 2012, Iain Sandoe wrote:
>
> > Hi,
> >
> > On 20 Jun 2012, at 09:23, Richard Guenther wrote:
> >
> > > On Tue, 19 Jun 2012, Iain Sandoe wrote:
> > >
> > >>
> > >> On 19 Jun 2012, at 22:41, Mike Stump wrote:
> > >>
> > >>> On Jun 1
On Wed, 20 Jun 2012, Iain Sandoe wrote:
> Hi,
>
> On 20 Jun 2012, at 09:23, Richard Guenther wrote:
>
> > On Tue, 19 Jun 2012, Iain Sandoe wrote:
> >
> >>
> >> On 19 Jun 2012, at 22:41, Mike Stump wrote:
> >>
> >>> On Jun 19, 2012, at 12:22 PM, Iain Sandoe wrote:
> On 19 Jun 2012, at 13
dnovi...@google.com (Diego Novillo) writes:
> Ian, could you please take a look to double check I have not missed
> anything? There was more code dealing with it than I was expecting.
It all looks plausible to me.
Ian
On Wed, Jun 20, 2012 at 9:35 AM, Alan Modra wrote:
> On Wed, Jun 20, 2012 at 09:10:44AM -0400, David Edelsohn wrote:
>> inline static void
>> release_set (volatile obj_addr_t *addr, obj_addr_t new_val)
>> {
>> - __asm__ __volatile__ ("sync" : : : "memory");
>> - *addr = new_val;
>> + __atomi
Hi,
On 20 Jun 2012, at 09:23, Richard Guenther wrote:
> On Tue, 19 Jun 2012, Iain Sandoe wrote:
>
>>
>> On 19 Jun 2012, at 22:41, Mike Stump wrote:
>>
>>> On Jun 19, 2012, at 12:22 PM, Iain Sandoe wrote:
On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote:
> On Tue, 19 Jun 2012
On 06.06.2012 11:00, Ramana Radhakrishnan wrote:
Ok with those changes. Ramana .
Hi Ramana,
How about this version?
Christophe.
commit f57ce4b63ca1c30ee88e8c1a431d6e90ffbecb82
Author: Christophe Lyon
Date: Wed Jun 20 15:30:50 2012 +0200
2012-06-20 Christophe Lyon
* gc
On Wed, Jun 20, 2012 at 09:10:44AM -0400, David Edelsohn wrote:
> inline static void
> release_set (volatile obj_addr_t *addr, obj_addr_t new_val)
> {
> - __asm__ __volatile__ ("sync" : : : "memory");
> - *addr = new_val;
> + __atomic_store_n(addr, val, __ATOMIC_RELEASE);
A typo seems to hav
Alan and I both re-implemented the locks and settled on the following
patch. This uses the __atomic intrinsics, not the __sync instrinsics,
to avoid generating expensive instructions for a memory model that is
stricter than necessary.
If these intrinsics correctly represent the semantics of the l
Hi ,
This improves the vdup_n intrinsics where one tries to form constant
vectors. This uses targetm.fold_builtin to fold these vector
initializations to actual vector constants. The vdup_n cases are fine
with both endian-ness as the vector constant is just duplicated. In
addition I've made the *n
As both tests are checking already for !ia32, there is no additiona
check beside the targets necessary.
Cheers,
Kai
On Mon, Jun 18, 2012 at 10:09 AM, Kai Tietz wrote:
> 2012/6/18 JonY :
>> Hi,
>>
>> I am told that this ABI test does not apply to mingw targets. OK to apply?
>
> Hi JonY,
>
> The test doesn't apply to x64 windows targets, as for it sse is part of its
> ABI.
> As test already checks for !ia32, we
Hi,
ChangeLog
2012-06-20 Kai Tietz
* gcc.target/i386/pr23943.c (size_t): Use compatible type-definition
for LLP64 targets.
* gcc.target/i386/pr38988.c: Likewise.
Regression-tested for x86_64-w64-mingw32, and x86_64-unknown-linux-gnu.
Ok for apply?
Regards,
Kai
Index
On Wed, Jun 20, 2012 at 4:44 AM, Steven Bosscher wrote:
> I'm posting this as an RFC: Does this look like the right approach?
> Have I overlooked other things than just documentation updates? I hope
> this would not cause too much trouble for branches like the
> cxx-conversion branch?
It should
Richard Guenther wrote:
> In this testcase the alignment of arr[i] should be irrelevant - it is
> not part of the stmts that are going to be vectorized.
Agreed.
> But of course this may be
> simply an odering issue in how we analyze data-references / statements
> in basic-block vectorization (th
This concludes the VRP and anti-ranges series for now (well, it was
the motivation for this patch which was pending for quite some time).
This re-implements PLUS_EXPR support on integer ranges to cover
all cases, even those that generate an anti-range as result.
Bootstrapped and tested on x86_64-
Richard Guenther writes:
> On Tue, Jun 19, 2012 at 11:36 PM, Mikael Pettersson wrote:
> > Richard Guenther writes:
> > > On Fri, Jun 15, 2012 at 5:00 PM, Ulrich Weigand
> > wrote:
> > > > Richard Guenther wrote:
> > > >> On Fri, Jun 15, 2012 at 3:13 PM, Ulrich Weigand
> > wrote:
> >
On Wed, 20 Jun 2012 11:56:39 +0100
Ramana Radhakrishnan wrote:
> Hi,
>
> This patch helps use the __builtin_shuffle intrinsics to implement the
> Neon permute intrinsics following on from Julian's and my patch last
> week. It needed support for __builtin_shuffle in the C++ frontend
> which is no
On Wed, Jun 20, 2012 at 12:30 PM, Eric Botcazou wrote:
> Hi,
>
> this is a repost of
> http://gcc.gnu.org/ml/gcc-patches/2010-10/msg02468.html
> earlier in the development cycle, so with hopefully more time for discussion.
>
> The command line option -fcallgraph-info is added and makes the compil
On Thu, Jun 14, 2012 at 3:21 PM, William J. Schmidt
wrote:
> Pro forma ping. :)
;)
I notice (with all of these functions)
+unsigned
+negate_cost (enum machine_mode mode, bool speed)
+{
+ static unsigned costs[NUM_MACHINE_MODES];
+ rtx seq;
+ unsigned cost;
+
+ if (costs[mode])
+return c
Hello!
2012-06-20 Uros Bizjak
* config/i386/i386.md (rounding_insn): New int attribute.
(xf2): Macroize insn from
{floor,ceil,btrunc}xf2 using FRNDINT_ROUNDING int iterator.
(lxf2): Rename from lxf2.
2012-06-20 Uros Bizjak
* config/i386/i386.md (IEE
Hello,
These macros are now gone and should be poisoned. I'll commit this
later today unless someone objects.
Ciao!
Steven
* system.h: Poison ASM_OUTPUT_IDENT and IDENT_ASM_OP.
Index: system.h
===
--- system.h(revision 18
On Jun 19, 2012, at 6:47 PM, Richard Henderson wrote:
> On 2012-06-18 05:22, Tristan Gingold wrote:
>> +/* Output assembly code to get the establisher frame (Windows x64 only).
>> + This corresponds to what will be computed by Windows from Frame Register
>> + and Frame Register Offset fields
On Wed, Jun 20, 2012 at 12:40 PM, Steven Bosscher wrote:
> On Wed, Jun 20, 2012 at 12:30 PM, Eric Botcazou wrote:
>> * cgraph.c: Include expr.h and output.h.
>
> What for?
Never mind, I see why you need this.
On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote:
> When promote_debug_loc was first introduced, it would never be called
> with a NULL loc list. However, because of the strategy of temporarily
> resetting loc lists before recursion introduced a few months ago in
> alias.c, the earl
On Wed, Jun 20, 2012 at 12:52:12AM -0300, Alexandre Oliva wrote:
> On Jun 16, 2012, "H.J. Lu" wrote:
> from Alexandre Oliva
>
> PR debug/53671
> PR debug/49888
> * alias.c (memrefs_conflict_p): Improve handling of AND for
> alignment.
> from Alexandre Oliva
>
>
On Wed, Jun 20, 2012 at 12:30 PM, Eric Botcazou wrote:
> * cgraph.c: Include expr.h and output.h.
What for?
Ciao!
Steven
Hi,
this is a repost of
http://gcc.gnu.org/ml/gcc-patches/2010-10/msg02468.html
earlier in the development cycle, so with hopefully more time for discussion.
The command line option -fcallgraph-info is added and makes the compiler
generate another output file (xxx.ci) for each compilation unit
PING.
On Mon, May 28, 2012 at 10:51:27AM +0100, Matthew Gretton-Dann wrote:
> All,
>
> This patch adds a variety of predefined macros to reveal the presence of
> various features of the ARM architecture. These are detailed in the ARM
> C Language Extensions specification, available here:
> http:/
On 19/06/12 23:44, Steven Bosscher wrote:
> Hello,
>
> Only a few front-end files to go that need output.h, and some of them
> are in the c_target_objs: arm, mep, m32c, and rl78.
>
> This patch tackles the ARM case. arm-c.c needs output.h because
> EMIT_EABI_ATTRIBUTE wants to print to asm_out_f
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse wrote:
> Hello,
>
> currently, tree-ssa-ifcombine handles pairs of imbricated "if"s that share
> the same then branch, or the same else branch. There is no particular reason
> why it couldn't also handle the case where the then branch of one is the
> el
Hello,
m32c-pragma.c doesn't need the includes that the patch below removes.
Tested with a cross from powerpc64-unknown-linux-gnu to m32c-elf.
Will commit as obvious unless someone objects.
Ciao!
Steven
* config/m32c/m32c-pragma.c: Remove unnecessary includes.
Index: config/m32c/m32c-p
Hello,
rl78-c.c includes many files that it doesn't need. The patch below removes them.
I could not really test this, because rl78 build is broken:
/home/stevenb/devel/cross/build-rl78-test/./gcc/xgcc
-B/home/stevenb/devel/cross/build-rl78-test/./gcc/ -nostdinc
-B/home/stevenb/devel/cross/build-
On Tue, 19 Jun 2012, Dominique Dhumieres wrote:
> On Tue, 19 Jun 2012, Richard Guenther wrote:
> >
> > > Richard Guenther writes:
> > > > We are too eager to bump alignment of some decls when vectorizing.
> > > > The fix is to not bump alignment of decls the user explicitely
> > > > aligned or t
Tailcalls have no argument setup cost and no return value cost.
This patch adjusts estminate_num_insns to reflect that.
Honza, does this look correct?
Bootstrapped and tested on x86_64-unknown-linux-gnu.
Thanks,
Richard.
2012-06-20 Richard Guenther
* tree-inline.c (estimate_num_ins
On Wed, Jun 20, 2012 at 10:44 AM, Steven Bosscher wrote:
> Hello,
>
> Attached is a concept patch to move the C front end to its own
> sub-directory of the main gcc directory. Things like updates of
> sourcebuild.texi are not yet included.
>
> I'm posting this as an RFC: Does this look like the ri
On Wed, Jun 20, 2012 at 9:26 AM, Jason Merrill wrote:
> The recent regression in libgomp leads me to want to add libgomp tests to
> the check-c++ target. OK for trunk?
Ok. (what about libitm?)
Thanks,
Richard.
On Wed, Jun 20, 2012 at 5:52 AM, Alexandre Oliva wrote:
> On Jun 16, 2012, "H.J. Lu" wrote:
>
>> If I understand it correctly, the new approach fails to handle push
>> properly.
>
> It's actually cselib that didn't deal with push properly, so it thinks
> incoming stack arguments may be clobbered
Hello,
Attached is a concept patch to move the C front end to its own
sub-directory of the main gcc directory. Things like updates of
sourcebuild.texi are not yet included.
I'm posting this as an RFC: Does this look like the right approach?
Have I overlooked other things than just documentation u
On Jun 19, 2012, at 6:47 PM, Richard Henderson wrote:
> On 2012-06-18 05:22, Tristan Gingold wrote:
>> + /* Win64 SEH, very large frames need a frame-pointer as maximum stack
>> + allocation is 4GB (add a safety guard for saved registers). */
>> + if (TARGET_64BIT_MS_ABI && get_frame_size
On Tue, Jun 19, 2012 at 11:36 PM, Mikael Pettersson wrote:
> Richard Guenther writes:
> > On Fri, Jun 15, 2012 at 5:00 PM, Ulrich Weigand
> wrote:
> > > Richard Guenther wrote:
> > >> On Fri, Jun 15, 2012 at 3:13 PM, Ulrich Weigand
> wrote:
> > >> > However, there is a second case where we
> Did you mean to put a bugzilla link here?
Yes;-( http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53565
copy and paste from the wrong window).
Dominique
On Tue, 19 Jun 2012, Iain Sandoe wrote:
>
> On 19 Jun 2012, at 22:41, Mike Stump wrote:
>
> > On Jun 19, 2012, at 12:22 PM, Iain Sandoe wrote:
> >> On 19 Jun 2012, at 13:53, Dominique Dhumieres wrote:
> >>
> >>> On Tue, 19 Jun 2012, Richard Guenther wrote:
>
> > Richard Guenther wri
Hi Michael
It seems the wiki page describes 64bit operations on NEON only. My
patches improves 64bit operations on core registers only. I touched
the neon patterns simply because those DI mode operations are enabled
separately according to the TARGET_NEON value, so in the neon patterns
I duplicate
On 20 June 2012 10:35, Jason Merrill wrote:
> Applied, thanks. Note that your dg-error regexp doesn't make much sense:
>
>
> // { dg-error "expected type-specifier before 'final'||expected
> ';'||declaration doesn't declare anything" }
>
> Regular expression "or" uses a single |, so this ends up
> I've applied the patch below. I'll backport it release branches.
Thanks!
--
Eric Botcazou
>> This is wrong, please remove the dg-options line and do like the other
>> targets.
>
> I'll revert that line and use my patch in the trail #11 of
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53621.
I've applied the patch below. I'll backport it release branches.
Regards,
kaz
--
201
On 06/15/2012 02:59 PM, Dominique Dhumieres wrote:
Back when we added C++11 auto deduction, I thought we could shortcut the
normal deduction in some templates, when the type is adequately
describable (thus the late, unlamented function describable_type). Over
time various problems with this have
Applied, thanks. Note that your dg-error regexp doesn't make much sense:
// { dg-error "expected type-specifier before 'final'||expected
';'||declaration doesn't declare anything" }
Regular expression "or" uses a single |, so this ends up being a long
way of writing
// { dg-error "" }
I a
Eric Botcazou wrote:
> This is wrong, please remove the dg-options line and do like the other
> targets.
I'll revert that line and use my patch in the trail #11 of
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53621.
Regards,
kaz
The recent regression in libgomp leads me to want to add libgomp tests
to the check-c++ target. OK for trunk?
commit 3eaa6c5b268115cbf4ab762b5d7b50022389ef25
Author: Jason Merrill
Date: Tue Jun 19 18:16:34 2012 -0700
* Makefile.tpl (check-target-libgomp-c++): New.
* Makefile.def (c+
> I've applied the attached patch which is a tiny SH specific
> change of gcc.dg/stack-usage-1.c test. Tested on sh-linux
> and i686-pc-linux-gnu.
This is wrong, please remove the dg-options line and do like the other targets.
--
Eric Botcazou
On Wed, Jun 20, 2012 at 2:21 AM, Hans-Peter Nilsson wrote:
> On Tue, 19 Jun 2012, Steven Bosscher wrote:
>> I've now committed this, see r188791.
>
> Breaking cris-elf. Just try rebuilding cc1:
> ./gcc/gcc/../libdecnumber/dpd -I../libdecnumber \
> /tmp/hpautotest-gcc1/gcc/gc
93 matches
Mail list logo
