2013/2/7 Georg-Johann Lay :
> This patch adds more fixed-point features from embedded-C:
>
> - roundfx rounding functions
> - countlsfx bit counting functions
> - add implementations of builtins so that an implementation / symbol is
> available when the address of such a function is taken
>
> The
The attached revised patch eliminates the failure of the libitm.c++/eh-1.C
execution test on
darwin10 and later, PR55693. With the introduction of c++ weak-symbol
coalescing in Mac OS X 10.6,
dyld no longer looks up weak symbols that reside in object files rather than
shared libraries. This
p
On Thu, Feb 7, 2013 at 2:43 PM, Uros Bizjak wrote:
>
> I did hit one new error that seems related:
>
> --- FAIL: TestChtimes (0.00 seconds)
> os_test.go:681: AccessTime didn't go backwards;
> was={63495872497 0 47130825733376}, after={63495872497 0
> 47130825733376}
> os_test.go:685:
From: Jakub Jelinek
Date: Fri, 8 Feb 2013 01:27:02 +0100
> +void
> +fn (void)
> +{
> + if (b)
> +{
> + int *p, *q;
> + char g;
> +
> + if (f++)
> + for (;; e++);
> +lbl:
> + for (b = 0; b < 2; b++)
> + t /= d + 1 ? : i || a < c < (d = f) ? : 1 | (j = 2);
> +
>
On Thu, Feb 7, 2013 at 5:34 PM, Michael Meissner
wrote:
> The following patch fixes a segmentation fault for the powerpc-none-eabi
> toolchain on the vsx-mass-1.c test. Some of the builtin functions are not
> defined in that environment, and the compiler did not check whether a pointer
> was NULL
Hi!
As the following testcase shows, get_reload_reg could sometimes return
a reused reg in a wrong mode. If curr_insn_input_reloads[i].input
and original have VOIDmode, they could compare equal (same integer
constant), yet mode might be different from
GET_MODE (curr_insn_input_reloads[i].reg).
If
On Thu, Feb 7, 2013 at 6:29 AM, Jason Merrill wrote:
> On 02/06/2013 08:39 PM, Sriraman Tallam wrote:
>>
>> +// Test to check if an error is generated when virtual functions
>> +// are multiversioned.
>
>
> This seems like a TODO, rather than something to test for. I don't see any
> reason why we
From: Jakub Jelinek
Date: Thu, 7 Feb 2013 20:43:32 +0100
> This and earlier patch are ok, if it bootstraps/regtests fine, and suitable
> ChangeLog entry is provided.
> Running gdb testsuite before and after wouldn't hurt though.
I've done all of this, and committed to trunk and the gcc-4.7 branc
On Thu, Feb 07, 2013 at 11:01:13PM +0100, Steven Bosscher wrote:
> FWIW, left trunk vs. LRA right (with PR54009 patch on rs6000.c):
[snip]
>
> I don't speak POWER but perhaps you can make sense of it :-)
No real difference in "r" function, and interesting that LRA does a
better job of "w", about
On Thu, Feb 7, 2013 at 5:27 PM, Ian Lance Taylor wrote:
> On Thu, Feb 7, 2013 at 2:26 AM, Uros Bizjak wrote:
>>
>> Thanks, I have tested it on CentOS 5.9. However, for some reason, I
>> got unrelated linker errors that prevents further testing:
>>
>> === libgo tests ===
>>
>> ./a.
The following patch fixes a segmentation fault for the powerpc-none-eabi
toolchain on the vsx-mass-1.c test. Some of the builtin functions are not
defined in that environment, and the compiler did not check whether a pointer
was NULL before using it. I also fixed the test so that it would only ru
Hello,
this is the last remaining patch to fix an infinite recursion when
creating the middle-end type of a function having a dummy
procedure whose interface is the function itself.
This patch is a slight variation of the comment 4 fix that was attached
to the PR. It sets the procedure's backend_
On Thu, Feb 7, 2013 at 2:15 PM, Uros Bizjak wrote:
>
> Just a small patch to remove a compile warning.
>
> Tested on x86_64-pc-linux-gnu.
Committed.
Thanks.
Ian
> Index: runtime/goc2c.c
> ===
> --- runtime/goc2c.c (revision 1
Hello!
Just a small patch to remove a compile warning.
Tested on x86_64-pc-linux-gnu.
Uros.
Index: runtime/goc2c.c
===
--- runtime/goc2c.c (revision 195856)
+++ runtime/goc2c.c (working copy)
@@ -633,7 +633,7 @@
sys
On Thu, Feb 7, 2013 at 4:09 AM, Alan Modra wrote:
> After fixing PR54009 (again), I thought I'd take a look at why reload
> is generating the following correct but poor code
>
> stw 10,8(1)
> stw 11 12(1)
> ...
> lfd 0,8(1)
> stfd 0,x+32764@l(9)
>
> rather than
>
> addi 9,x+32764@l(9)
> ...
PR 56173 points out a number of checksum failures that turn out to be
related to a bug in the crypto/md5 package: some optimizations had been
added that make it only work correctly on little-endian systems. This
patch corrects the problem. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux
Hi!
build_constructor assumes the elts it is called with have all non-NULL
value, otherwise one shouldn't add those elts at all.
The following patch makes sure that we don't push elts with NULL value
into the ctor element vectors.
Bootstrapped/regtested on x86_64-linux and i686-linux, approved b
From: David Miller
Date: Thu, 07 Feb 2013 14:38:18 -0500 (EST)
> From: Jakub Jelinek
> Date: Thu, 7 Feb 2013 18:22:32 +0100
>
>> Then supposedly somewhere in dwarf2out we do some adjustment,
>> but still end up with d/e loclist of:
>> .LLST2:
>> .uaxword.LVL0-.Ltext0 ! Locatio
On Thu, Feb 07, 2013 at 02:38:18PM -0500, David Miller wrote:
> The fix for this is trivial, just a missing leaf renumbering in dwarf2out.c:
>
> diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c
> index 06cfb18..765d5c5 100644
> --- a/gcc/dwarf2out.c
> +++ b/gcc/dwarf2out.c
> @@ -10864,7 +10864,16 @@
From: Jakub Jelinek
Date: Thu, 7 Feb 2013 18:22:32 +0100
> Then supposedly somewhere in dwarf2out we do some adjustment,
> but still end up with d/e loclist of:
> .LLST2:
> .uaxword.LVL0-.Ltext0 ! Location list begin address
> (*.LLST2)
> .uaxword.LVL1-.Ltext0
Vladimir,
On 25/01/13 16:36, Vladimir Makarov wrote:
> On 01/25/2013 08:05 AM, Tom de Vries wrote:
>> Vladimir,
>>
>> this patch adds analysis of register usage of functions for usage by IRA.
>>
>> The patch:
>> - adds analysis in pass_final to track which hard registers are set or
>> clobbered
>
On Feb 7, 2013, at 7:51 AM, Kyrylo Tkachov wrote:
> Ok to put it in with the other two ARM rounding vectorization patches that
> have been okayed for stage1?
Ok.
On Feb 7, 2013, at 3:49 AM, Richard Biener wrote:
> On Thu, Feb 7, 2013 at 11:51 AM, Alan Modra wrote:
>> On Thu, Feb 07, 2013 at 10:11:02AM +0100, Richard Biener wrote:
>>> On Thu, Feb 7, 2013 at 9:45 AM, Alan Modra wrote:
I think this one counts as obvious, but I'll ask for permission any
This is hopefully the final nail in the coffin for PR target/50678, which is
the failure of ACATS c52104y at -O2 on x86-64/Darwin. The root cause was
tracked down to a bug in the unwind info of the system sigtramp and a ticket
was opened with Apple (radar #10302855). Pending its resolution, a
On Thu, Feb 7, 2013 at 9:11 AM, Andreas Schwab wrote:
> Ian Lance Taylor writes:
>
>> +dnl Test whether we need to use DejaGNU or whether we can use the
>> +dnl simpler gotest approach. We can only use gotest for a native
>> +dnl build.
>> +USE_DEJAGNU=no
>> +case ${host} in
>> + *-*-rtems*) US
Hi!
As discussed in the PR, () isn't valid unary expression, so
((type ()) () needs to be parsed as a function call instead.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
approved by Jason in the PR, committed to trunk.
2013-02-07 Jakub Jelinek
PR c++/56239
Hi!
If DECL_LANG_SPECIFIC is created for a local var due to other reason
than push_local_name, it might not have u.base.u2sel bit set and
so DECL_DISCRIMINATOR can't be used on it (instead, should be assumed to be
0).
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
approved b
When bootstrapping with BOOT_FLAGS='-O3 -g', I get:
/home/polacek/src/gcc/gcc/c/c-parser.c: In function ‘c_expr
c_parser_postfix_expression_after_primary(c_parser*, location_t, c_expr)’:
/home/polacek/src/gcc/gcc/c/c-parser.c:6908:16: error: ‘origtypes’ may be used
uninitialized in this function
On Wed, Feb 06, 2013 at 03:18:27PM -0500, David Miller wrote:
> From: Eric Botcazou
> Date: Wed, 06 Feb 2013 11:13:30 +0100
>
> > I think testing crtl->uses_only_leaf_regs is sufficient here (and
> > while you're at it, you could also test the value of
> > HAVE_window_save, which can be 0 if -mfl
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56225
The patch was successfully bootstrapped and tested on x86/x86-64.
Committed as rev. 195856.
2013-02-07 Vladimir Makarov
PR rtl-optimization/56225
* lra-constraints.c (process_alt_operands): Check th
Ian Lance Taylor writes:
> +dnl Test whether we need to use DejaGNU or whether we can use the
> +dnl simpler gotest approach. We can only use gotest for a native
> +dnl build.
> +USE_DEJAGNU=no
> +case ${host} in
> + *-*-rtems*) USE_DEJAGNU=yes ;;
> + ${target}) ;;
This is a target library, s
On Thu, Feb 7, 2013 at 8:51 AM, Rainer Orth
wrote:
> Ian Lance Taylor writes:
>
>> On Solaris the libgo networking library uses select. This patch
>> implements some fixes for that code. The Close method is changed to
>> actually do something; this is mainly for testing purposes. More
>> impo
Ian Lance Taylor writes:
> On Solaris the libgo networking library uses select. This patch
> implements some fixes for that code. The Close method is changed to
> actually do something; this is mainly for testing purposes. More
> importantly, when a socket is closed, the select needs to be res
On Thu, Feb 07, 2013 at 02:56:48PM +0100, Richard Biener wrote:
> > + /* Check the headers. */
> > + FOR_EACH_BB (bb)
> > +{
> > + /* Skip BBs in the root tree. */
> > + if (bb->loop_father == current_loops->tree_root)
> > + continue;
>
> You shouldn't need this ... it will mis
On Thu, Feb 07, 2013 at 04:40:56PM +0100, Richard Biener wrote:
>
> This fixes fallout from the loops-exist verifier. And it optimizes
> compile-time of IPA pure-const.
>
> Not yet tested in any way (but quite obvious).
FYI, I've regtested/bootstrapped this one on x86_64, together with my loop
On Thu, Feb 7, 2013 at 2:26 AM, Uros Bizjak wrote:
>
> Thanks, I have tested it on CentOS 5.9. However, for some reason, I
> got unrelated linker errors that prevents further testing:
>
> === libgo tests ===
>
> ./a.out: error while loading shared libraries:
> /home/uros/gcc-build/
Converting a bit-field reference from lvalue to xvalue obscures its
unlowered type, and isn't useful for scalars anyway.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 4351fe77d1cc018b9f84cbc775489336fb295fb5
Author: Jason Merrill
Date: Thu Feb 7 11:06:19 2013 -0500
PR c++/56235
This patch solves a problem with VIEW_CONVERT_EXPR folding for fixed_cst and
that use fixed-value.c:fixed_from_double_int.
The patch sign/zero extends the double_int input according to the requested
fixed-point mode.
The patch bootstraps on x86-linux-gnu and passes testsuite on avr-unknown one.
On Thu, 7 Feb 2013, Vladimir Makarov wrote:
> On 02/07/2013 09:56 AM, Richard Biener wrote:
> > The following gets rid of ira_loops (ok, without touching all uses
> > yet but with a #define - I'll finish this if you are ok with it
> > in general).
> >
> > Bootstrap and regtest running on x86_64-u
Hi all,
This testsuite patch has been okayed at
http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01141.html
but I have not applied it and I'd like to add a couple of things to it.
Currently, add_options_for_arm_v8_neon would explicitly set
-mfloat-abi=softfp which would break
check_effective_target_ar
This fixes fallout from the loops-exist verifier. And it optimizes
compile-time of IPA pure-const.
Not yet tested in any way (but quite obvious).
Richard.
2013-02-07 Richard Biener
* Makefile.in (tree-tailcall.o): Add $(CFGLOOP_H) dependency.
* ipa-pure-const.c (analyze_fun
2013/2/7 Uros Bizjak :
> Hello!
>
> Attached (trivial) patch substitutes a bunch of "ll" conversion types
> with HOST_LONG_LONG_FORMAT defines. Additionally, it fixes wrong usage
> of HOST_WIDE_INT_PRINT_HEX_PURE in the argument of internal_error.
>
> The patch reportedly fixes bootstrap failures o
On 02/07/2013 09:56 AM, Richard Biener wrote:
The following gets rid of ira_loops (ok, without touching all uses
yet but with a #define - I'll finish this if you are ok with it
in general).
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Ok for trunk for the IRA parts? (well, given
The following gets rid of ira_loops (ok, without touching all uses
yet but with a #define - I'll finish this if you are ok with it
in general).
Bootstrap and regtest running on x86_64-unknown-linux-gnu.
Ok for trunk for the IRA parts? (well, given I search & replace
uses of ira_loops)
Thanks,
On 02/07/2013 01:24 AM, Jakub Jelinek wrote:
Hi!
- http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01477.html
PR56154
fix invalid .debug_loc entries if an empty range is emitted
at .Ltext0 symbol
This is good. Please install.
jeff
On 02/07/2013 04:18 AM, Jonathan Wakely wrote:
On 7 February 2013 03:05, Ed Smith-Rowland wrote:
greetings,
On SO someone noticed that g++ still has in the basic_ios class
operator void*();
instead of
explicit operator bool() const;
The old C++98 operator allows things like
std::cout <
On 02/06/2013 08:39 PM, Sriraman Tallam wrote:
+// Test to check if an error is generated when virtual functions
+// are multiversioned.
This seems like a TODO, rather than something to test for. I don't see
any reason why we couldn't support multiversioned virtual functions.
error_at
On Thu, 7 Feb 2013, Marek Polacek wrote:
> On Thu, Feb 07, 2013 at 10:14:14AM +0100, Richard Biener wrote:
> > I'd say "loop with header %d not in loop tree" here. Eventually
> > the header detection should be shared with the code in flow_loops_find
> > to avoid divergence. I'm making changes to
On Thu, Feb 07, 2013 at 10:14:14AM +0100, Richard Biener wrote:
> I'd say "loop with header %d not in loop tree" here. Eventually
> the header detection should be shared with the code in flow_loops_find
> to avoid divergence. I'm making changes to the flow_loops_find code
> at the moment and will
As promised, a bugs.html entry.
Ok?
Thanks,
Richard.
2013-02-07 Richard Biener
* bugs/index.html: Add "Loops do not terminate" case to non-bugs.
Index: index.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/bugs/index.html,v
On Thu, Feb 07, 2013 at 12:49:03PM +0100, Richard Biener wrote:
> It should be possible to add a -msavres-inline switch, right?
Sure, but doesn't rs6000 have enough options already?
Perhaps the best solution would be to move __do_global_ctors_aux out
of crtend.o for targets that support hidden gl
On Thu, Feb 7, 2013 at 1:34 AM, Cary Coutant wrote:
> OK for google/main?
OK.
Diego.
Hello!
Attached (trivial) patch substitutes a bunch of "ll" conversion types
with HOST_LONG_LONG_FORMAT defines. Additionally, it fixes wrong usage
of HOST_WIDE_INT_PRINT_HEX_PURE in the argument of internal_error.
The patch reportedly fixes bootstrap failures on i686-pc-mingw32.
2013-02-07 Uro
On Thu, Feb 7, 2013 at 11:51 AM, Alan Modra wrote:
> On Thu, Feb 07, 2013 at 10:11:02AM +0100, Richard Biener wrote:
>> On Thu, Feb 7, 2013 at 9:45 AM, Alan Modra wrote:
>> > I think this one counts as obvious, but I'll ask for permission anyway.
>> > Bootstrapped etc. powerpc-linux. OK everywhe
This rewrites fix_loop_structure to be equivalent to loop
detection from scratch via flow_loops_find with re-using
of existing loop tree entries.
This addresses a few shortcomings of fix_loop_structure. First,
as shown by the testcase in PR56181, fix_loop_structure does
not handle loop nesting ch
Now committed as obvious (Rev. 195845).
Tobias
Tobias Burnus wrote:
* ping *
http://gcc.gnu.org/ml/fortran/2013-01/msg00223.html
Tobias Burnus:
As promised: A patch for the "regression" PR54339. I am sure there is
room for improvement :-)
Everyone: Please feel free to comment on the current
On Thu, Feb 07, 2013 at 10:11:02AM +0100, Richard Biener wrote:
> On Thu, Feb 7, 2013 at 9:45 AM, Alan Modra wrote:
> > I think this one counts as obvious, but I'll ask for permission anyway.
> > Bootstrapped etc. powerpc-linux. OK everywhere?
>
> Isn't there a way to just disable the out-of-lin
On Tue, Feb 5, 2013 at 9:00 PM, Ian Lance Taylor wrote:
> On Thu, Jan 3, 2013 at 1:26 AM, Uros Bizjak wrote:
>> Hello!
>>
>>> I've committed a patch to upgrade libgo to the current version of the
>>> master Go library. As usual, this mail message only includes the
>>> changes to files that are s
As described in the PR, g++.dg/debug/dwarf2/thunk1.C was failing on
Solaris/x86 (both 32 and 64-bit), as well as 64-bit Linux/i686. The
following patch implements Jason's suggestions in the PR.
Tested with the appropriate runtest invocations on i386-pc-solaris2.10,
i686-unknown-linux-gnu, and x86
On Thu, 7 Feb 2013, Jakub Jelinek wrote:
> On Tue, Feb 05, 2013 at 04:51:36PM +0100, Jakub Jelinek wrote:
> > On Tue, Feb 05, 2013 at 04:27:15PM +0100, Jan Hubicka wrote:
> > > I managed to get an accidental commit on those two testcases without an
> > > ChangeLog entry.
> > > I just commit the m
On Tue, Feb 05, 2013 at 04:51:36PM +0100, Jakub Jelinek wrote:
> On Tue, Feb 05, 2013 at 04:27:15PM +0100, Jan Hubicka wrote:
> > I managed to get an accidental commit on those two testcases without an
> > ChangeLog entry.
> > I just commit the missing ChangeLog and the following patch that should
On Wed, 6 Feb 2013, Marek Polacek wrote:
> This patch extends verify_loop_structure by checking that header's
> are really its own headers (this proved as useful in PR56181).
> The bulk of the code is taken from flow_loops_find.
>
> Bootstrapped on x86_64 linux. The only fallout now is (for C/C+
On Thu, Feb 7, 2013 at 9:45 AM, Alan Modra wrote:
> I think this one counts as obvious, but I'll ask for permission anyway.
> Bootstrapped etc. powerpc-linux. OK everywhere?
Isn't there a way to just disable the out-of-line register save/restore
functions? Adding -O2 looks odd.
On a related no
I think this one counts as obvious, but I'll ask for permission anyway.
Bootstrapped etc. powerpc-linux. OK everywhere?
PR target/45053
* config/rs6000/t-crtstuff (CRTSTUFF_T_CFLAGS): Add -O2.
Index: libgcc/config/rs6000/t-crtstuff
Hi!
- http://gcc.gnu.org/ml/gcc-patches/2013-01/msg01477.html
PR56154
fix invalid .debug_loc entries if an empty range is emitted
at .Ltext0 symbol
Jakub
Hi!
As described in the PR, when the ppc64 indirect aix call patterns use
ld insn, we need to use "Y" constraint for the memory operand instead
of "m", otherwise the offset might not be multiple of 4.
Fixed thusly, approved by David in the PR, bootstrapped/regtested
on powerpc-linux and powerpc64
66 matches
Mail list logo
