On 06/22/2012 02:15 PM, Cary Coutant wrote:
But if the consensus turns out to be that enumerators should be in
pubnames, wouldn't it also be fairly easy to change prune_unused_types
so that it doesn't mark enumerators, and change output_pubnames to
skip enumerators that have been pruned?
This m
On Fri, 2012-06-22 at 10:44 +0200, Richard Guenther wrote:
> On Thu, 21 Jun 2012, William J. Schmidt wrote:
> > I ran into a glitch with multiply_by_const_cost. The original code
> > declared a static htab_t in the function and allocated it on demand.
> > When I tried adding a second one in the s
>
> 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-
On 06/11/2012 08:01 AM, Paolo Bonzini wrote:
Il 11/06/2012 13:56, rbmj ha scritto:
1. VxWorks does not have a variadic open - it must receive three
arguments. gcc/gcov.c however opens a file for reading and does not
pass in a mode argument, which causes an error on vxWorks. This just
adds a p
Here is an alternative patch that just changes the configure test
controlling %; so it will elide the ; only for an assembler that
also accepts 'rep bsf', 'rep bsr', and 'rep ret', and just uses
%; for these cases too. You'll need to have built binutils from its trunk
within the last five minutes
> prune_unused_types marks everything in the pubnames_table. If the
> enumerators go in the pubname table (but the enum itself goes in the
> pubtype table), then the unused enum becomes reachable from the
> pubnames table.
>
> The least complicated solution is to put them back in the
> pubtypes_tab
> Anyway, I'll commit the attached patch later this evening ...
Committed as r188902.
Cheers,
Janus
>>> 2012-06-17 Janus Weil
>>>
>>> PR fortran/47710
>>> PR fortran/53328
>>> * interface.c (count_types_test,generic_correspondence,
>>> gfc_compare_interfaces): Ig
Iain Sandoe writes:
> Hello,
>
> While working on mips recently, I noticed that all the execute tests
> fail for simulator.
>
> This appears to be caused by an oversight in the move from gcc/config
> => libgcc, where t-elf defined extra parts including crt{begin,end}.o
> but these have been omitte
Yes.
Original Message
From: Richard Guenther
Sent: Fri, Jun 22, 2012 02:39 AM
To: Jason Merrill
CC: Xinliang David Li ; GCC Patches
; Michael Matz
Subject: Re: New option to turn off stack reuse for temporaries
On Fri, Jun 22, 2012 at 11:29 AM, Jason Merrill wrote:
> O
On Fri, Jun 22, 2012 at 2:47 AM, Dominique Dhumieres wrote:
>> I've heard reports of new test failures due to this patch:
>>
>> FAIL: gcc.dg/pubtypes-2.c scan-assembler long+[ \\t]+0x6a+[ \\t]+[#;]+[
>> \\t]+Length of Public Type Names Info
>> FAIL: gcc.dg/pubtypes-3.c scan-assembler long+[ \\t]+
On Fri, Jun 22, 2012 at 2:35 AM, Jason Merrill wrote:
> FAIL: g++.dg/debug/dwarf2/static-data-member2.C -std=gnu++98
> scan-assembler-not DW_TAG_enumerator
> FAIL: g++.dg/debug/dwarf2/static-data-member2.C -std=gnu++98
> scan-assembler-not DW_TAG_enumeration_type
> FAIL: g++.dg/debug/dwarf2/static
On Fri, Jun 22, 2012 at 11:57 AM, Uros Bizjak wrote:
> Please do not introduce new macro, just use #ifdef
> HAVE_AS_IX86_REP_BSFBSR directly in i386.md.
Would you want the same #ifdef in two places if I extend this to handle
'rep ret' too, or would a macro then be preferable?
To me, the macro is
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Spanish team of translators. The file is available at:
http://translationproject.org/latest/gcc/es.po
(This file, 'gcc-4.7.1.es.po', has just
On 06/21/2012 09:40 PM, Hans-Peter Nilsson wrote:
On Wed, 20 Jun 2012, rbmj wrote:
There is an alternate solution- I could use fixincludes to add a macro to wrap
over mkdir on VxWorks. A couple of possible ways to do this:
1. Define a normal macro to posix-ify it, i.e. #define mkdir(a, b)
((m
> On Fri, Jun 22, 2012 at 11:15 AM, Jan Hubicka wrote:
> >> >
> >>
> >> Here is the patch for invoke.texi. OK to install?
> >>
> >> I will prepare a similar one for news.html.
> >>
> >> Thanks.
> >>
> >> --
> >> H.J.
> >> ---
> >> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> >> index
Hello!
> The GNU assembler now (just as of today) accepts 'rep bsf ...' or 'rep bsr
> ...'
> syntax. It's always better to put a prefix on the instruction itself
> rather than to write 'rep; ...'. This changes 'rep; bsf ...' to 'rep bsf ...'
> when the assembler accepts the latter.
> 2012-06-2
On Fri, Jun 22, 2012 at 11:15 AM, Jan Hubicka wrote:
>> >
>>
>> Here is the patch for invoke.texi. OK to install?
>>
>> I will prepare a similar one for news.html.
>>
>> Thanks.
>>
>> --
>> H.J.
>> ---
>> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
>> index 029a7ab..87e0d1c 100644
>> -
The GNU assembler now (just as of today) accepts 'rep bsf ...' or 'rep bsr ...'
syntax. It's always better to put a prefix on the instruction itself
rather than to write 'rep; ...'. This changes 'rep; bsf ...' to 'rep bsf ...'
when the assembler accepts the latter.
Thanks,
Roland
gcc/
2012-06
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'cpplib' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/c
> >
>
> Here is the patch for invoke.texi. OK to install?
>
> I will prepare a similar one for news.html.
>
> Thanks.
>
> --
> H.J.
> ---
> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> index 029a7ab..87e0d1c 100644
> --- a/gcc/doc/invoke.texi
> +++ b/gcc/doc/invoke.texi
> @@ -1357
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/gcc-
On Fri, Jun 22, 2012 at 10:11 AM, Jan Hubicka wrote:
>> >>> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
>> >>> index 4c5c79f..daa1f3a 100644
>> >>> --- a/gcc/doc/invoke.texi
>> >>> +++ b/gcc/doc/invoke.texi
>> >>> @@ -13521,6 +13521,12 @@ Attempt to keep the stack boundary aligned to a
On 20 June 2012 14:37, Christophe Lyon wrote:
> On 06.06.2012 11:00, Ramana Radhakrishnan wrote:
>>
>> Ok with those changes. Ramana .
>
>
> Hi Ramana,
>
> How about this version?
>
> Christophe.
>
OK -
This should also go into the release branches as it fixes wrong code
with an intrinsic .
Th
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'cpplib' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/c
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to .)
A new POT file for textual domain 'gcc' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/gcc-
Marcus Shawcroft wrote:
On 14/06/12 13:24, Sofiane Naci wrote:
Hi,
This patch updates LINK_SPEC in the AArch64 port.
Thanks
Sofiane
-
2012-06-14 Sofiane Naci
[AArch64] Update LINK_SPEC.
* config/aarch64/aarch64-linux.h (LINUX_TARGET_LINK_SPEC): Remove
%{version
Marcus Shawcroft wrote:
On 01/06/12 09:53, Sofiane Naci wrote:
Hi,
This patch replaces instances of sprintf with snprintf with sizeof(..) in
the AArch64 port.
It also fixes layout issues in the code it touches.
Thanks
Sofiane
-
ChangeLog
2012-06-01 Sofiane Naci
[AArch64] Repla
Marcus Shawcroft wrote:
On 11/06/12 09:54, Sofiane Naci wrote:
(-march and -mcpu feature modifiers): New subsubsection.
Copy and paste subsub?
/Marcus
I've fixed up the typo in the CHangelog and checked this in on aarch64-branch
for Sofiane.
Tejas.
Marcus Shawcroft wrote:
On 01/06/12 09:56, Sofiane Naci wrote:
Hi,
This patch re-factors TLS dialect option selection in the AArch64 port to
use the generic support for enumerated option arguments.
Thanks
Sofiane
-
2012-06-01 Sofiane Naci
[AArch64] Use Enums for TLS option sele
Marcus Shawcroft wrote:
On 13/06/12 14:38, Sofiane Naci wrote:
Hi,
I discovered a bug in my previous patch, so I attach a new one.
The ChangeLog hasn't changed.
OK to commit?
Thanks
Sofiane
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
> >>> diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> >>> index 4c5c79f..daa1f3a 100644
> >>> --- a/gcc/doc/invoke.texi
> >>> +++ b/gcc/doc/invoke.texi
> >>> @@ -13521,6 +13521,12 @@ Attempt to keep the stack boundary aligned to a
> >>> 2 raised to @var{num}
> >>> byte boundary. If @opt
On 06/22/2012 06:46 AM, H.J. Lu wrote:
> Ping.
>
> I have been running Linux kernel compiled with this option for
> weeks. Since -mpreferred-stack-boundary=3 will only be allowed
> with -mno-sse, it will be used only with incompatible ABI. Any
> objections to this patch?
>
> Thanks.
The patch
On Fri, Jun 22, 2012 at 2:47 AM, Dominique Dhumieres wrote:
>> I've heard reports of new test failures due to this patch:
>>
>> FAIL: gcc.dg/pubtypes-2.c scan-assembler long+[ \\t]+0x6a+[ \\t]+[#;]+[
>> \\t]+Length of Public Type Names Info
>> FAIL: gcc.dg/pubtypes-3.c scan-assembler long+[ \\t]+
On 06/21/2012 02:48 PM, Steven Bosscher wrote:
Index: config/microblaze/microblaze.h
===
--- config/microblaze/microblaze.h (revision 188855)
+++ config/microblaze/microblaze.h (working copy)
@@ -743,8 +743,6 @@ extern int
On Wed, Jun 6, 2012 at 12:19 PM, Edmar wrote:
> On 06/06/2012 08:57 AM, David Edelsohn wrote:
>>
>> On Tue, Jun 5, 2012 at 4:21 PM, Edmar wrote:
>>>
>>> David, Michael,
>>>
>>> Here is the new type "popcnt" patch that I had separated from previous
>>> E5500/E6500 submission, also added the change
On Fri, Jun 22, 2012 at 2:39 AM, Richard Guenther
wrote:
> On Fri, Jun 22, 2012 at 11:29 AM, Jason Merrill wrote:
>> On 06/22/2012 01:30 AM, Richard Guenther wrote:
What other issues? It enables more potential code motion, but on the
other hand, causes more conservative stack reuse
Hi Tobias,
>> Updated patch: After fixing two small errors, the patch also fixes PR
>> 47710 (test case added).
>
> Thanks for the patch! It is okay, but I have a few nits:
Thanks for the review!
> + by this test. This subroutine implements rule 1 of section
> + F03:16.2.3 in the Fortran 95
On 06/21/2012 12:48 AM, Tristan Gingold wrote:
> 2012-06-18 Tristan Gingold
>
> * config/i386/winnt.c (i386_pe_seh_end_prologue): Move code to ...
> (seh_cfa_adjust_cfa): ... that function.
> (seh_emit_stackalloc): Do not emit out of range values.
> * config/i386/i386.md
On Tue, Jun 19, 2012 at 11:36 AM, Richard Henderson wrote:
> On 2012-06-16 04:19, Eric Botcazou wrote:
>>> @@ -179,7 +179,11 @@ extern const unsigned char
>>> mode_class[NUM_MACHINE_MODES];
>>>
>>> extern CONST_MODE_SIZE unsigned char mode_size[NUM_MACHINE_MODES];
>>> #define GET_MODE_SIZE(MODE)
On Wed, May 30, 2012 at 11:13 AM, H.J. Lu wrote:
> On Fri, May 25, 2012 at 6:53 AM, H.J. Lu wrote:
>> On Sun, May 20, 2012 at 7:47 AM, H.J. Lu wrote:
>>> Hi,
>>>
>>> This patch allows -mpreferred-stack-boundary=3 on x86-64 when SSE is
>>> disabled. Since this option changes ABI, I also added a
On 15 Jun 2012, at 14:31, Iain Sandoe wrote:
> When building for, say, mips-linux-gnu, the build of objects from fixed-bit.c
> produces a lot of "set but not used" warnings for min_high & min_low.
>
> looking at the code, these appear to be genuine.
> Fixed as below.
>
> OK for trunk?
> Iain
>
This bumps the requirement to enable Graphite to using cloog 0.17.0
which is the last release from upstream. The patch removes the
support for the legacy cloog versions, too.
I am bootstrapping and testing this now with cloog 0.17.0 built
against the upstream ISL 0.10 version.
If this ends up b
On 06/22/2012 01:38 PM, Richard Guenther wrote:
This cherry-picks two fixes from the move-to-isl-and-isl-scheduler
git graphite branch.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Hi Richard,
if this patches still pass tests, it would be great to see them committed.
Tobi
Hello,
While working on mips recently, I noticed that all the execute tests fail for
simulator.
This appears to be caused by an oversight in the move from gcc/config =>
libgcc, where t-elf defined extra parts including crt{begin,end}.o but these
have been omitted from the re-built libgcc/confi
ping.
On 15 Jun 2012, at 14:31, Iain Sandoe wrote:
> Hi,
>
> When building for, say, mips-linux-gnu, the build of objects from fixed-bit.c
> produces a lot of "set but not used" warnings for min_high & min_low.
>
> looking at the code, these appear to be genuine.
> Fixed as below.
>
> OK for
On Fri, Apr 27, 2012 at 9:43 AM, H.J. Lu wrote:
> On Fri, Apr 27, 2012 at 3:01 AM, Paolo Bonzini wrote:
>> This patch teaches phiopt to look at phis whose arguments are -1 and 0,
>> and produce negated setcc statements.
>>
>> Bootstrapped/regtested x86_64-pc-linux-gnu, together with the patch
>>
Jason,
I have reverted your patch and the failures
FAIL: g++.dg/debug/dwarf2/static-data-member2.C -std=gnu++98
scan-assembler-not DW_TAG_enumerator
FAIL: g++.dg/debug/dwarf2/static-data-member2.C -std=gnu++98
scan-assembler-not DW_TAG_enumeration_type
FAIL: g++.dg/debug/dwarf2/static-data
On Fri, 2012-06-22 at 10:44 +0200, Richard Guenther wrote:
> On Thu, 21 Jun 2012, William J. Schmidt wrote:
>
> > As promised, this breaks out the changes to the IVOPTS cost model and
> > the added function in double-int.c. Please let me know if you would
> > rather see me attempt to consolidate
This cherry-picks two fixes from the move-to-isl-and-isl-scheduler
git graphite branch.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-06-22 Richard Guenther
Merge from graphite branch
2011-08-10 Sebastian Pop
* graphite-sese-to-p
On Fri, 22 Jun 2012, Hans-Peter Nilsson wrote:
> > > > Fix configure test for "stack protector support in target C
> > > > library".
> > > > * configure.ac (test_prefix, test_exec_prefix): Move setting
> > > > from
> > > > inside sysroot handling to before and outside it.
On 20 June 2012 12:29, Julian Brown 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 fo
On Thu, 21 Jun 2012, Richard Guenther wrote:
>
> This fixes an annoying thing on release branches - the gcov
> version changes from 407p to 407* at the single revisions that
> are releases and then back to 407p. That makes it impossible
> to consume profile-feedback data across the revisions on
> I've heard reports of new test failures due to this patch:
>
> FAIL: gcc.dg/pubtypes-2.c scan-assembler long+[ \\t]+0x6a+[ \\t]+[#;]+[
> \\t]+Length of Public Type Names Info
> FAIL: gcc.dg/pubtypes-3.c scan-assembler long+[ \\t]+0x6a+[ \\t]+[#;]+[
> \\t]+Length of Public Type Names Info
> FAIL
On Fri, Jun 22, 2012 at 11:29 AM, Jason Merrill wrote:
> On 06/22/2012 01:30 AM, Richard Guenther wrote:
>>>
>>> What other issues? It enables more potential code motion, but on the
>>> other hand, causes more conservative stack reuse. As far I can tell,
>>> the handling of temporaries is added in
This (r188876) breaks bootstrap:
see http://gcc.gnu.org/ml/gcc-regression/2012-06/msg00311.html
/export/gnu/import/git/gcc-test-intel64corei7/bld/./prev-gcc/g++
-B/export/gnu/import/git/gcc-test-intel64corei7/bld/./prev-gcc/
-B/usr/local/x86_64-unknown-linux-gnu/bin/ -nostdinc++
-B/export/gnu/i
On 06/21/2012 11:18 AM, Sterling Augustine wrote:
Committed as attached. Thanks for your reviews.
I've heard reports of new test failures due to this patch:
FAIL: gcc.dg/pubtypes-2.c scan-assembler long+[ \\t]+0x6a+[ \\t]+[#;]+[
\\t]+Length of Public Type Names Info
FAIL: gcc.dg/pubtypes-3.c
On 06/22/2012 01:30 AM, Richard Guenther wrote:
What other issues? It enables more potential code motion, but on the
other hand, causes more conservative stack reuse. As far I can tell,
the handling of temporaries is added independently after the clobber
for scoped variables are introduced. This
Steve Ellcey writes:
> That is basically what I am trying to do, build a GCC cross compiler for
> the mips-linux-gnu target that can support both ABI's/architectures that
> have synci and ones that don't.
>
> I haven't verified this but if I build a GCC for the mips64-linux-gnu
> target and config
"Joseph S. Myers" writes:
> Since I updated texinfo.tex, update_web_docs_svn has produced errors that
> don't make any sense to me.
Most likely due to this missing change:
2012-06-05 Karl Berry
* doc/texinfo.tex (\txiescapepdf): check \pdfescapestring
against \thisisundefin
> Does this make sense to you?
I cannot answer the question, but the patch survived clean bootstrap on
x86_64-apple-darwin10 and powerpc-apple-darwin9 and a full regtest
on the former with the following new failures between r188841 and
r188858:
FAIL: gcc.dg/pubtypes-2.c scan-assembler long+[ \\t]
On 18 June 2012 14:04, Ramana Radhakrishnan
wrote:
> Hi,
>
> This patch following on from the fix for turning on __builtin_shuffle
> for c++ , enables folding of vec_perm_exprs in the front-end for
> constexpr and constructor style values. I was originally going to go
> with Marc's patch but then
On Thu, 21 Jun 2012, William J. Schmidt wrote:
> As promised, this breaks out the changes to the IVOPTS cost model and
> the added function in double-int.c. Please let me know if you would
> rather see me attempt to consolidate the IVOPTS logic into expmed.c per
> Richard H's suggestion.
If we s
On Thu, Jun 21, 2012 at 8:27 PM, Xinliang David Li wrote:
> On Thu, Jun 21, 2012 at 2:21 AM, Richard Guenther
> wrote:
>> On Thu, Jun 21, 2012 at 7:28 AM, Xinliang David Li
>> wrote:
>>> I modified the documentation and it now looks like this:
>>>
>>> @item -ftemp-stack-reuse
>>> @opindex ftemp
On Thu, Jun 21, 2012 at 5:53 PM, Michael Matz wrote:
> Hi,
>
> On Thu, 21 Jun 2012, Richard Guenther wrote:
>
>> The flag is not restricted to the C++ compiler and applies to all
>> automatic variables.
>
> The use of that flag in the gimplifier (->in_cleanup_expr) makes it
> actually c++ specific
On Thu, Jun 21, 2012 at 10:58:05PM -0300, Alexandre Oliva wrote:
> for gcc/ChangeLog
> from Alexandre Oliva
>
> * var-tracking.c (vt_add_function_parameter): Use a preserved
> VALUE for the MEM address of an incoming parameter.
Okay.
Jakub
On Jun 21, 2012, at 8:19 PM, Richard Henderson wrote:
> On 2012-06-21 00:48, Tristan Gingold wrote:
>> @@ -9142,9 +9152,12 @@ ix86_compute_frame_layout (struct ix86_frame *frame)
>> {
>> HOST_WIDE_INT diff;
>>
>> - /* If we can leave the frame pointer where it is, do so. */
>> +
On 06/17/2012 11:22 PM, Janus Weil wrote:
Updated patch: After fixing two small errors, the patch also fixes PR
47710 (test case added).
Thanks for the patch! It is okay, but I have a few nits:
+ by this test. This subroutine implements rule 1 of section
+ F03:16.2.3 in the Fortran 95 stan
This fixes the PR by making the is_fortran predicate functional on Darwin too.
Bootstrapped/regtested on Darwin by Dominique (thanks!), pre-approved by Jakub
and applied on the mainline.
2012-06-22 Eric Botcazou
PR debug/53704
* dwarf2out.c (gen_compile_unit_die): Use DW_LAN
68 matches
Mail list logo
