Hello,
On 05.09.2012 19:38, Jakub Jelinek wrote:
Hi!
As discussed in the PR, sel-sched doesn't handle correctly
tidying of empty blocks if fallthru predecessor ends with asm goto
that has some labels on the empty block in addition to the fallthru edge.
cfgrtl.c can handle that, so this patch ju
On Wed, 5 Sep 2012, Gabriel Dos Reis wrote:
On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V wrote:
Let's say we have two for loops like this:
int my_func (int x, int y);
For (ii = 0; ii < 1; ii++)
X[ii] = my_func (Y[ii], Z[ii]);
I assume X, Y and Z are __restrict pointers (or so
Steve Ellcey writes:
> On Tue, 2012-09-04 at 23:55 +0100, Richard Sandiford wrote:
>> If we do that, then your DRIVER_SELF_SPECS can further have:
>>
>> MIPS_ISA_SYNCI_SPEC
>>
>> where the definition:
>>
>> /* Infer a -msynci setting from a -mips argument, on the assumption that
>>-msynci
Hi Richard,
Thanks very much for comments.
> > Ping?
> >
> > Hi Ramana, could you help me review this patch?
> > Hi Eric, Richard, could you help me review the change in regcprop.c?
>
> Subtraction of zero isn't canonical rtl though. Passes after peephole2
would
> be well within their rights to
This patch to libgo fixes a couple of things in the DWARF line reader.
It adds support for DW_AT_high_pc as a constant offset from
DW_AT_low_pc. It adds support for DW_AT_ranges. Bootstrapped and ran
Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7
branch.
This fixes some
Hi,
The inlininer likes to recreate some MEM_REF, it copies most of the
bits (TREE_THIS_NOTRAP, TREE_THIS_VOLATILE, etc.) but forgets about
TREE_SIDE_EFFECTS. This causes the strlen optimization to think the
memory store does not have a side effects.
OK? Bootstrapped and tested on x86_64-linux
Commited as r191006 and r191007.
Ollie
On Wed, Sep 5, 2012 at 10:50 PM, Paul Pluzhnikov wrote:
> On Wed, Sep 5, 2012 at 6:26 PM, Ollie Wild wrote:
>
>> Okay for google/integration and google/gcc-4_7?
>
> Approved for google/* branches.
>
> Thanks,
> --
> Paul Pluzhnikov
On Wed, Sep 5, 2012 at 6:26 PM, Ollie Wild wrote:
> Okay for google/integration and google/gcc-4_7?
Approved for google/* branches.
Thanks,
--
Paul Pluzhnikov
On Mon, 3 Sep 2012, nick clifton wrote:
> I like the full "xstormy16" as well. I think that the fact that the gcc
> backend sources are in a directory called "stormy16" is just a historical
> curiosity...
Now that we are using svn, you could do a mv and change this. :)
And the following patch up
>> This patch is for the google/gcc-4_7 branch.
>
> Approved for google/gcc-4_7 branch.
Thanks, committed at r191005.
-cary
On Wed, Sep 5, 2012 at 7:46 PM, Cary Coutant wrote:
> This patch is for the google/gcc-4_7 branch.
Approved for google/gcc-4_7 branch.
Thanks,
--
Paul Pluzhnikov
Remove inconsistent code and macro define in score backend,
Use gcc default code instead.
--liqin
ChangeLog:
2012-09-06 Chen Liqin
* config/score/score.c : Remove TARGET_LEGITIMIZE_ADDRESS define
and score_legitimize_address function, use compiler default code
instead.
Index
That was an interesting one to find and understand: this was not
supposed to be a link, but setting of an anchor.
Fix vs confusion in codingrationale.html; applied.
Gerald
Index: codingrationale.html
===
RCS file: /cvs/gcc/wwwdocs
This patch is for the google/gcc-4_7 branch.
Backport upstream patch to fix a problem where type signature does
not include the type's context.
Tested with bootstrap and regression tests.
2012-07-19 Jason Merrill
PR debug/53235
* dwarf2out.c (get_die_parent): New.
What progress about my patch?
This is a Google-local fix to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54482.
When configured with --with-pic, libstdc++.a includes versioned
symbols, preventing it from being linked into shared libraries. The
ultimate cause of this is a misuse of -DPIC as a proxy for, "I'm being
compiled into
On Wed, Sep 5, 2012 at 5:09 PM, Iyer, Balaji V wrote:
> Hello Everyone,
> Attached, please find the 1st of ~22 patches that implements Cilk
> Plus. This patch will implement Elemental Functions into the C compiler.
> Please check it in to the trunk if it looks OK.
>
> Below, I w
On Wed, 5 Sep 2012, Iyer, Balaji V wrote:
> Hello Everyone,
> Attached, please find a patch that will add regression test cases
> for elemental function implementation in C.
Can the tests be arranged so that all functionality that isn't
intrinsically target-specific is tested for all targ
On Wed, 5 Sep 2012, Iyer, Balaji V wrote:
> Attached, please find the 1st of ~22 patches that implements Cilk
> Plus. This patch will implement Elemental Functions into the C compiler.
> Please check it in to the trunk if it looks OK.
>
> Below, I will give you a small example abou
On 09/05/2012 02:17 PM, Paolo Carlini wrote:
In fact, something seems weird earlier, in
cp_parser_check_template_parameters. It has:
/* If there are the same number of template classes and parameter
lists, that's OK. */
if (parser->num_template_parameter_lists == num_templates)
Hello Everyone,
Attached, please find a patch that will add regression test cases for
elemental function implementation in C.
Here are the Changelog entries:
===
gcc/testsuite/ChangeLog
2012-09-05 Balaji V. Iyer
On Wed, 2012-09-05 at 08:15 +0200, Jakub Jelinek wrote:
> The debug insns generally shouldn't extend the lifetime of pseudos (see the
> valtrack.c stuff), so if you hit this, there is probably some earlier bug
> that didn't reset/adjust the debug insns in question.
> I'm not saying the ira.c patch
On Wed, Sep 5, 2012 at 2:36 PM, Jakub Jelinek wrote:
> On Wed, Sep 05, 2012 at 02:10:03PM -0700, Andrew Pinski wrote:
>> The problem here is the strlen optimization tries to remove a null
>> character store as we already have done it but it does it for a
>> volatile store which is not a valid th
Hi, Richi,
Sorry if this comes in late. I'd saved your message for careful
analysis and I only got back to it now.
On Aug 1, 2012, Richard Guenther wrote:
> On Fri, Dec 2, 2011 at 8:28 PM, Jakub Jelinek wrote:
>> PR debug/50317
>> * tree-ssa.c (target_for_debug_bind): Also allow is_gimple_re
On Aug 20, 2012, Jakub Jelinek wrote:
> On the testcase from this PR on AVR (from libgcc, thus not including
> it into testsuite/) we ICE, because dead_debug_insert_temp is called
> several times on the same insn, for multi-register hard register for each
> regno in it (except the first which doe
Hello,
Although the recent optimization I have committed to use Neon vext
instruction for suitable builtin_shuffle calls does not support
big-endian yet, I have written a patch to the existing testcases such
they now support big-endian mode.
I think it's worth improving these tests since writing
On Sep 5, 2012, Richard Guenther wrote:
> Yes, the goal is definitely to avoid the jumping back and forth
> on source lines you can see when debugging optimized programs.
Hmm... If that's the goal, how about adding to the mix the Statement
Frontier Notes proposal I advanced in some GCC Summit?
On Wed, Sep 05, 2012 at 11:55:54PM +0200, Tobias Burnus wrote:
> Rather obvious fix.
>
> Build on x86-64-linux.
> OK for the trunk when regtesting has succeeded?
>
Yes.
--
Steve
Rather obvious fix.
Build on x86-64-linux.
OK for the trunk when regtesting has succeeded?
Tobias
2012-09-06 Tobias Burnus
PR fortran/54463
* trans-intrinsic.c (gfc_conv_intrinsic_funcall): Fix matmul
call to BLAS if the default-kind has been promoted.
2012-09-06 Tobias Burnus
PR fortra
On Wed, Sep 05, 2012 at 02:10:03PM -0700, Andrew Pinski wrote:
> The problem here is the strlen optimization tries to remove a null
> character store as we already have done it but it does it for a
> volatile store which is not a valid thing to do.
> This patch fixes the problem by ignoring state
On Wed, Sep 5, 2012 at 11:30 PM, Uros Bizjak wrote:
>> * config/fpu-387.h (set_fpu): Use __builtin_ia32_stmxcsr and
>> __builtin_ia32_ldmxcsr intrinsics.
>>
>> Tested on x86_64-pc-linux-gnu {,-m32}, committed to mainline an 4.7 branch.
>
> I forgot that these builtins are enabled
On Wed, Sep 5, 2012 at 9:53 PM, Uros Bizjak wrote:
> This patch substitutes volatile asms with equivalent intrinsics.
>
> 2012-09-05 Uros Bizjak
>
> * config/fpu-387.h (set_fpu): Use __builtin_ia32_stmxcsr and
> __builtin_ia32_ldmxcsr intrinsics.
>
> Tested on x86_64-pc-linux-g
On Tue, 2012-09-04 at 23:55 +0100, Richard Sandiford wrote:
> If we do that, then your DRIVER_SELF_SPECS can further have:
>
> MIPS_ISA_SYNCI_SPEC
>
> where the definition:
>
> /* Infer a -msynci setting from a -mips argument, on the assumption that
>-msynci is desired where possible. *
Hi,
The problem here is the strlen optimization tries to remove a null
character store as we already have done it but it does it for a
volatile store which is not a valid thing to do.
This patch fixes the problem by ignoring statements which have
volatile operands.
OK? Bootstrapped and tested o
Hi,
bitmap.c:bitmap_last_set_bit() is not used by any code in the current
GCC trunk, but I'm using it and I noticed it returns an incorrect
result. This patch rewrites most of the function to return the correct
result.
Not sure how to test this other than to say that my code, that uses
this funct
Hello,
Just some cleanups I did while working on something bigger.
OK for trunk?
Ciao!
Steven
* graphite.c (print_global_statistics): Use EDGE_COUNT instead
of VEC_length.
(print_graphite_scop_statistics): Likewise.
* graphite-scop-detection.c (get_bb_type): Use
On 9/5/12, Marc Glisse wrote:
> On Wed, 5 Sep 2012, Lawrence Crowl wrote:
>> On 9/5/12, Richard Guenther wrote:
>>> The tmin.cmp (tmax, uns) > 0 kind of things look odd - definitely
>>> methods like tmin.gt (tmax, uns) would be nice to have. Or even
>>> better, get rid of the 'uns' parameters an
Hello!
This patch substitutes volatile asms with equivalent intrinsics.
2012-09-05 Uros Bizjak
* config/fpu-387.h (set_fpu): Use __builtin_ia32_stmxcsr and
__builtin_ia32_ldmxcsr intrinsics.
Tested on x86_64-pc-linux-gnu {,-m32}, committed to mainline an 4.7 branch.
Uros.
In
On 09/05/2012 11:58 AM, Paolo Carlini wrote:
Hi,
On 09/04/2012 10:08 PM, François Dumont wrote:
Hi
I managed to do the test with Valgrind and so confirm the fix
with the attached patch (unmodified since last proposal).
Patch is Ok, thanks for your patience and thanks again for all your
g
On Wed, 5 Sep 2012, Lawrence Crowl wrote:
On 9/5/12, Richard Guenther wrote:
The tmin.cmp (tmax, uns) > 0 kind of things look odd - definitely
methods like tmin.gt (tmax, uns) would be nice to have. Or even
better, get rid of the 'uns' parameters and provide a
struct double_int_with_signedne
On Wed, 2012-09-05 at 14:39 -0400, DJ Delorie wrote:
> I don't feel the m32c change needs my specific ack, it's a harmless
> change that goes with the ack for the feature itself.
>
> However, I will note that m32c does have different costs for addresses
> in different address spaces, at least when
I don't feel the m32c change needs my specific ack, it's a harmless
change that goes with the ack for the feature itself.
However, I will note that m32c does have different costs for addresses
in different address spaces, at least when -Os.
On 09/05/2012 06:52 PM, Paolo Carlini wrote:
On 09/05/2012 06:41 PM, Paolo Carlini wrote:
Thanks for looking into this. Now I wonder if we made progress on a
couple of long standing PRs where we weren't strict enough by one
with the number of 'template <>'. Let me check...
Nope, apparently c++/
On 9/5/12, Richard Guenther wrote:
> On Tue, 4 Sep 2012, Lawrence Crowl wrote:
>> Modify gcc/*.[hc] double_int call sites to use the new interface.
>> This change entailed adding a few new methods to double_int.
>>
>> Other changes will happen in separate patches. Once all uses of
>> the old inte
On Wed, 2012-09-05 at 13:44 +0400, Denis Chertykov wrote:
> 2012/9/5 Oleg Endo :
> > Updated ACK table:
> >
> > [x] target-independent bits
> > [ ] alpha [x] arm [ ] avr [ ] bfin
> > [ ] cr16 [x] cris [x] epiphany[ ] i386
> > [ ] ia64 [x] iq2000[ ] lm32
Rather obvious fix. gfc_undo_symbols segfaulted when the COMMON
statement aborted before the common symtree was created. Committed as
Rev. 190989.
Hopefully, that's the last fall out of my memory clean up patch.
The hopefully last issue with the current FINAL patch has already been
fixed by M
On 09/05/2012 06:41 PM, Paolo Carlini wrote:
Thanks for looking into this. Now I wonder if we made progress on a
couple of long standing PRs where we weren't strict enough by one with
the number of 'template <>'. Let me check...
Nope, apparently c++/24314 is still there. But maybe it's easier to
Hi,
On 08/31/2012 11:32 PM, Jason Merrill wrote:
Since you're traveling, I poked at this myself some more. The issue
here is that there are too many template headers for the declaration,
so we want to figure out what the right number is and give an
appropriate message.
Tested x86_64-pc-linu
Caught by Clang (which also checks uninstantiated templates).
PR bootstrap/54484
* vec.h (vec_t::lower_bound): Fix spelling of LESSTHAN
argument.
diff --git a/gcc/vec.h b/gcc/vec.h
index 441c9b5..fbf95d2 100644
--- a/gcc/vec.h
+++ b/gcc/vec.h
@@ -1075,7 +1075,7 @@ vec_
On Wed, Sep 5, 2012 at 9:13 AM, H.J. Lu wrote:
> On Wed, Sep 5, 2012 at 8:50 AM, Teresa Johnson wrote:
>> On Wed, Sep 5, 2012 at 8:44 AM, H.J. Lu wrote:
>>> On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote:
Sorry about that. I am right now trying to reproduce the
profiledbootstrap
On Thu, Aug 30, 2012 at 12:29:51PM +0200, Thiago Macieira wrote:
> 2012-08-30 Thiago Macieira
>
> PR libstdc++/54172
> * libsupc++/guard.cc (__cxa_guard_acquire): Don't compare_exchange
> from a finished state back to a waiting state.
This is ok for trunk and 4.7 branch.
T
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01100.html
- C++ -Wsizeof-pointer-memaccess support (C is already in)
On 2012-09-05 11:40 , Jakub Jelinek wrote:
2012-09-05 Jakub Jelinek
PR middle-end/54486
* builtins.c (fold_builtin_strspn, fold_builtin_strcspn): Use
build_int_cst with size_type_node instead of size_int.
* c-c++-common/pr54486.c: New test.
OK.
On Wed, Sep 5, 2012 at 8:50 AM, Teresa Johnson wrote:
> On Wed, Sep 5, 2012 at 8:44 AM, H.J. Lu wrote:
>> On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote:
>>> Sorry about that. I am right now trying to reproduce the
>>> profiledbootstrap problem that H.J. reported, which is on
>>> x86_64-un
Hi,
This patch implements __builtin_bswap16() on ARM (v6 and above) using
revsh with a signed input and rev16 with an unsigned input.
It is pretty much equal to the patch posted some time ago
http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00962.html, but it's hard
to write such patterns differently
On Wed, Sep 5, 2012 at 8:44 AM, H.J. Lu wrote:
> On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote:
>> Sorry about that. I am right now trying to reproduce the
>> profiledbootstrap problem that H.J. reported, which is on
>> x86_64-unknown-linux-gnu where I had successfully done a
>> profiledbo
On Wed, Sep 5, 2012 at 8:09 AM, Teresa Johnson wrote:
> Sorry about that. I am right now trying to reproduce the
> profiledbootstrap problem that H.J. reported, which is on
> x86_64-unknown-linux-gnu where I had successfully done a
> profiledbootstrap before my commit.
>
> Unfortunately after svn
Hi!
As discussed in the PR, sel-sched doesn't handle correctly
tidying of empty blocks if fallthru predecessor ends with asm goto
that has some labels on the empty block in addition to the fallthru edge.
cfgrtl.c can handle that, so this patch just gives up on it on the sel-sched
side. The testca
Hi!
fold_builtin_str*spn in this case returns sizetype typed constant instead of
size_t, which makes -Wformat warn. Fixed thusly, bootstrapped/regtested on
x86_64-linux and i686-linux, ok for trunk/4.7?
2012-09-05 Jakub Jelinek
PR middle-end/54486
* builtins.c (fold_builtin_s
On Wed, Sep 5, 2012 at 3:18 PM, Matthew Gretton-Dann
wrote:
> On 5 September 2012 13:45, Richard Earnshaw wrote:
>> On 05/09/12 13:02, Steven Bosscher wrote:
>>> On Wed, Sep 5, 2012 at 1:42 PM, Matthew Gretton-Dann wrote:
Whilst this fix works for this particular case I am not sure it is the
Hello!
Attached patch removes various *_not_xmm0_operand hacks. These were
used to prevent combine from moving xmm0 register to the wrong place,
but recently we implemented better approach.
2012-09-05 Uros Bizjak
* config/i386/sse.md (_blendv):
Use register_operand instead of
Sorry about that. I am right now trying to reproduce the
profiledbootstrap problem that H.J. reported, which is on
x86_64-unknown-linux-gnu where I had successfully done a
profiledbootstrap before my commit.
Unfortunately after svn updating my client I am hitting an unrelated
build problem with my
On Wed, Sep 5, 2012 at 7:23 AM, Simon Baldwin wrote:
> On 5 September 2012 16:03, Ian Lance Taylor wrote:
>> On Wed, Sep 5, 2012 at 6:56 AM, Simon Baldwin wrote:
>>> Add a configure option to disable system header canonicalizations.
>>
>> Why should this be a configure option rather than a comma
On 08/03/2012 02:05 PM, Bernd Schmidt wrote:
> This patch allows us to change
>
> rn++
> rm=[rn]
>
> into
>
> rm=[rn + 4]
> rn++
Ping.
Bernd
On 5 September 2012 16:03, Ian Lance Taylor wrote:
> On Wed, Sep 5, 2012 at 6:56 AM, Simon Baldwin wrote:
>> Add a configure option to disable system header canonicalizations.
>
> Why should this be a configure option rather than a command-line option?
The underlying problem is a niche one, like
On Wed, Sep 5, 2012 at 6:56 AM, Simon Baldwin wrote:
> Add a configure option to disable system header canonicalizations.
Why should this be a configure option rather than a command-line option?
Ian
> Libcpp may canonicalize system header paths with lrealpath() for diagnostics,
> dependency o
Add a configure option to disable system header canonicalizations.
Libcpp may canonicalize system header paths with lrealpath() for diagnostics,
dependency output, and similar. If gcc is held in a symlink farm the
canonicalized paths may be meaningless to users, and will also conflict with
build
> Basic block reordering is disabled for -Os from gcc 4.7 since the pass will
> lead to big code size regression. But benchmarks logs also show there are
> lots of regression due to poor code layout compared with 4.6.
>
> The patch is to enable bbro for -Os. When optimizing for size, it
> * avoid
> The file contains 3 RTL optimization passes, the gate and worker functions
> of which are strangely intertwined.
More cosmetic changes: a few clarifications in the head comment of the file, a
handful of long lines and other formatting nits. No functional changes.
Tested on x86_64-suse-linux,
On 04/09/2012 00:19, Dominique Dhumieres wrote:
> Hi Tobias,
>
> The lines 6 and 12 of gfortran.dg/coarray_poly_3.f90 need some adjustment
> along the line:
For what's worth, the testsuite change was part of patch (b) at
http://gcc.gnu.org/ml/fortran/2012-08/msg00201.html
while it should have bee
This cuts another 250s off the testcase in PR46590 by calling
update_ssa from complete unrolling only after all innermost
loops are processed once.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-09-05 Richard Guenther
PR tree-optimization/46590
On 5 September 2012 13:45, Richard Earnshaw wrote:
> On 05/09/12 13:02, Steven Bosscher wrote:
>> On Wed, Sep 5, 2012 at 1:42 PM, Matthew Gretton-Dann wrote:
>>> Whilst this fix works for this particular case I am not sure it is the
>>> best fix for the general issue, and so if others have a bette
Hi,
On Tue, Sep 04, 2012 at 04:26:43PM +0200, Richard Guenther wrote:
> On Tue, Sep 4, 2012 at 3:12 PM, Martin Jambor wrote:
> > Hi,
> >
> > while looking into how to remove push/pop_cfun from dwarf2out.c, I
> > have noticed the following wrong use of cfun in premark_used_types,
> > which is the
On Wed, Sep 5, 2012 at 6:08 AM, Paolo Carlini wrote:
> Uli, I'm not sure to understand why that commit of yours changed that
> specific regexp,
Completely unintended, I thought I mentioned this already. The
problem is emacs's whitespace mode which made those changes.
On 05/09/12 13:02, Steven Bosscher wrote:
> On Wed, Sep 5, 2012 at 1:42 PM, Matthew Gretton-Dann wrote:
>> Whilst this fix works for this particular case I am not sure it is the
>> best fix for the general issue, and so if others have a better idea how
>> to fix this I would be very happy.
>
> pos
On Wed, Sep 5, 2012 at 1:42 PM, Matthew Gretton-Dann wrote:
> Whilst this fix works for this particular case I am not sure it is the
> best fix for the general issue, and so if others have a better idea how
> to fix this I would be very happy.
postreload-gcse.c is broken in "interesting" ways. Loo
On Wed, Sep 5, 2012 at 1:25 PM, Matthew Gretton-Dann wrote:
> + /* If the two blocks are in different partitions we do not want to mark
> + this as a fallthru edge. */
> + if (BB_PARTITION (b) != BB_PARTITION (c))
> +return;
> +
I think you should look for a REG_CROSSING_JUMP note on
Ok for trunk, 4.7, 4.6?
2012/8/17 Anna Tikhonova :
> Ping one more time.
>
> 2012/8/10 Anna Tikhonova :
>> Ping.
>>
>> 2012/8/8 Anna Tikhonova :
>>> Hi,
>>>
>>> while running check for Android NDK compiler (I've used
>>> contrib/test_installed for it) I've noticed that gcov name is
>>> hardcoded i
All,
When implementing ARM/Thumb support for -freorder-blocks-and-partition I
encountered the following silent code generation fault.
Given the following CFG:
| |
93 97
| |
(FALLTHRU)(CROSSING)
\ /
\\ /---/
All,
This is the first patch in a series with the ultimate aim of enabling
-freorder-blocks-and-partition in the ARM backend.
However, whilst working on this I have come across a number of midend issues
which should be fixed individually.
This patch fixes an ICE during if-conversion.
The probl
On 2012-09-04 18:49 , Steven Bosscher wrote:
Did you mean to use DEF_VEC_O here?
(Not sure it matters after the vec rewrite for c++)
It doesn't anymore. But it will matter for backports.
Diego.
> -Original Message-
> From: Ramana Radhakrishnan
> Sent: Tuesday, September 04, 2012 4:03 PM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] PR45070: Fix wrong epilogue code for cortex-m0/Os
>
> >
> > I ran regression test with/without Os for cortex-m0 and everythin
On 09/05/2012 11:53 AM, Paolo Carlini wrote:
On 08/30/2012 06:37 PM, Benjamin De Kosnik wrote:
Nice! Thanks.
Here's a small patchlet to set the abi version to .18. With this,
check-abi will pass.
tested x86/linux
Benjamin, is this still uncommitted? I'm seeing abi_check failing...
Ok, now I s
On Wed, Sep 5, 2012 at 12:10 PM, Gopalasubramanian, Ganesh
wrote:
> FMA4 and FMA3 ISA are implemented in bdver2 target.
> FMA3 is selected by default.
> This patch supports the use of FMA4 intrinsics for bdver2 targets.
>
> Is it OK for trunk?
OK. I will backport this patch, together with my pre
On 08/28/2012 01:49 AM, Jason Merrill wrote:
OK.
Thanks Jason. I have now committed the patch together with the three
additional tests which depended on instantiation_dependent_p.
Thanks again,
Paolo.
Hello,
FMA4 and FMA3 ISA are implemented in bdver2 target.
FMA3 is selected by default.
This patch supports the use of FMA4 intrinsics for bdver2 targets.
Is it OK for trunk?
Regards
Ganesh
2012-09-05 Ganesh Gopalasubramanian
* config/i386/i386.md : Comments on fma4 instruction
On 09/01/2012 08:10 PM, Steven Bosscher wrote:
Hello,
r190783 breaks bootstrap on powerpc64-unknown-linux-gnu. The problem
is caused by a regexp that used to check for space|tab before the
patch but now only looks for tab. The attached patch fixes this
problem for me, but I'm not sure why (I hav
On Tue, 2012-09-04 at 18:17 +0100, Bryce McKinlay wrote:
> libgcj wouldn't actually use it for unwinding, we already have all
> that. We'd just use it to read DWARF debug info and give us the source
> code line numbers.
Casey Marshell did also write that part some time ago, but it was never
finish
> -Original Message-
> From: Richard Guenther [mailto:richard.guent...@gmail.com]
> Sent: Tuesday, September 04, 2012 6:31 PM
> To: Zhenqiang Chen
> Cc: Steven Bosscher; gcc-patches@gcc.gnu.org
> Subject: Re: Ping: [PATCH] Enable bbro for -Os
>
> On Wed, Aug 29, 2012 at 10:42 AM, Zhenqian
On 08/30/2012 06:37 PM, Benjamin De Kosnik wrote:
Nice! Thanks.
Here's a small patchlet to set the abi version to .18. With this,
check-abi will pass.
tested x86/linux
Benjamin, is this still uncommitted? I'm seeing abi_check failing...
Thanks,
Paolo.
On Wed, Sep 5, 2012 at 12:49 AM, Steven Bosscher wrote:
> On Tue, Sep 4, 2012 at 6:43 PM, Andi Kleen wrote:
>> +/* Compact representation of a index <-> resolution pair. Unpacked to an
>> + vector later. */
>> +struct res_pair
>> +{
>> + ld_plugin_symbol_resolution_t res;
>> + unsigned index;
2012/9/5 Oleg Endo :
> Updated ACK table:
>
> [x] target-independent bits
> [ ] alpha [x] arm [ ] avr [ ] bfin
> [ ] cr16 [x] cris [x] epiphany[ ] i386
> [ ] ia64 [x] iq2000[ ] lm32[ ] m32c
> [x] m32r [x] mcore [ ] mep [x] microblaze
On 05/09/12 09:16, Oleg Endo wrote:
> Updated ACK table:
>
> [x] target-independent bits
> [ ] alpha [x] arm [ ] avr [ ] bfin
> [ ] cr16 [x] cris [x] epiphany[ ] i386
> [ ] ia64 [x] iq2000[ ] lm32[ ] m32c
> [x] m32r [x] mcore [ ] mep
On Tue, Sep 4, 2012 at 6:42 PM, Andi Kleen wrote:
> From: Andi Kleen
>
> Make it clear in the documentation that with -fno-fat-lto-objects the
> gcc-* wrappers should be used to pass the linker plugin.
Ok.
Thanks,
Richard.
> gcc/:
>
> 2012-09-04 Andi Kleen
>
> * doc/invoke.texi (-ffa
On Wed, Sep 05, 2012 at 11:07:17AM +0200, Richard Guenther wrote:
> > But that's not a problem with inlining, that's a problem with allowing
> > things to happen out of order (for some definition of things and
> > order) - which in my understanding -Og is going to tie down.
>
> Yes, the goal is de
> Date: Wed, 5 Sep 2012 10:55:27 +0300
> From: Jonathan Wakely
>
> I'll look at this more carefully later today when I get back from a
> holiday, but it looks ok after a quick glance.
Great!
> I've CC'd gcc-patches, as required for all patches.
Sorry 'bout that. Bit of a brain fart on my side
On Wed, 5 Sep 2012, Matthew Gretton-Dann wrote:
> On 5 September 2012 09:55, Steven Bosscher wrote:
> > On Wed, Sep 5, 2012 at 10:46 AM, Matthew Gretton-Dann
> > wrote:
> >>> Please, no inlining. Think of stack back-traces and their use
> >>> when debugging.
> >>
> >> I would argue [without suf
On 5 September 2012 09:55, Steven Bosscher wrote:
> On Wed, Sep 5, 2012 at 10:46 AM, Matthew Gretton-Dann
> wrote:
>>> Please, no inlining. Think of stack back-traces and their use
>>> when debugging.
>>
>> I would argue [without sufficient knowledge of how easy this would
>> actually be to do i
On Wed, Sep 5, 2012 at 10:46 AM, Matthew Gretton-Dann
wrote:
>> Please, no inlining. Think of stack back-traces and their use
>> when debugging.
>
> I would argue [without sufficient knowledge of how easy this would
> actually be to do in a real compiler :-)] that this is a debugger
> problem and
On 4 September 2012 21:42, Hans-Peter Nilsson wrote:
> On Mon, 3 Sep 2012, Richard Guenther wrote:
>> On Fri, Aug 10, 2012 at 1:30 PM, Richard Guenther wrote:
>> >
>> > This adds a new optimization level, -Og, as previously discussed.
>> > It aims at providing fast compilation, a superior debuggi
1 - 100 of 108 matches
Mail list logo
