Hello world,
the attached patch fixes the PR by converting
if (foo) then
...
else if (bar) then
...
end if
to if (foo) then
else
if (bar) then
end if
end if
so inserting a block for temporary variables around the converted
if statement works.
OK for trunk?
Thomas
2012-01-29 Tho
I wrote:
OK for trunk?
I forgot: Regression-tested.
Thomas
Georg-Johann Lay writes:
> This patch fixes PR51374 by more strictly updating mem_last_set.
> Sloppy handling of mem_last_set can lead to error in volatile correctness
> because combine then is allowed to drag one volatile access over an other
> volatile thing (volatile access, asm volatile, unspe
The patch fixes a build error. The function was accidentally added Rev.
183620 of 2012-01-27 for PR 52016.
The function is part of the (uncommitted) draft patch for PR 50981.
Committed as Rev. 183675.
Tobias
I forgot to attach the committed patch ...
Tobias Burnus wrote:
The patch fixes a build error. The function was accidentally added
Rev. 183620 of 2012-01-27 for PR 52016.
The function is part of the (uncommitted) draft patch for PR 50981.
Committed as Rev. 183675.
Tobias
Index: ChangeLog
===
> As discussed in Bugzilla, this is the patch implementing Paolo's suggestion
> of killing REG_EQUAL/REG_EQUIV notes from df_kill_notes. The code assumes
> there is at most one such note per insn.
That's wrong though and wreaks havoc during reload, e.g.:
(insn 169 60 62 4 (set (reg:TF 158)
Hello,
this patch adds thiscall-call feature via libffi for 32-bit Windows
native target.
ChangeLog
2012-01-29 Kai Tietz
PR target/51500
* interpret.cc (_Jv_init_cif): Handle thiscall
convention for 32-bit Windows.
* java/lang/reflect/natMethod.cc (_Jv_CallAny
On Sat, 28 Jan 2012, Sandra Loosemore wrote:
> 2012-01-28 Sandra Loosemore
>
> gcc/
> * doc/invoke.texi: Make usage of "compile time" and
> "run time"/"runtime" consistent throughout the file.
OK. Could you post a patch to codingconventions.html to document the
conventions
> > As discussed in Bugzilla, this is the patch implementing Paolo's
> > suggestion of killing REG_EQUAL/REG_EQUIV notes from df_kill_notes. The
> > code assumes there is at most one such note per insn.
>
> That's wrong though and wreaks havoc during reload, e.g.:
>
> (insn 169 60 62 4 (set (reg:T
> [ BTw, I think the MIPS comment is wrong. HARD_REGNO_NREGS (fpreg, SFmode)
> is (now) 1, even when using paired FPRs. And the suggestion doesn't
> seem at all ideal to me. OK to remove? ]
You're the MIPS maintainer, so it's up to you.
> A simple fix, which I've used below, is to make sur
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 Wed, Jan 18, 2012 at 14:45, Mikael Morin wrote:
> On 18.01.2012 10:12, Tobias Burnus wrote:
>>
>>
>> Dear Janne,
>>
>>> the attached patch changes the binding labels that are needed for
>>> bind(C) symbols to be heap allocated rather than, as currently, being
>>> fixed size arrays of size 127 (
On 1/28/12 5:00 PM, Gabriel Charette wrote:
Why do you need two functions: pph_stream_close and
pph_stream_close_no_flush only to call pph_stream_close_1 with a
different flag? Seems like leaving
pph_stream_close_1 as pph_stream_close and simply making the calls
themselves with true/false flags i
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the Russian team of translators. The file is available at:
http://translationproject.org/latest/cpplib/ru.po
(This file, 'cpplib-4.7-b20120128
cpplib-4.7-b20120128.ru.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
On Fri, Jan 20, 2012 at 12:39 AM, Ian Lance Taylor wrote
>> Attached patch adds -mieee to tests that need full IEEE compliance to
>> pass. While working on patch, I have noticed that go-test.exp doesn't
>> pass DEFAULT_GOCFLAGS to go_target_compile procedure in expected
>> format (so, these simpl
Dear all,
for derived type assignment one needs to call "vtab->_copy" for
polymorphic components. Currently, this does not happen.
The attached draft patch adds support for this, fixing the
gfortran.dg/class_allocate_12.f90 test case (with STOP removed),
chapter07/strategy_surrogate_f2003 of
cpplib-4.7-b20120128.fi.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the Finnish team of translators. The file is available at:
http://translationproject.org/latest/cpplib/fi.po
(This file, 'cpplib-4.7-b20120128
Dear Tobias,
I cry foul at this point :-) I have gone gallivanting off to try to
fix really horrid regressions like 52012, whilst you are have fun
doing interesting things.
Pah! Good call - OK for trunk
Thanks for the patch.
I notice that you are making good use of recent additions to
tra
Eric Botcazou writes:
>> [ BTw, I think the MIPS comment is wrong. HARD_REGNO_NREGS (fpreg, SFmode)
>> is (now) 1, even when using paired FPRs. And the suggestion doesn't
>> seem at all ideal to me. OK to remove? ]
>
> You're the MIPS maintainer, so it's up to you.
OK, I'll remove it.
>>
[ Nick, you might remember mentioning this bug a few months back.
I think I've finally got a proper fix, rather than the failed
attempt I originally sent. ]
2005-12-20 Richard Guenther
PR middle-end/24306
* builtins.c (std_gimplify_va_arg_expr): Do not align
va fram
Dear Paul,
thanks for the review!
I cry foul at this point :-) I have gone gallivanting off to try to
fix really horrid regressions like 52012, whilst you are have fun
doing interesting things.
Well, that way my revenge for not getting a review for my
default-initializer patch since ove
Hi,
the attached patch reduces the size of the pointer_info tree used when
reading and writing module files by making the module and symbol names
pointers rather than arrays. As the strings are already put into heap
memory and resized to their correct size during parsing, we have
already paid the
Dear Tobias,
> Well, that way my revenge for not getting a review for my
> default-initializer patch since over a week ... ;-)
To be perfectly frank, I was hoping that somebody else would get their
ass in gear :-)
I'll do it.
Paul
Dear Tobias,
As I said just a moment ago - I was hoping somebody else would get
involved. I looked at this earlier today and it is OK for trunk.
Thanks for the patch.
Cheers
Paul
On Sat, Jan 28, 2012 at 3:09 PM, Dominique Dhumieres wrote:
> Dear Tobias,
>
> I have this patch in my working tr
On 01/30/2012 05:22 AM, Uros Bizjak wrote:
> 2012-01-29 Uros Bizjak
>
> * config/alpha/alpha.c (alpha_option_overrride): Default to
> full IEEE compliance mode for Go language.
I'm not keen on this, but I also don't have an alternative to suggest.
Ok.
r~
On 01/29/2012 07:57 AM, Tom de Vries wrote:
> Richard,
>
> [now cc-ing gcc-patches]
>
> this patch fixes PR50283 in a target-independent way.
>
> it asserts on frame-related insns in the delay slot of insns that can throw,
> and prevents the assert by testing for the same condition in
> eligible
On 01/27/2012 06:19 PM, Dodji Seketeli wrote:
+ /* BINDING_VALUE must be a template parm. */
+ if (binding_value == NULL_TREE
+ && (!DECL_P (binding_value)
+ || !DECL_TEMPLATE_PARM_P (binding_value)))
+return false;
I think the && should be || here.
OK with that change.
J
> I'm not sure REGNO_REG_CLASS is meaningful here, because many ports define
> subclasses of the natural architectural classes. E.g. MIPS has MD0_REG
> and MD1_REG for HI and LO (which is which depends on endianness), even
> though both are logically the same in terms of architectural class and
>
OK, just please add a comment about why we aren't using locate_ctor.
Jason
On Sun, 29 Jan 2012, Janne Blomqvist wrote:
> Taking into account the suggestions by Tobias and Mikael, attached is
> the patch I just committed. Thanks for the reviews!
I am now seeing the following bootstrap error on all my FreeBSD testers,
and this does not appear to be operating system-specif
On Sun, Jan 29, 2012 at 23:36, Gerald Pfeifer wrote:
> On Sun, 29 Jan 2012, Janne Blomqvist wrote:
>> Taking into account the suggestions by Tobias and Mikael, attached is
>> the patch I just committed. Thanks for the reviews!
>
> I am now seeing the following bootstrap error on all my FreeBSD tes
Gerald Pfeifer wrote:
On Sun, 29 Jan 2012, Janne Blomqvist wrote:
Taking into account the suggestions by Tobias and Mikael, attached is
the patch I just committed. Thanks for the reviews!
I am now seeing the following bootstrap error on all my FreeBSD testers,
and this does not appear to be ope
This is the first of a few patches to update GCC's documentation with
the proposed conventions in the GCC Development Conventions document
that Joseph and I published last year
(https://docs.google.com/a/google.com/document/pub?id=10LO8y0YhjlKHya_PKM3jEGrJu0rllv-Nc9qP5LXqH_I#h.qpg2rjcas9fw)
Janne Blomqvist wrote:
Have you tried r183679, which should fix this?
Aha, I missed the follow up patch - that explains why I didn't see it.
While I can speak only for myself, regtesting on x86_64-unknown-linux-gnu. For
some reason this was/is flagged as a warning for me, not an error, so I
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the German team of translators. The file is available at:
http://translationproject.org/latest/cpplib/de.po
(This file, 'cpplib-4.7-b20120128.
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the German team of translators. The file is available at:
http://translationproject.org/latest/cpplib/de.po
(This file, 'cpplib-4.7-b20120128.
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 German team of translators. The file is available at:
http://translationproject.org/latest/gcc/de.po
(This file, 'gcc-4.7-b20120128.de.po', h
On Fri, 20 Jan 2012, Georg-Johann Lay wrote:
> Adding AVR-specific release notes to wwwdocs/htdocs/gcc-4.7/changes.html
Index: changes.html
===
+The AVR port's libgcc has been improved and its multilib structure
+ has been en
See PR. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
2012-01-29 John David Anglin
PR testsuite/51875
dg-require-visibility needs an argument. Tested on hppa2.0w-hp-hpux11.11
and hppa64-hp-hpux11.11.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
2012-01-29 John David Anglin
Work around limited common alignment. Tested on hppa2.0w-hp-hpux11.11
and hppa64-hp-hpux11.11.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
2012-01-29 John David Anglin
> On 01/29/2012 07:57 AM, Tom de Vries wrote:
> > Richard,
> >
> > [now cc-ing gcc-patches]
> >
> > this patch fixes PR50283 in a target-independent way.
> >
> > it asserts on frame-related insns in the delay slot of insns that can
> > throw, and prevents the assert by testing for the same condi
As discussed in PR c++/51910, my patch from last summer
http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01368.html
to make HAVE_LD_DEMANGLE the default when using GNU ld exposed a bug in
collect2's link-time -frepo handling -- it depends on the linker passing
back mangled names of undefined symbo
On 01/30/2012 11:07 AM, Paul Brook wrote:
> However this is a issue of debug experiance rather than code correctness. My
> guess is most architectures don't allow you to singlestep into delay slots,
> so
> any use of delay slots may cause lossage[1], not just frame related ones.
Certainly Sp
[Ooops, resending to include the attachment, this time]
As discussed in PR c++/51910, my patch from last summer
http://gcc.gnu.org/ml/gcc-patches/2011-06/msg01368.html
to make HAVE_LD_DEMANGLE the default when using GNU ld exposed a bug in
collect2's link-time -frepo handling -- it depends
On 01/30/2012 11:07 AM, Paul Brook wrote:
> It's worth noting that -fasync-unwind-tables does not guarantee
> backtrace/unwind from arbitrary points. Only from those instructions that
> may
> result in a synchronous trap (which matches the semantics of this patch).
... and we're not talking ab
On 1/29/2012 3:40 PM, Richard Henderson wrote:
On 01/30/2012 05:22 AM, Uros Bizjak wrote:
2012-01-29 Uros Bizjak
* config/alpha/alpha.c (alpha_option_overrride): Default to
full IEEE compliance mode for Go language.
I'm not keen on this, but I also don't have an alternative t
On 01/29/2012 07:31 AM, Joseph S. Myers wrote:
On Sat, 28 Jan 2012, Sandra Loosemore wrote:
2012-01-28 Sandra Loosemore
gcc/
* doc/invoke.texi: Make usage of "compile time" and
"run time"/"runtime" consistent throughout the file.
OK. Could you post a patch to coding
> On 01/30/2012 11:07 AM, Paul Brook wrote:
> > It's worth noting that -fasync-unwind-tables does not guarantee
> > backtrace/unwind from arbitrary points. Only from those instructions
> > that may result in a synchronous trap (which matches the semantics of
> > this patch).
>
> ... and we're not
On 01/29/2012 04:09 PM, Eric Botcazou wrote:
As discussed in Bugzilla, this is the patch implementing Paolo's
suggestion of killing REG_EQUAL/REG_EQUIV notes from df_kill_notes. The
code assumes there is at most one such note per insn.
That's wrong though and wreaks havoc during reload, e.g.:
53 matches
Mail list logo
