Dear All,
It is only now that I see that my mail to Mikael and the release
managers, to say that I would commit, bounced because of excess MIME
content. I apologise for that. I can only say in mitigation that
fortran is not release critical and regressions are unlikely because
of the conditions
> Of course this matters only to >64bit (i.e. >registersize) values like
> TImode, alias __int128. The problem here is that group-loading a
> constant for a function return-value doesn't work; it's passed to
> simplify_gen_subreg which horks on the VOIDmode constant. Thankfully,
> the code below
Hello!
> The following proposed patch fixed the problem. Note that for Atom,
> only the CWD instruction is slow with 5 cycle latency, the rest sign
> extension instructions are fast -- the fix for Atom needs finer grain
> control and can be done separately.
>
> 2010-11-30 Xinliang David Li
>
>
Kenneth Zadeck writes:
> 2) The patch does not work for rtxes at all. Rtxes have to copied.
> Trees could be pointer copied.
> The problem is that CONST_INTs are not canonized in a way that wide-ints
> are or that trees could be.
> This comes from the use of the GEN_INT macro that does not t
Eric Botcazou writes:
> This is a segfault on a degenerate bitfield case introduced by the rewrite of
> the bitfield machinery. In Ada, we have bitfields of size zero and we ask
> the
> middle-end to generate accesses to them. This doesn't work anymore because
> get_best_mode now returns VOI
On reflection, I think it'd be better if I stood down as an rtl maintainer.
I'll still try to keep the MIPS stuff ticking over though.
Applied as obvious.
Richard
* MAINTAINERS: Remove self as RTL optimization maintainer.
Index: MAINTAINERS
=
On Sat, Dec 1, 2012 at 4:04 AM, Jakub Jelinek wrote:
> Hi!
>
> When I've tried to compile the attached testcase (I was trying to see
> if tsan could discover the emutls.c data race), I got ICEs because
> expr_ptr in certain cases wasn't is_gimple_val and thus was invalid to
> pass it directly to a
Hello,
as advised by Alexandre Oliva at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53860#c3 , I am submitting a
testcase for PR53860. If Ok, I need someone to commit this patch.
Tested by make check-g++.
Thanks,
Zdenek Sojka
Changelog:
PR debug/53860
* g++.dg/debug/pr53860.C: New testcase.Ind
Richard Sandiford wrote:
>On reflection, I think it'd be better if I stood down as an rtl
>maintainer.
>I'll still try to keep the MIPS stuff ticking over though.
>
>Applied as obvious.
You might consider instead denoting yourself to reviewer status. We do not
have too many people reviewing rt
Hi all,
here is a straightforward patch for the intrinsic procedure
SYSTEM_CLOCK. It does two things:
1) It reduces the resolution of the int8 version from 1 nanosecond to
1 microsecond (COUNT_RATE = 100).
2) It adds an int16 version with nanosecond precision.
The motivation for item #1 was m
On Sat, Dec 01, 2012 at 01:53:52PM +0400, Dmitry Vyukov wrote:
> On Sat, Dec 1, 2012 at 4:04 AM, Jakub Jelinek wrote:
> > Hi!
> >
> > When I've tried to compile the attached testcase (I was trying to see
> > if tsan could discover the emutls.c data race), I got ICEs because
> > expr_ptr in certain
On Sat, Dec 1, 2012 at 10:36 AM, Richard Sandiford
wrote:
> On reflection, I think it'd be better if I stood down as an rtl maintainer.
> I'll still try to keep the MIPS stuff ticking over though.
>
> Applied as obvious.
Hmm, this isn't really so obvious to me. In fact, it's rather surprising to
On Sat, Dec 1, 2012 at 4:23 PM, Jakub Jelinek wrote:
>> > Hi!
>> >
>> > When I've tried to compile the attached testcase (I was trying to see
>> > if tsan could discover the emutls.c data race), I got ICEs because
>> > expr_ptr in certain cases wasn't is_gimple_val and thus was invalid to
>> > pas
On Mon, Nov 26, 2012 at 4:46 PM, Richard Biener wrote:
> Btw, I of course have my own CFG dumper (producing graphviz input)
> in my local tree - attached for reference (I'm simply using it from
> gdb sessions).
Here's my version of it. I still have to fix some minor fall-out of
not flushing the pr
On 12/01/2012 04:28 AM, Richard Sandiford wrote:
Kenneth Zadeck writes:
2) The patch does not work for rtxes at all. Rtxes have to copied.
Trees could be pointer copied.
The problem is that CONST_INTs are not canonized in a way that wide-ints
are or that trees could be.
This comes from the u
> Attached is a different fix. It splits DF_REF_IN_NOTE in two: One flag
> for each kind of note. This allows the dead note removal code to
> distinguish the source note for the EQ_USES. I needed to remove one
> flag to keep the df_ref_flags 16-bit, but the DF_REF_SUBREG flag looks
> completely un
On Sat, Dec 1, 2012 at 2:23 AM, Zdeněk Sojka wrote:
> Hello,
>
> as advised by Alexandre Oliva at
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53860#c3 , I am submitting a
> testcase for PR53860. If Ok, I need someone to commit this patch.
> Tested by make check-g++.
>
> Thanks,
> Zdenek Sojka
>
On Fri, Nov 30, 2012 at 11:00:28PM +0100, Eric Botcazou wrote:
> OK, let's tweak the patch as follows:
> 1) when current_loops is not NULL, we compute may_be_loop_header and whether
> the loop has more than 1 latch edge exactly,
> 2) when current_loops is NULL, we use your above method to do the
On Sat, Dec 1, 2012 at 3:54 PM, Eric Botcazou wrote:
>> The patch also includes all places I've found so far where the
>> compiler could create self-referencing notes:
>>
>> 1. optabs.c: Not sure what it was trying to do, but now it just
>> refuses to add a note if TARGET is mentioned in one of the
I've merged revision 194015 from trunk to the gccgo branch.
Ian
Hello,
here is a patch. If it is accepted, I'll extend it to other vm patterns
(mul, div, min, max are likely candidates, but I need to check the doc).
It passed bootstrap+testsuite on x86_64-linux.
2012-12-01 Marc Glisse
PR target/54855
gcc/
* config/i386/sse.md (_vm3):
On Sat, Dec 1, 2012 at 2:23 PM, Steven Bosscher wrote:
> On Mon, Nov 26, 2012 at 4:46 PM, Richard Biener wrote:
>> Btw, I of course have my own CFG dumper (producing graphviz input)
>> in my local tree - attached for reference (I'm simply using it from
>> gdb sessions).
>
> Here's my version of it.
On Sat, Dec 01, 2012 at 04:55:35PM +0400, Dmitry Vyukov wrote:
> > No idea what do you mean by thread leak. What exactly is leaking?
>
>
> Thread leak is joinable but not joined thread.
> I have a pending todo to aggregate them by stack, so in this case it
> will "3 threads leaked here". Perhaps
i think that this is really a mistake. you are one of the best at this
part of the compiler.
kenny
Hi,
On 12/01/2012 07:13 AM, Jason Merrill wrote:
On 11/30/2012 04:05 PM, Paolo Carlini wrote:
@@ -219,10 +219,15 @@ cp_convert_to_pointer (tree type, tree expr, tsubs
-expr = build_int_cst (type, 0);
+expr = (TREE_SIDE_EFFECTS (expr)
+? build_nop (type, expr)
+: build_in
On 11/30/2012 03:00 PM, Steven Bosscher wrote:
Hello,
This epilogue_delay_list probably existed only for text epilogues, but
it is now unused.
Tested by building a set of cc1-i files at -O2 for SPARC with and
without the patch, and verifying that there are no code gen changes.
OK for trunk?
O
On Wed, Oct 31, 2012 at 3:39 AM, Jan Hubicka wrote:
> Hi,
> this patch implements the logic to remove statements that are known to be
> undefined and thus expected to not be executed after unrolling. It also
> removes redundant exits that I originally tried to do at once, but it
> does not fly, s
The attached change adds the "-std=gnu++0x -pthread" options on
hppa*-hp-hpux11*. Test passes with the change.
Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to
trunk and 4.7 branch.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
Nationa
Tested on hppa1.1.-hp-hpux10.20. Commetted to trunk and 4.7.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
2012-12-01 John David Anglin
PR ada/52110
* s-osin
Test no longer fails on trunk. Committed to trunk.
Dave
--
J. David Anglin dave.ang...@nrc-cnrc.gc.ca
National Research Council of Canada (613) 990-0752 (FAX: 952-6602)
2012-12-01 John David Anglin
PR middle-end/52450
* gcc.dg/to
On Sat, Dec 1, 2012 at 1:17 PM, Janus Weil wrote:
> Hi all,
>
> here is a straightforward patch for the intrinsic procedure
> SYSTEM_CLOCK. It does two things:
> 1) It reduces the resolution of the int8 version from 1 nanosecond to
> 1 microsecond (COUNT_RATE = 100).
> 2) It adds an int16 vers
Rainer,
This patch is completely wrong and unacceptable because your test
applies too generally. Why, exactly, is it appropriate to assume that
the "-nH" assembler command line option means the same thing on all
systems and one wants to clear HW capabilities on all systems?
Because it is not appr
Hi,
recent 4,8 has regressions in g++.old-deja/g++.eh for the catch*.C
tests, if exception-mechanism is SjLj. This is due an off by one
failure in an decreasing loop.
ChangeLog
2012-12-01 Kai Tietz
* stmt.c (expand_sjlj_dispatch_table): Fix off by one.
Tested for i686-w64-mingw32, x8
On Sat, Dec 1, 2012 at 10:59 PM, Kai Tietz wrote:
> Hi,
>
> recent 4,8 has regressions in g++.old-deja/g++.eh for the catch*.C
> tests, if exception-mechanism is SjLj. This is due an off by one
> failure in an decreasing loop.
>
> ChangeLog
>
> 2012-12-01 Kai Tietz
>
> * stmt.c (expand_sj
The attached change fixes the compilation of the following asm
in libquadmath/math/fmaq.c:
asm volatile ("" : : "m" (v.value));
The issue arises because there is no support for directly loading TFmode
objects.
Ok for trunk?
Dave
--
J. David Anglin dave.ang...@n
Hi,
I'm adding the testcase and closing the PR.
Thanks,
Paolo.
///
2012-12-01 Paolo Carlini
PR c++/8
* g++.dg/cpp0x/decltype46.C: New.
Index: g++.dg/cpp0x/decltype46.C
===
--- g++.dg/cpp0x/de
Hi Janne,
thanks for your feedback ...
>> here is a straightforward patch for the intrinsic procedure
>> SYSTEM_CLOCK. It does two things:
>> 1) It reduces the resolution of the int8 version from 1 nanosecond to
>> 1 microsecond (COUNT_RATE = 100).
>> 2) It adds an int16 version with nanosec
On Sat, Dec 1, 2012 at 6:50 AM, Xinliang David Li wrote:
> 2010-11-30 Xinliang David Li <>
>
> * config/i386/i386.c: Allow sign extend instructions (cltd etc)
> on modern CPUs.
You installed the patch without the ChangeLog entry...
(http://gcc.gnu.org/ml/gcc-cvs/2012-12/msg00027.
Fixed.
thanks,
David
On Sat, Dec 1, 2012 at 4:08 PM, Steven Bosscher wrote:
> On Sat, Dec 1, 2012 at 6:50 AM, Xinliang David Li wrote:
>> 2010-11-30 Xinliang David Li <>
>>
>> * config/i386/i386.c: Allow sign extend instructions (cltd etc)
>> on modern CPUs.
>
> You installed
Change LTO-related hash tables from htab_t to hash_table:
lto-streamer.h output_block::string_hash_table
lto-streamer-in.c file_name_hash_table
lto-streamer.c tree_htab
The struct string_slot moves from data-streamer.h to lto-streamer.h to
resolve compilation dependences.
Tested on x86-64.
Oka
Change graphite-related hash tables from htab_t to hash_table:
graphite-clast-to-gimple.c ivs_params::newivs_index
graphite-clast-to-gimple.c ivs_params::params_index
graphite-clast-to-gimple.c print_generated_program::params_index
graphite-clast-to-gimple.c gloog::newivs_index
graphite-clast-to-g
Change gimplify.c gimplify_ctx::temp_htab hash table from htab_t to
hash_table.
Move struct gimple_temp_hash_elt and struct gimplify_ctx to a new
gimplify-ctx.h, because they are used few places.
Tested on x86-64.
Okay for branch?
Index: gcc/ChangeLog
2012-11-30 Lawrence Crowl
*
Change tree-related hash tables from htab_t to hash_table:
tree-complex.c complex_variable_components
tree-parloops.c eliminate_local_variables_stmt::decl_address
tree-parloops.c separate_decls_in_region::decl_copies
Move hash table declarations to a new tree-hasher.h, to resolve
compilation depe
Change ggc-common hash tables from htab_t to hash_table:
ggc-common.c loc_hash
ggc-common.c ptr_hash
Add a new hash_table method elements_with_deleted to meet the needs of
gcc-common.c.
Correct many methods with parameter types compare_type to the correct
value_type. (Correct code was unlikely
Hello,
in PR50829, HJ Lu pointed me to this PR for which I already had a patch (I
hadn't submitted it because I didn't have a good use case for it).
bootstrap+testsuite on x86_64-linux.
2012-12-02 Marc Glisse
PR target/44551
gcc/
* simplify-rtx.c (simplify_binary_operation
The attached patch eliminates PR 55521/sanitizer by switching libasan on
darwin
from using mach_override to mac function interposition via the importation of
the
asan/dynamic/asan_interceptors_dynamic.cc file from llvm.org's compiler-rt svn.
The changes involve defining USING_MAC_INTERPOSE in
Richard,
The testcases assume default signed char and fail on systems with
different semantics. I believe that both testcases need to declare c
as signed char to consistently test the desired behavior, right?
Thanks, David
This patch is another in my series of copy-edits to extend.texi. This
installment covers random problems with grammar, punctuation,
terminology, etc that jumped out at me when reading through the chapter,
rather than being a systematic search-and-destroy on pervasive usage
problems (as most of
Hi Jack,
IIUC the wrappers for dispatch_async_f, dispatch_sync_f and other
dispatch_smth_f do not need blocks support in the compiler, since
regular functions are passed into them. So you may want to add the
dynamic interceptors for those back.
The remaining problem is that dispach_async and other
2012-12-01 Teresa Johnson
PR gcov-profile/1
* lto-cgraph.c (merge_profile_summaries): Handle scaled histogram
entries that map to the same index.
Index: lto-cgraph.c
===
--- lto-cgraph.c(revisio
50 matches
Mail list logo
