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
I wrote:
OK for trunk?
I forgot: Regression-tested.
Thomas
Georg-Johann Lay a...@gjlay.de 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
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 kti...@redhat.com
PR target/51500
* interpret.cc (_Jv_init_cif): Handle thiscall
convention for 32-bit Windows.
*
On Sat, 28 Jan 2012, Sandra Loosemore wrote:
2012-01-28 Sandra Loosemore san...@codesourcery.com
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
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)
[ 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 sure
Hello, gentle maintainer.
This is a message from the Translation Project robot. (If you have
any questions, send them to coordina...@translationproject.org.)
A new POT file for textual domain 'cpplib' has been made available
to the language teams for translation. It is archived as:
On Wed, Jan 18, 2012 at 14:45, Mikael Morin mikael.mo...@sfr.fr 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
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.fi.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
coordina...@translationproject.org
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,
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
Eric Botcazou ebotca...@adacore.com 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
[ 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 rguent...@suse.de
PR middle-end/24306
* builtins.c (std_gimplify_va_arg_expr): Do not
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
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 domi...@lps.ens.fr wrote:
Dear Tobias,
I have this patch
On 01/30/2012 05:22 AM, Uros Bizjak wrote:
2012-01-29 Uros Bizjak ubiz...@gmail.com
* 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
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.
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
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,
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,
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 dave.ang...@nrc-cnrc.gc.ca
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 condition in
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
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 Sparc
[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
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 about
On 1/29/2012 3:40 PM, Richard Henderson wrote:
On 01/30/2012 05:22 AM, Uros Bizjak wrote:
2012-01-29 Uros Bizjakubiz...@gmail.com
* 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
On 01/29/2012 07:31 AM, Joseph S. Myers wrote:
On Sat, 28 Jan 2012, Sandra Loosemore wrote:
2012-01-28 Sandra Loosemoresan...@codesourcery.com
gcc/
* doc/invoke.texi: Make usage of compile time and
run time/runtime consistent throughout the file.
OK. Could you post
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
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.:
44 matches
Mail list logo