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
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
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 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 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 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, 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 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
> 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]
"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
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
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
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
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 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
> 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 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
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 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.
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, 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
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, 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
>>
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
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
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
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 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
>
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 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 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
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 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
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 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 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/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
> >>> 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
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]
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 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: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 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
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-
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
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
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
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-
> >
>
> 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 'cpplib' has been made available
to the language teams for translation. It is archived as:
http://translationproject.org/POT-files/c
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
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
>> -
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
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
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 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
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 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]+
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
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
> 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
> 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
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
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
>
> 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 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
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
68 matches
Mail list logo
