> On Mon, Nov 11, 2013 at 8:22 AM, Jan Hubicka wrote:
> >> I have a warning like that already in drop_profile(). Is that
> >
> > I think it should be warning (or silent) for COMDAT and error/note for
> > other functions (depending on flag_profile_correction).
> > I guess drop_profile is better pla
On 11/07/13 10:58, Martin Jambor wrote:
Sorry for the delay. I'd just like to re-iterate one comment from my
previous email which is that I do not think tree-sra.c should export
any function to the outside world apart from the entry points of the
passes (yes, there is already build_ref_for_offs
On Mon, Nov 11, 2013 at 9:13 AM, Jan Hubicka wrote:
>> On Mon, Nov 11, 2013 at 8:22 AM, Jan Hubicka wrote:
>> >> I have a warning like that already in drop_profile(). Is that
>> >
>> > I think it should be warning (or silent) for COMDAT and error/note for
>> > other functions (depending on flag_p
Il 08/11/2013 10:37, Richard Biener ha scritto:
> /* If we have a gate, combine the properties that we could have with
> and without the pass being examined. */
> if (pass->has_gate)
> properties &= new_properties;
> else
> properties = new_properties;
>
> But throw what? It is up to the runtimes of -fnon-call-exceptions languages
> to decide if they actually want to throw or do something else in the signal
> handlers, and what exactly to throw.
Throw nothing per se, __builtin_throw would simply trap and ensure that the
exception thrown by the s
Ping.
On Thu, Nov 7, 2013 at 11:14 AM, Easwaran Raman wrote:
> Before r193504, if a method can not be overridden, LOOKUP_NONVIRTUAL
> is set and the call is direct. The changes at r193504 (to fix PR
> c++/11750) caused a regression to this behavior. This patch attempts
> to fix that. Bootstraps
In Go 1.2 the language was changed slightly to prohibit taking the
address of a field in a struct given a nil pointer to the struct
(http://golang.org/s/go12nil). This patch by Chris Manghane implements
that check in gccgo. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed
On 11 November 2013 15:57, Jan Hubicka wrote:
>> +2013-10-29 Martin Liska
>> + Jan Hubicka
>> +
>> + * cgraph.c (dump_cgraph_node): Profile dump added.
>> + * cgraph.h (struct cgraph_node): New time profile variable added.
>> + * cgraphc
Hi,
thanks for the followup. I like it, I only don't understand...
On Fri, Nov 08, 2013 at 10:48:43AM -0700, Aldy Hernandez wrote:
> On 11/07/13 09:09, Martin Jambor wrote:
>
<...>
> --- a/gcc/ipa-prop.h
> +++ b/gcc/ipa-prop.h
> @@ -609,6 +609,31 @@ extern alloc_pool ipcp_values_pool;
> exte
On Nov 11, 2013, at 8:35 AM, David Edelsohn wrote:
> I continue to see extra Cilk Plus failures on AIX. Tests that require
> LTO should check that the target supports LTO.
Iyer, sprinkle /* { dg-require-effective-target lto } */ on those test cases
and see if they then work ok as expected.
On Nov 10, 2013, at 8:28 AM, H.J. Lu wrote:
> This patch documents __builtin_setjmp and __builtin_longjmp. OK to
> install?
I like it. I think these two should be documented.
> But for "USE mod_name, only: nml", one is supposed to generate a
> DW_TAG_imported_declaration.
>
> And there I am stuck. For normal variables, the DW_TAG_imported_declaration
> refers to a DW_TAG_variable die. Analogously, for a namelist one would have
> to refer to a DW_TAG_namelist die. But su
On Mon, Nov 11, 2013 at 7:02 AM, Jan Hubicka wrote:
>> 2013-11-04 Rong Xu
>>
>> * libgcc/libgcov.c: Delete as part of re-factoring.
>> * libgcc/libgcov-profiler.c (__gcov_interval_profiler): Moved from
>> libgcov.c
>> (__gcov_pow2_profiler): Ditto.
>> (__gcov_one_v
On Mon, Nov 11, 2013 at 9:41 AM, Eric Botcazou wrote:
>> But throw what? It is up to the runtimes of -fnon-call-exceptions languages
>> to decide if they actually want to throw or do something else in the signal
>> handlers, and what exactly to throw.
>
> Throw nothing per se, __builtin_throw wou
On Mon, Nov 11, 2013 at 06:57:39PM +0100, Martin Jambor wrote:
> > --- a/gcc/ipa-prop.h
> > +++ b/gcc/ipa-prop.h
> > @@ -609,6 +609,31 @@ extern alloc_pool ipcp_values_pool;
> > extern alloc_pool ipcp_sources_pool;
> > extern alloc_pool ipcp_agg_lattice_pool;
> >
> > +/* Operation to be perform
On 11/11/13 07:32, Ulrich Weigand wrote:
Hello,
when implementing the new ABI for powerpc64le-linux, I ran into an assertion
failure in store_unaligned_arguments_into_pseudos:
gcc_assert (args[i].partial % UNITS_PER_WORD == 0);
This can happen in the new ABI since we pass "homogene
On 11/11/13 02:32, Richard Biener wrote:
On Fri, 8 Nov 2013, Cong Hou wrote:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59050
This is my bad. I forget to check the test result for gfortran. With
this patch the bug should be fixed (tested on x86-64).
Ok.
Btw, requirements are to bootstrap a
2013-11-11 Joern Rennecke
* gcc.dg/tree-ssa/forwprop-28.c: Adjust for ARC
LOGICAL_OP_NON_SHORT_CIRCUIT definition.
* gcc.dg/tree-ssa/ssa-dom-thread-4.c: Likewise.
* gcc.dg/tree-ssa/ssa-ifcombine-ccmp-1.c: Likewise.
* gcc.dg/tree-ssa/ssa-ifcombine-ccm
> Simply trapping doesn't tell you anything about caused the trap.
> There are at least two distinct possibilities: NULL pointer
> dereference and integer division by zero. There should be some way to
> distinguish those two cases. It's reasonably easy to do so in a
> signal handler. When should
On Mon, Nov 11, 2013 at 7:17 AM, Jan Hubicka wrote:
>> Here is the patch that includes profile-tool.
>> Profile-tool now has two functions: merge and rewrite. I'll add diff later.
>>
>> Compiler is tested with spec2006 and profiledbootstrap.
>> profile-tool is tested with spec2006 profiles.
>
> Hi
Hi Jeff
I have committed the fix. Please update your repo.
Thank you!
Cong
On Mon, Nov 11, 2013 at 10:32 AM, Jeff Law wrote:
> On 11/11/13 02:32, Richard Biener wrote:
>>
>> On Fri, 8 Nov 2013, Cong Hou wrote:
>>
>>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59050
>>>
>>> This is my bad.
Thank you for your advice! I will follow this instruction in future.
thanks,
Cong
On Mon, Nov 11, 2013 at 1:32 AM, Richard Biener wrote:
> On Fri, 8 Nov 2013, Cong Hou wrote:
>
>> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59050
>>
>> This is my bad. I forget to check the test result for gfor
On Sun, 10 Nov 2013, H.J. Lu wrote:
> +@code{__builtin_setjmp} and @code{__builtin_longjmp} may not be used
> +in the same function with the same @code{setjmp} buffer.
Are there any other restrictions? Do they, for example, guarantee to
restore all call-saved registers to their values at the ti
Jeff Law wrote:
> On 11/11/13 07:32, Ulrich Weigand wrote:
> > However, looking more closely, it seems
> > store_unaligned_arguments_into_pseudos
> > is not really useful for PARALLEL arguments in the first place. What this
> > routine does is load arguments into args[i].aligned_regs. But if we
On Mon, Nov 11, 2013 at 10:55 AM, Eric Botcazou wrote:
>> Simply trapping doesn't tell you anything about caused the trap.
>> There are at least two distinct possibilities: NULL pointer
>> dereference and integer division by zero. There should be some way to
>> distinguish those two cases. It's
Hi James
Sorry for the late reply.
On Fri, Nov 8, 2013 at 2:55 AM, James Greenhalgh
wrote:
>> On Tue, Nov 5, 2013 at 9:58 AM, Cong Hou wrote:
>> > Thank you for your detailed explanation.
>> >
>> > Once GCC detects a reduction operation, it will automatically
>> > accumulate all elements in th
On Fri, Nov 8, 2013 at 1:10 AM, Uros Bizjak wrote:
> On Fri, Nov 8, 2013 at 12:39 AM, Ian Lance Taylor wrote:
>
>>> Recent Go mega-patch broke Alpha bootstrap. Following fixlet is needed:
>>>
>> Thanks for the patch and report. This patch should fix them.
>> Bootstrapped and tested on x86_64-unk
On 2013-11-10 23:21, Jason Merrill wrote:
On 11/10/2013 02:39 PM, Adam Butcher wrote:
I assumed that tsubst simply doesn't do anything with a
null tree substitution (i.e. it is an identity op).
Substituting NULL_TREE for a template parameter gives a template
parameter with a reduced level; thi
On Fri, Nov 8, 2013 at 1:10 AM, Uros Bizjak wrote:
>
> panic: runtime error: invalid memory address or nil pointer dereference
> [signal 0xb code=0x1 addr=0x1c]
>
> goroutine 5 [running]:
> syscall.Exitsyscall
> ../../../gcc-svn/trunk/libgo/runtime/proc.c:1986
> pprof.profileWriter
>
Hello!
Attached patch fixes a typo in ix86_expand_set_or_movmem, where a
wrong define was used in a condition. The patch also adds additional
condition (as proposed by H.J.) as a correctness improvement (although
patched gcc bootstraps and regression tests OK without).
I took the liberty to rena
This was/is relatively painful, and there will be another set after this
to really finish it.
The front ends and the middle end both make extensive use of
gimplification There is no way at this early point to really separate
them unless we created a gimplify-fe.[ch] and gimplfy-be.[ch] so a
This one covers the middle and back end files which included gimple.h
Bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK?
Andrew
* asan.c: Include gimplify.h rather than gimple.h.
* cfgloopmanip.c: Likewise.
* cgraphunit.c: Likewise.
* cilk-common.c: Likewise.
* dwarf2out.
On 11/05/2013 11:05 PM, Kirill Yukhin wrote:
> Hello,
> Small correction.
>
> On 01 Nov 16:19, Kirill Yukhin wrote:
>> +(define_insn "avx512f_store_mask"
>> + [(set (match_operand:VI48F_512 0 "memory_operand" "=m")
>> +(vec_merge:VI48F_512
>> + (match_operand:VI48F_512 1 "register_operan
This one covers the front end files which included gimple.h
Bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK?
Andrew
* ada/gcc-interface/trans.c: Include gimplify.h.
* c/c-typeck.c: Include gimplify.h.
* c-family/c-common.c: Include gimplify.h.
* c-family/c-gimplify.c:
This one covers the config target files which included gimple.h
Bootstraps on x86_64-unknown-linux-gnu with no new regressions, and
compiled stage1 on each of the targets utilizing these config/*.c files
to ensure they compiled.
targets I used were:
aarch64-linux-gnu
alpha-linux-gnu
ia64-lin
On Fri, Nov 8, 2013 at 4:01 AM, Rainer Orth
wrote:
> The recent Go patch (couldn't find the submission on gcc-patches) broke
> Solaris bootstrap: on Solaris 10/x86 I get
>
> /vol/gcc/src/hg/trunk/local/libgo/go/net/fd_unix.go:414:72: error: reference
> to undefined identifier 'syscall.F_DUPFD_CL
On Mon, Nov 11, 2013 at 7:08 AM, Uros Bizjak wrote:
>
> Building latest gcc+go on CentOS 5.10 breaks with following build failure:
>
> ../../../gcc-svn/trunk/libgo/go/net/fd_unix.go:414:72: error:
> reference to undefined identifier ‘syscall.F_DUPFD_CLOEXEC’
>r0, _, e1 := syscall.Syscall(sysca
On Fri, Nov 8, 2013 at 4:01 AM, Rainer Orth
wrote:
> The recent Go patch (couldn't find the submission on gcc-patches) broke
> Solaris bootstrap: on Solaris 10/x86 I get
>
> /vol/gcc/src/hg/trunk/local/libgo/go/net/tcpsockopt_unix.go:26:103: error:
> reference to undefined identifier 'syscall.TC
This patch fixes PR 59054, which is a bug I introduced in early October, when I
added initial patches for allowing DFmode values in the traditional Altivec
registers on ISA 2.06, and SFmode values on ISA 2.07. This patch eliminates
the constraints that allowed DImode values to go into the Altivec
On Mon, Nov 11, 2013 at 04:45:36PM -0500, Michael Meissner wrote:
> [gcc]
> 2013-11-11 Michael Meissner
>
> PR target/59054
> * config/rs6000/rs6000.md (movdi_internal32): Eliminate
> constraints that would allow DImode into the traditional Altivec
> registers, but cause
On Mon, 11 Nov 2013, Gerald Pfeifer wrote:
> Working to address a user question, I noticed that many of our pages use
> the spelling of "backend" when http://gcc.gnu.org/codingconventions.html
> suggest "back end" (noun) and "back-end" (adjective).
>
> Joseph, if you confirm that back end it is
On Mon, 11 Nov 2013, Ulrich Weigand wrote:
> Hello,
>
> this is the first patch in the series to add support for the ELFv2 ABI.
>
> The ELFv2 ABI is the intended ABI for the new powerpc64le-linux port.
> However, it is not inherently tied to the byte order; it it possible
> in principle to use t
On Mon, Nov 11, 2013 at 03:40:48PM +0100, Ulrich Weigand wrote:
> @@ -355,7 +364,11 @@ extern int dot_symbols;
> #define LINK_OS_DEFAULT_SPEC "%(link_os_linux)"
>
> #define GLIBC_DYNAMIC_LINKER32 "/lib/ld.so.1"
> -#define GLIBC_DYNAMIC_LINKER64 "/lib64/ld64.so.1"
> +#ifdef LINUX64_DEFAULT_ABI_E
Ping?
The original post is here:
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01588.html
Thanks,
Christophe.
On 28 June 2013 13:41, Christophe Lyon wrote:
> Hi,
> Following the discussion on
> http://gcc.gnu.org/ml/gcc/2013-05/msg00208.html
> here is a patch to change the frame direction.
>
>
Hi,
this issue points out that we aren't handling correctly alias template
declarations of enums. The core problem in lookup_template_class_1 is
easy to spot: we check for (and handle) TREE_CODE (template_type) ==
ENUMERAL_TYPE *before* DECL_ALIAS_TEMPLATE_P (gen_tmpl). Besides this
more or l
Hi Balaji,
any chance to get this fixed? Bootstrap on FreeBSD has now been
broken for some two weeks overall and I hope there aren't other
things hidden behind this one.
Gerald
On Tue, 5 Nov 2013, Gerald Pfeifer wrote:
> thanks for looking into this. Unfortunately, with SVN revision 204424
> (
On Mon, 11 Nov 2013, Richard Biener wrote:
On Sun, Nov 10, 2013 at 4:27 PM, Marc Glisse wrote:
Hello,
I am posting this patch to get some feedback on the approach. The goal is to
replace malloc+free with a stack allocation (a decl actually) when the size
is a small constant.
For testing, I h
On Mon, 11 Nov 2013, Ondřej Bílka wrote:
On Sun, Nov 10, 2013 at 04:27:00PM +0100, Marc Glisse wrote:
Hello,
I am posting this patch to get some feedback on the approach. The
goal is to replace malloc+free with a stack allocation (a decl
actually) when the size is a small constant.
Why const
On Thu, Oct 10, 2013 at 08:40:05PM +0200, Jan Hubicka wrote:
> --- config/i386/x86-tune.def (revision 203387)
> +++ config/i386/x86-tune.def (working copy)
> +/* X86_TUNE_AVX256_UNALIGNED_LOAD_OPTIMAL: if true, unaligned loads are
> + split. */
> +DEF_TUNE (X86_TUNE_AVX256_UNALIGNED_LOAD_OPTI
On Mon, Nov 11, 2013 at 4:18 PM, Jakub Jelinek wrote:
> On Thu, Oct 10, 2013 at 08:40:05PM +0200, Jan Hubicka wrote:
>> --- config/i386/x86-tune.def (revision 203387)
>> +++ config/i386/x86-tune.def (working copy)
>
>> +/* X86_TUNE_AVX256_UNALIGNED_LOAD_OPTIMAL: if true, unaligned loads are
>> +
OK.
Jason
On 11/11/2013 02:44 PM, Adam Butcher wrote:
Yes, using add_outermost_template_args.
I have done this (diff -w -b follows). The route to this seems even
more costly though; current_template_parms needs to first be converted
to a vec by current_template_args() then the last nesting level is
thro
On 11/09/13 03:29, Tom de Vries wrote:
Michael,
This patch removes duplicate contents in gcc/config/rs6000/t-xilinx.
Looks like a patch was applied twice.
Committed as trivial.
Thanks,
- Tom
2013-11-08 Tom de Vries
* config/rs6000/t-xilinx: Remove duplicate contents.
Thanks. That'
> -Original Message-
> From: Gerald Pfeifer [mailto:ger...@pfeifer.com]
> Sent: Monday, November 11, 2013 6:52 PM
> To: Iyer, Balaji V
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Fix for PR bootstrap/58951
>
> Hi Balaji,
>
> any chance to get this fixed? Bootstrap on FreeBSD h
On 11/11/13 21:30, Iyer, Balaji V wrote:
-Original Message-
From: Gerald Pfeifer [mailto:ger...@pfeifer.com]
Sent: Monday, November 11, 2013 6:52 PM
To: Iyer, Balaji V
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] Fix for PR bootstrap/58951
Hi Balaji,
any chance to get this fixed?
> -Original Message-
> From: David Edelsohn [mailto:dje@gmail.com]
> Sent: Monday, November 11, 2013 11:36 AM
> To: Iyer, Balaji V
> Cc: GCC Patches; Joseph S. Myers
> Subject: Re: PING Cilk Plus failures on non-LTO targets
>
> On Mon, Nov 4, 2013 at 11:06 AM, David Edelsohn
> wrote
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Jeff Law
> Sent: Monday, November 11, 2013 11:40 PM
> To: Iyer, Balaji V; Gerald Pfeifer
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH] Fix for PR bootstrap/58951
>
> O
Hello,
this patch passed bootstrap+testsuite. I don't have a testcase, the code
just looked strange.
2013-11-05 Marc Glisse
PR tree-optimization/59077
* ipa-pure-const.c (better_state): Update *state.
--
Marc GlisseIndex: gcc/ipa-pure-const.c
=
For RTOS who need to relocate executable, PC relative and GOTOFF cannot be
used as the offset between any sections won't be fixed. Only GOT can be
used, just as VxWorks RTP does.
This patch introduces a new option enable user to choose between fixed
offset or not. Enabled for VxWorks RTP to keep i
Ping in this one.
Hi Bernd, could you please help us on this one?
Please ignore the previous ping message because I used the wrong email
account.
Sorry for the inconvenience.
Thanks,
bin
> -Original Message-
> From: Bin.Cheng [mailto:amker.ch...@gmail.com]
> Sent: Monday, November 04, 20
Hello!
> Currently on trunk the option -mpreferred-stack-boundary does not work
> together with #pragma GCC target("sse") or __attribute__((target("sse"))).
>
> There is already a test case that detects this:
> gcc.target/i386/fastcall-sseregparm.c
>
> The attached patch fixes this test case unde
There was something wrong with Bernd's address, retrying.
>> Currently on trunk the option -mpreferred-stack-boundary does not work
>> together with #pragma GCC target("sse") or __attribute__((target("sse"))).
>>
>> There is already a test case that detects this:
>> gcc.target/i386/fastcall-ssere
On Mon, Nov 11, 2013 at 8:52 PM, Ian Lance Taylor wrote:
> On Fri, Nov 8, 2013 at 1:10 AM, Uros Bizjak wrote:
>>
>> panic: runtime error: invalid memory address or nil pointer dereference
>> [signal 0xb code=0x1 addr=0x1c]
>> FAIL: runtime/pprof
>> gmake[2]: *** [runtime/pprof/check] Error 1
>>
101 - 163 of 163 matches
Mail list logo
