Hi!
>> * c-c++-common/asan/no-asan-stack.c (this triggers read overflow
>> because we haven't found a cross-platform way to grep for stack
>> redzones instrumentation)
> I'd prefer no test in that case, or just some semi-platform specific
test
> (scan that the 0x41b58ab3 constant doesn't appear
On Wed, Jan 8, 2014 at 10:58 PM, Jakub Jelinek wrote:
> On Wed, Jan 08, 2014 at 10:51:53PM +0100, Steven Bosscher wrote:
>> Hello Uros, and everyone else,
>>
>> Now that LRA is always used for the i386 targets, reload_in_progress
>> is never set so all code conditional on it is now dead. The attac
On Wed, 8 Jan 2014, Jakub Jelinek wrote:
> On Wed, Jan 08, 2014 at 12:19:02PM +0100, Richard Biener wrote:
> > On Wed, 8 Jan 2014, Jakub Jelinek wrote:
> >
> > > On Wed, Jan 08, 2014 at 12:15:40PM +0100, Richard Biener wrote:
> > > > > I start to think this is a too complex transform for stmt fol
> This fixes the gimple verification ICEs in the Fortran testsuite.
> ...
This patch causes 6300+ regressions (-m32/-m64 all languages but go).
However the following change
--- ../_clean/gcc/lto/lto.c 2014-01-04 15:51:44.0 +0100
+++ gcc/lto/lto.c 2014-01-08 08:26:09.0 +0
Jakub Jelinek writes:
> Hi!
>
> Here is an updated version which doesn't warn about #include_next.
> Ok for trunk?
>
> 2013-12-10 Jakub Jelinek
>
> * sanitizer_common/Makefile.am (AM_CPPFLAGS): Add
> -isystem $(top_srcdir)/include/system.
> * sanitizer_common/Makefile.in: Reg
Bernd Edlinger writes:
> I found another test case that still fails with today's trunk:
>
> #include
>
> __m256 a[10], b[10], c[10];
>
> void __attribute__((target ("sse2"), optimize (3)))
> foo (void)
> {
> }
>
> void __attribute__((target ("avx"), optimize (3)))
> bar (void)
> {
> a[0] = _mm2
Jakub Jelinek writes:
> This is a second attempt at libsanitizer symbolization using
> libbacktrace. The compiler-rt maintained bit have been
> already added by the recent merge from compiler-rt, so this
> patch is mostly configury/Makefile stuff. Rather than using
> libbacktrace.la built in li
This fixes PR59715 by splitting critical edges again before
code sinking. The critical edge splitting done before PRE
was designed to survive until sinking originally, but at least
since 4.5 PRE now eventually cleans up the CFG and thus undos
critical edge splitting. This results in less than op
On Thu, Jan 09, 2014 at 09:02:28AM +, Richard Sandiford wrote:
> > I think Jakub's patch will fix this case, but I did not try.
> > However even if the i368 is now clean, there are
> > still many targets that use target_reinit() in
> > target_set_current_function.
>
> FWIW I only see three oth
On Thu, 9 Jan 2014 10:28:43, Jakub Jelinek wrote:
>
> On Thu, Jan 09, 2014 at 09:02:28AM +, Richard Sandiford wrote:
>>> I think Jakub's patch will fix this case, but I did not try.
>>> However even if the i368 is now clean, there are
>>> still many targets that use target_reinit() in
>>> targe
On Thu, Jan 09, 2014 at 10:36:43AM +0100, Bernd Edlinger wrote:
> On Thu, 9 Jan 2014 10:28:43, Jakub Jelinek wrote:
> >
> > On Thu, Jan 09, 2014 at 09:02:28AM +, Richard Sandiford wrote:
> >>> I think Jakub's patch will fix this case, but I did not try.
> >>> However even if the i368 is now cle
On Wed, Jan 8, 2014 at 8:46 PM, Steven Bosscher wrote:
> On Wed, Jan 8, 2014 at 8:27 PM, Richard Sandiford wrote:
>> gcc/
>> PR rtl-optimization/59137
>> * reorg.c (steal_delay_list_from_target): Call update_block for
>> elided insns.
>> (steal_delay_list_from_fallt
On 01/08/2014 11:47 PM, Tim Shen wrote:
On Wed, Jan 8, 2014 at 5:38 PM, Paolo Carlini wrote:
I agree, it's probably fine for now, but please actually attach the patch ;)
Oops sorry >.<
Jon, is this version Ok with you?
Thanks,
Paolo.
On Thu, Jan 09, 2014 at 09:02:28AM +, Richard Sandiford wrote:
> It looks like a correct fix, but the memcpy is going to be pretty
> expensive, since in most cases there will be no difference.
Perhaps we should add another tree code, which would represent the
combination of TARGET_OPTION_NODE
Hello!
Attached patch skip some go tests to avoid testsuite failures:
- peano.go tests recursive call. The test fails on targets that don't
support -fsplit-stack (Centos 5.3, alpha)
- rotate[0123]-out.go take too long to compile. The test should be
skipped for the same reason as rotate.go:
On Thu, Jan 09, 2014 at 09:46:11AM +0100, Richard Biener wrote:
> > + /* If fndecl (like __builtin_unreachable or
> > +__cxa_pure_virtual) takes no arguments, doesn't have
> > +return value and is noreturn, if the call doesn't have
> > +lhs or lhs isn't S
On Thu, Jan 9, 2014 at 11:24 AM, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 09:02:28AM +, Richard Sandiford wrote:
>> It looks like a correct fix, but the memcpy is going to be pretty
>> expensive, since in most cases there will be no difference.
>
> Perhaps we should add another tree code
On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 09:46:11AM +0100, Richard Biener wrote:
> > > + /* If fndecl (like __builtin_unreachable or
> > > + __cxa_pure_virtual) takes no arguments, doesn't have
> > > + return value and is noreturn, if the call does
On Thu, Jan 09, 2014 at 12:11:12PM +0100, Richard Biener wrote:
> On Thu, Jan 9, 2014 at 11:24 AM, Jakub Jelinek wrote:
> > On Thu, Jan 09, 2014 at 09:02:28AM +, Richard Sandiford wrote:
> >> It looks like a correct fix, but the memcpy is going to be pretty
> >> expensive, since in most cases
On Thu, Jan 09, 2014 at 12:15:00PM +0100, Richard Biener wrote:
> > Well, then the __cxa_pure_virtual testcases ICE again, but the pr59622-5.C
> > testcase ICEs anyway, so here is a different patch (untested so far except
> > for the tests). The issue with the calls is that when fold_stmt is done
Ping!
The patch is also ok with me. Regarding the wording I'd vote for
Tobias' suggestion "NULL() intrinsic not permitted in data-transfer
statement", but I'm also ok with the other variants.
Paul, please commit.
Cheers,
Janus
2013/12/1 Tobias Burnus :
> Paul Richard Thomas wrote:
>>
>> This
On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 12:15:00PM +0100, Richard Biener wrote:
> > > Well, then the __cxa_pure_virtual testcases ICE again, but the pr59622-5.C
> > > testcase ICEs anyway, so here is a different patch (untested so far except
> > > for the tests). The is
Jakub Jelinek a écrit:
>
> 2013-12-10 Jakub Jelinek
>
> * sanitizer_common/sanitizer_symbolizer_libbacktrace.h
> (LibbacktraceSymbolizer::Demangle): New declaration.
> * sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc
> (POSIXSymbolizer::Demangle): Use libbacktr
On Thu, 9 Jan 2014, Dominique Dhumieres wrote:
> > This fixes the gimple verification ICEs in the Fortran testsuite.
> > ...
>
> This patch causes 6300+ regressions (-m32/-m64 all languages but go).
> However the following change
>
> --- ../_clean/gcc/lto/lto.c 2014-01-04 15:51:44.0 +0
Hello,
and Ping for this patch:
http://gcc.gnu.org/ml/gcc-patches/2013-12/msg01552.html
note however, that cross-building is probably broken
anyway in the moment by r205690,
Thanks
Bernd.
> there is a small problem with SSIZE_MAX, because it is not always
> defined, especially not in gcc/gli
On Thu, Jan 09, 2014 at 12:31:47PM +0100, Richard Biener wrote:
> On Thu, 9 Jan 2014, Jakub Jelinek wrote:
>
> > On Thu, Jan 09, 2014 at 12:15:00PM +0100, Richard Biener wrote:
> > > > Well, then the __cxa_pure_virtual testcases ICE again, but the
> > > > pr59622-5.C
> > > > testcase ICEs anyway,
>
> I don't understand why static variables can cause any safety issue in
> multi-thread programs.
> For any process, gcov_dump will be called only once and there will be
> one instance of globals.
(v)fork, failing exec and other cases can lead to gcov_flush being called
several times and that c
On 09-01-14 10:16, Richard Biener wrote:
This fixes PR59715 by splitting critical edges again before
code sinking. The critical edge splitting done before PRE
was designed to survive until sinking originally, but at least
since 4.5 PRE now eventually cleans up the CFG and thus undos
critical ed
On Fri, Dec 27, 2013 at 10:27 AM, François Dumont wrote:
> Hi
>
> Here is a patch to fix an issue in normal mode during the move
> assignment. The destination vector allocator instance is moved too during
> the assignment which is wrong.
>
> As I discover this problem while working on issu
On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 12:31:47PM +0100, Richard Biener wrote:
> > On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> >
> > > On Thu, Jan 09, 2014 at 12:15:00PM +0100, Richard Biener wrote:
> > > > > Well, then the __cxa_pure_virtual testcases ICE again, but th
On Thu, 9 Jan 2014, Tom de Vries wrote:
> On 09-01-14 10:16, Richard Biener wrote:
> >
> > This fixes PR59715 by splitting critical edges again before
> > code sinking. The critical edge splitting done before PRE
> > was designed to survive until sinking originally, but at least
> > since 4.5 PR
On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 12:48:49PM +0100, Richard Biener wrote:
> > *** gimple_canonical_types_compatible_p (tre
> > *** 458,465
> > return true;
> >
> > /* Can't be the same type if they have different alignment, or mode. *
On Thu, Jan 09, 2014 at 01:30:53PM +0100, Richard Biener wrote:
> > gimplify_modify_expr has:
> >
> > if (!gimple_call_noreturn_p (assign))
> > gimple_call_set_lhs (assign, *to_p);
>
> Ok, it seems to be too early then - move it after the folding.
That wouldn't help all the other e
On Thu, Jan 09, 2014 at 12:48:49PM +0100, Richard Biener wrote:
> *** gimple_canonical_types_compatible_p (tre
> *** 458,465
> return true;
>
> /* Can't be the same type if they have different alignment, or mode. */
> ! if (TYPE_ALIGN (t1) != TYPE_ALIGN (t2)
> !
On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 01:30:53PM +0100, Richard Biener wrote:
> > > gimplify_modify_expr has:
> > >
> > > if (!gimple_call_noreturn_p (assign))
> > > gimple_call_set_lhs (assign, *to_p);
> >
> > Ok, it seems to be too early then - move i
Hello,
I have reproduced the problem with mklog mentioned by Jakub:
> In my experience mklog is pretty much useless, e.g. if you add a new
> function, it will list the previous function as being modified rather
> than the new one, etc.
My focus was on functions from headers of diff-log chunks.
On Thu, Jan 09, 2014 at 02:13:39PM +0100, Richard Biener wrote:
> On Thu, 9 Jan 2014, Jakub Jelinek wrote:
>
> > On Thu, Jan 09, 2014 at 01:30:53PM +0100, Richard Biener wrote:
> > > > gimplify_modify_expr has:
> > > >
> > > > if (!gimple_call_noreturn_p (assign))
> > > > gimple_cal
On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> On Thu, Jan 09, 2014 at 02:13:39PM +0100, Richard Biener wrote:
> > On Thu, 9 Jan 2014, Jakub Jelinek wrote:
> >
> > > On Thu, Jan 09, 2014 at 01:30:53PM +0100, Richard Biener wrote:
> > > > > gimplify_modify_expr has:
> > > > >
> > > > > if (!gim
After noticing that the bug is actually a regression (see PR 57042):
Ok to backport to 4.7 and 4.8?
Cheers,
Janus
2013/12/29 Janus Weil :
> Hi all,
>
> I have just committed an obvious patch for a segfault with
> -fdump-fortran-original (plus a small documentation fix):
>
> http://gcc.gnu.org/v
On Tue, Dec 10, 2013 at 3:38 PM, Jakub Jelinek wrote:
> On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote:
>> There was a recent buggy patch to the demangler that added calls to
>> malloc and realloc (2013-10-25 Gary Benson ).
>> That patch must be fixed or reverted before the 4.9 r
On 25/12/13 14:02, Tom de Vries wrote:
> On 07-12-13 16:07, Tom de Vries wrote:
>> Richard,
>>
>> This patch implements the target hook TARGET_FN_OTHER_HARD_REG_USAGE (posted
>> here: http://gcc.gnu.org/ml/gcc-patches/2013-03/msg01318.html) for MIPS, to
>> address the issue that $6 is sometimes use
On Thu, Jan 09, 2014 at 05:51:05PM +0400, Konstantin Serebryany wrote:
> On Tue, Dec 10, 2013 at 3:38 PM, Jakub Jelinek wrote:
> > On Fri, Dec 06, 2013 at 06:40:52AM -0800, Ian Lance Taylor wrote:
> >> There was a recent buggy patch to the demangler that added calls to
> >> malloc and realloc (201
On 01/09/14 06:02, Jeff Law wrote:
On 01/08/14 02:05, Laurent Alfonsi wrote:
All,
I was looking at PR49718. I have enclosed a simple fix for this bug report.
2014-01-07 Laurent Alfonsi
* c-family/c-common.c (handle_no_instrument_function_attribute): Allow
no_instrument_f
Hi all,
I'm confused by the annotation in shold_replace_address.
Here is the code in fwprop.c:
/* OLD is a memory address. Return whether it is good to use NEW instead,
for a memory access in the given MODE. */
static bool
should_replace_address (rtx old_rtx, rtx new_rtx, enum machin
As suggested by Honza, avoid bloating libgcov from gcc_assert by using
a new macro gcov_nonruntime_assert in gcov-io.c that is only mapped to
gcc_assert when not in libgcov.
Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk?
Thanks,
Teresa
2014-01-09 Teresa Johnson
*
On 30/03/13 16:10, Tom de Vries wrote:
> On 29/03/13 13:54, Tom de Vries wrote:
>> I split the patch up into 10 patches, to facilitate further review:
>> ...
>> 0001-Add-command-line-option.patch
>> 0002-Add-new-reg-note-REG_CALL_DECL.patch
>> 0003-Add-implicit-parameter-to-find_all_hard_reg_sets.p
> As suggested by Honza, avoid bloating libgcov from gcc_assert by using
> a new macro gcov_nonruntime_assert in gcov-io.c that is only mapped to
> gcc_assert when not in libgcov.
>
> Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk?
>
> Thanks,
> Teresa
>
> 2014-01-09 Teresa J
On Thu, Jan 9, 2014 at 2:54 AM, Uros Bizjak wrote:
>
> 2014-01-09 Uros Bizjak
>
> * go.test/go-test.exp (go-gc-tests): Don't run peano.go on systems
> which don't support -fsplit-stack. Skip rotate[0123]-out.go.
This is OK. Thanks.
You might want to tweak the comment just under wher
Hi Gerald,
Sorry for the late reply!
We're working on a list of all the ARM-related changes in 4.9. This will
also be included.
Kind regards,
Renlin
On 03/01/14 13:24, Gerald Pfeifer wrote:
Renlin Li wrote:
Hi all,
This patch will add armv7ve support to gcc. Armv7ve is basically a
armv7-a
Hi Sterling,
Attached please find version 2 of the patch.
I applied this updated patch (with small adaptations) to gcc-4.8.2
and carried out some tests.
I can execute the testcases in a simulator, which support
zero-overhead looping instructions.
First of all, I can successfully
On Sat, Nov 23, 2013 at 8:20 PM, Mike Stump wrote:
> Richi has asked the we break the wide-int patch so that the individual port
> and front end maintainers can review their parts without have to go through
> the entire patch.This patch covers the build system (make).
>
> Ok?
Needs updating
On Sat, Nov 23, 2013 at 8:21 PM, Mike Stump wrote:
> Richi has asked the we break the wide-int patch so that the individual port
> and front end maintainers can review their parts without have to go through
> the entire patch.This patch covers the documentation.
>
> Ok?
Ok.
Thanks,
Richard
On Thu, Jan 2, 2014 at 5:10 AM, Mike Stump wrote:
> On Nov 28, 2013, at 6:20 AM, Richard Biener
> wrote:
>> On Thu, Nov 28, 2013 at 12:58 PM, Richard Sandiford
>> wrote:
>>> Jakub Jelinek writes:
On Mon, Nov 25, 2013 at 12:24:30PM +0100, Richard Biener wrote:
> On Sat, Nov 23, 2013 at
On Thu, Jan 2, 2014 at 5:12 AM, Mike Stump wrote:
> On Nov 23, 2013, at 11:22 AM, Mike Stump wrote:
>> Richi has asked the we break the wide-int patch so that the individual port
>> and front end maintainers can review their parts without have to go through
>> the entire patch.This patch co
On Thu, Jan 2, 2014 at 5:53 AM, Mike Stump wrote:
> On Nov 23, 2013, at 11:22 AM, Mike Stump wrote:
>> Richi has asked the we break the wide-int patch so that the individual port
>> and front end maintainers can review their parts without have to go through
>> the entire patch.This patch co
On Thu, Jan 2, 2014 at 5:27 AM, Mike Stump wrote:
> On Nov 26, 2013, at 1:14 AM, Richard Biener
> wrote:
@@ -2662,8 +2661,8 @@ iv_number_of_iterations (struct loop *loop, rtx
insn, rtx condition,
iv1.step = const0_rtx;
if (INTVAL (iv0.step) < 0)
{
>
Hi, Richard.
This week I've been working on THUMB code size issues. Here is the
prototype of the patch for spilling into HI_REGS instead of memory.
The patch decreases number of generated insns and makes the code faster
as it removes a lot of loads/stores.
I am sending the patch for your ev
On 20/11/13 13:57, Richard Earnshaw wrote:
> On 19/11/13 17:48, Jeff Law wrote:
>> On 11/19/13 10:32, Steven Bosscher wrote:
>>>
>>> Yes. In the GCC3 days it was important for sincos on i386, and on mk68
>>> it used to be important for some of the funnier patterns. Not sure if
>>> it's still useful
Hi all,
the attached patch started out as an ICE-on-invalid regression fix,
but after the ICE had been fixed recently by other means, it was
degraded to a mere error-recovery improvement. It removes some rather
'hackish' code that was added by Paul quite a long time ago.
Regtests cleanly on x86_6
On 09/01/14 15:21, Vladimir Makarov wrote:
> Hi, Richard.
>
>This week I've been working on THUMB code size issues. Here is the
> prototype of the patch for spilling into HI_REGS instead of memory.
> The patch decreases number of generated insns and makes the code faster
> as it removes a lot
Tom de Vries writes:
> On 25/12/13 14:02, Tom de Vries wrote:
>> On 07-12-13 16:07, Tom de Vries wrote:
>>> Richard,
>>>
>>> This patch implements the target hook TARGET_FN_OTHER_HARD_REG_USAGE (posted
>>> here: http://gcc.gnu.org/ml/gcc-patches/2013-03/msg01318.html) for MIPS, to
>>> address the
Hi,
This patch and the preceding aarch64.md patches all look good to me, but
I cannot approve it.
Thanks for adding the support for these missing patterns and defines!
Yufeng
On 01/08/14 18:13, Richard Henderson wrote:
We have good support for TImode arithmetic, so no need to do anything
wi
On Wed, Dec 18, 2013 at 5:02 AM, Kirill Yukhin wrote:
> Hello,
>
> On 02 Dec 16:10, Kirill Yukhin wrote:
>> Hello,
>> On 19 Nov 12:11, Kirill Yukhin wrote:
>> > Hello,
>> > On 15 Nov 20:07, Kirill Yukhin wrote:
>> > > > Is it ok for trunk?
>> > > Ping.
>> > Ping.
>> Ping.
> Ping.
>
> Rebased patch
On 01/08/2014 12:39 PM, Joseph S. Myers wrote:
> On Wed, 8 Jan 2014, Richard Henderson wrote:
>
>> diff --git a/libgcc/soft-fp/soft-fp.h b/libgcc/soft-fp/soft-fp.h
>> index 696fc86..b54b1ed 100644
>> --- a/libgcc/soft-fp/soft-fp.h
>> +++ b/libgcc/soft-fp/soft-fp.h
>> @@ -237,6 +237,11 @@ typedef i
On 01/08/2014 04:45 AM, Jakub Jelinek wrote:
> 2014-01-07 Jakub Jelinek
>
> PR target/58115
> * tree-core.h (struct target_globals): New forward declaration.
> (struct tree_target_option): Add globals field.
> * tree.h (TREE_TARGET_GLOBALS): Define.
> (prepare_targ
Hi,
this patch fixes IPA-devirt testcase that gave me bad sleep for months. The
problem turned out to be combination of three issues (that greatly confused
me). This patch fixes first two. Here representation of BINFOs of multiple
inheritnace actually differs from my mental modem. For diamond sha
On 1/9/2014, 10:30 AM, Richard Earnshaw wrote:
On 09/01/14 15:21, Vladimir Makarov wrote:
Hi, Richard.
This week I've been working on THUMB code size issues. Here is the
prototype of the patch for spilling into HI_REGS instead of memory.
The patch decreases number of generated insns and ma
On 01/08/2014 10:34 AM, Jakub Jelinek wrote:
>struct target_globals *g;
> -
> - g = ggc_alloc_target_globals ();
> - g->flag_state = XCNEW (struct target_flag_state);
> - g->regs = XCNEW (struct target_regs);
> + struct target_globals_extra {
> +struct target_globals g;
> +struct ta
Hello Jakub,
Did you get a chance to look at this patch
(http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00116.html)? I think I have fixed
all the changes you requested. Is it ok for trunk?
Thanks,
Balaji V. Iyer.
On Thu, Jan 09, 2014 at 08:25:31AM -0800, Richard Henderson wrote:
> > + p = (struct target_globals_extra *)
> > + ggc_internal_cleared_alloc_stat (sizeof (struct target_globals_extra)
> > + PASS_MEM_STAT);
> > + g = (struct target_globals *) p;
> > + g->fla
On 01/09/2014 08:35 AM, Jakub Jelinek wrote:
> That would be fine for 1), but would mean 2). It is also fine to GC
> allocate each structure individually, but some (like bb_reorder) are say
> just 4 bytes long, so it might be overkill.
Hmm.. Perhaps define the whole structure as you do, but som
Hello Everyone,
The following patch will fix the bug in PR 59094. The main issue was
that version specific libraries are not stored in the correct location. The
patch below should fix that. It is committed since the person who filed the bug
has confirmed that the fix works.
Index: libci
On Wed, Jan 08, 2014 at 12:28:56PM +, Jakub Jelinek wrote:
> On Wed, Jan 08, 2014 at 11:49:08AM +, Vidya Praveen wrote:
> > On Tue, Jan 07, 2014 at 09:35:54PM +, Mike Stump wrote:
> > > On Dec 17, 2013, at 6:06 AM, Vidya Praveen wrote:
> > > > bind_pic_locally is broken for targets tha
Hi all,
When adding the testsuite options for the crypto tests we need to make sure that
don't end up adding -mfloat-abi=softfp to a hard-float target like
arm-none-linux-gnueabihf. This patch adds that code to figure out which
-mfpu/-mfloat-abi combination to use in a similar approach to the
On Thu, Jan 9, 2014 at 4:01 PM, Ian Lance Taylor wrote:
> On Thu, Jan 9, 2014 at 2:54 AM, Uros Bizjak wrote:
>>
>> 2014-01-09 Uros Bizjak
>>
>> * go.test/go-test.exp (go-gc-tests): Don't run peano.go on systems
>> which don't support -fsplit-stack. Skip rotate[0123]-out.go.
>
> This i
Hi all,
After my CRC32 intrinsics patch that added new entries into the bdesc_2arg
table, the arm_init_iwmmxt_builtins function tries to iterate over them and
blows up, causing an ICE when trying to compile with -mcpu=iwmmxt.
This patch fixes that by ignoring the non-iwmmxt entries in that ta
Hi all,
The Cortex-A53 and Cortex-A57 processors support the CRC32 extensions to
ARMv8-a, so we specify that in their definitions in arm-cores.def.
This also updates their big.LITTLE amalgamation and removes the redundant
FL_THUMB_DIV and FL_ARM_DIV there since ARMv8-a already implies those fla
Hi all,
SET RTXs don't have a mode, so the code to calculate a reg-to-reg set in the arm
rtx costs function needs to get the mode from one of the registers involved. We
already did that when the source is a CONST_INT.
This patch fixes that oversight and also prevents us from falling through o
While looking at PR 59335 (plugin doesn't build) I saw the comments about
tree-flow.h and tree-flow-inline.h not existing anymore. While these
files have been removed there are still some references to them in
Makefile.in, doc/tree-ssa.texi, and a couple of source files. This patch
removes the re
Hi,
this just adds the testcase to mainline.
Thanks,
Paolo.
//
2014-01-09 Paolo Carlini
PR c++/59730
* g++.dg/cpp0x/variadic145.C: New.
Index: g++.dg/cpp0x/variadic145.C
===
--- g++.dg/cp
ping patch
Am 29.11.2013 14:29, schrieb Matthias Klose:
> to get full advantage of the -fsplit-stack option, gccgo binaries have to be
> linked with gold, not the bfd linker. When the system linker defaults to the
> bfd linker, then gccgo should explicitly use the gold linker, passing
> fuse-ld=g
On Thu, Jan 9, 2014 at 2:54 AM, Uros Bizjak wrote:
>
> There are two remaining warnings:
>
> go.test/test/nilcheck.go: unrecognized test line: // errorcheck -0 -N -d=nil
> go.test/test/nilptr3.go: unrecognized test line: // errorcheck -0 -d=nil
Thanks, not sure how I missed those. Those tests ar
Thanks for the comments! Here is a second go-round at the patch with
improved documentation. I'm happy to change the wording if it can be
further improved.
Thanks,
Bill
2014-01-09 Bill Schmidt
* doc/invoke.texi: Add -maltivec={be,le} options, and document
default element-ord
Hi!
As discussed in the PR, if a var isn't addressable and has gimple reg type,
I don't see any point to honor it's DECL_ALIGN, we only refer to the
var through SSA_NAME_VAR of SSA_NAMEs, nothing is allocated on the stack
immediately and the SSA_NAMEs are turned into pseudos for which we only care
On Thu, Jan 09, 2014 at 02:27:40PM +0100, Richard Biener wrote:
> > Perhaps, if you don't like the !gimple_in_ssa_p (cfun) in the condition
> > we can just drop the lhs always in that case, just doing what we do for
> > __builtin_unreachable if lhs is SSA_NAME:
> > tree var = create_tmp_var (TREE
Hi,
This patch puts a breakpoint on __sanitizer::Report to help
with debugging sanitizer issues. OK to install?
Thanks.
--
H.J.
--
2014-01-09 H.J. Lu
* gdbasan.in: Put a breakpoint on __sanitizer::Report.
diff --git a/gcc/gdbasan.in b/gcc/gdbasan.in
index cf05825..3a6fca0 100644
--- a/gcc
On 01/06/14 13:40, Richard Henderson wrote:
On 12/19/2013 11:06 AM, Richard Biener wrote:
Aldy Hernandez wrote:
I'd still like to catch the common cases, like I do with this patch.
Perhaps we move this code to the .tmmark pass and handle the
uninstrumented case. rth?
tmmark is way way late
On Thu, Jan 09, 2014 at 10:28:56AM -0800, H.J. Lu wrote:
> Hi,
>
> This patch puts a breakpoint on __sanitizer::Report to help
> with debugging sanitizer issues. OK to install?
Ok.
> 2014-01-09 H.J. Lu
>
> * gdbasan.in: Put a breakpoint on __sanitizer::Report.
> diff --git a/gcc/gdbasan.in
On Thu, 9 Jan 2014, Richard Henderson wrote:
> > This isn't the right conditional. _FP_W_TYPE_SIZE is ultimately an
> > optimization choice and need not be related to whether any TImode
> > functions are being defined using soft-fp, or whether TImode is supported
> > at all. I think the most
On 9 January 2014 12:22, H.J. Lu wrote:
> On Fri, Dec 27, 2013 at 10:27 AM, François Dumont
> wrote:
>> Hi
>>
>> Here is a patch to fix an issue in normal mode during the move
>> assignment. The destination vector allocator instance is moved too during
>> the assignment which is wrong.
>>
>>
On Thu, 9 Jan 2014, Bill Schmidt wrote:
> +When -maltivec is used, rather than -maltivec=le or -maltivec=be, the
> +element order for Altivec intrinsics such as vec_splat, vec_extract,
> +and vec_insert will match array element order corresponding to the
> +endianness of the target. That is, elem
The gcc.dg/delay-slot-1.c test is failing for MIPS targets that
do not support the 64 bit ABI because it didn't check to see
if that support existed before using the -mabi=64 flag.
This patch fixes the problem by using the mips64 check.
OK to checkin?
Steve Ellcey
sell...@mips.com
2014-01-09
On Thu, Jan 9, 2014 at 1:14 PM, Bill Schmidt
wrote:
> Thanks for the comments! Here is a second go-round at the patch with
> improved documentation. I'm happy to change the wording if it can be
> further improved.
>
> Thanks,
> Bill
>
> 2014-01-09 Bill Schmidt
>
> * doc/invoke.texi: A
"Steve Ellcey " writes:
> 2014-01-09 Steve Ellcey
>
> * gcc.dg/delay-slot-1.c: Add check for 64 bit support.
OK, thanks. Pedantically it's "Restrict -mabi=64 to 64-bit processors.",
since we're not really checking whether the support is there, but whether
n64 is compatible with the curr
Hi Uros,
btver1 iis never used. This patch removes it. It avoids:
insn-attrtab.c:extern int internal_dfa_insn_code_btver1 (rtx);
insn-attrtab.c:extern int insn_default_latency_btver1 (rtx);
OK to install?
Thanks.
--
H.J.
--
2014-01-09 H.J. Lu
* config/i386/i386.md (cpu): Remove the unu
On Thu, Jan 9, 2014 at 8:32 PM, H.J. Lu wrote:
> btver1 iis never used. This patch removes it. It avoids:
>
> insn-attrtab.c:extern int internal_dfa_insn_code_btver1 (rtx);
> insn-attrtab.c:extern int insn_default_latency_btver1 (rtx);
>
> OK to install?
OK.
Thanks,
Uros.
On Wed, Jan 8, 2014 at 2:33 PM, Rong Xu wrote:
> On Fri, Dec 6, 2013 at 6:23 AM, Jan Hubicka wrote:
>>> @@ -325,6 +311,9 @@ static struct gcov_summary all_prg;
>>> #endif
>>> /* crc32 for this program. */
>>> static gcov_unsigned_t crc32;
>>> +/* Use this summary checksum rather the computed
My bad.
Thanks for the fix!
-Rong
On Thu, Jan 9, 2014 at 11:47 AM, H.J. Lu wrote:
> On Wed, Jan 8, 2014 at 2:33 PM, Rong Xu wrote:
>> On Fri, Dec 6, 2013 at 6:23 AM, Jan Hubicka wrote:
@@ -325,6 +311,9 @@ static struct gcov_summary all_prg;
#endif
/* crc32 for this program. *
Committed as obvious: Rev. 206487.
Tobias
Index: gcc/fortran/ChangeLog
===
--- gcc/fortran/ChangeLog (Revision 206486)
+++ gcc/fortran/ChangeLog (Arbeitskopie)
@@ -1,3 +1,8 @@
+2014-01-09 Tobias Burnus
+
+ * cpp.c (gfc_cpp_handle_
Jakub Jelinek wrote:
>On Thu, Jan 09, 2014 at 02:27:40PM +0100, Richard Biener wrote:
>> > Perhaps, if you don't like the !gimple_in_ssa_p (cfun) in the
>condition
>> > we can just drop the lhs always in that case, just doing what we do
>for
>> > __builtin_unreachable if lhs is SSA_NAME:
>> > tr
1 - 100 of 133 matches
Mail list logo
