On 11/27/2013 05:46 AM, Marek Polacek wrote:> On Wed, Nov 27, 2013 at
05:29:22AM -0500, mtewoodb...@gmail.com wrote:
>> From: Max TenEyck Woodbury
>
> This patch is badly missing a description. You also want to mention
> the PR number, if this fixes a bug. I guess this is to fix PR58687.
>
I
> On 11/27/13 10:30, Yvan Roux wrote:
> >>Please include either the patch you are pinging or at the least a link to it
> >>in the archives.
> >
> >Ok, sorry for that, here is the patch and Changelog
> >
> >Yvan
> >
> >
> >2013-11-17 Yvan Roux
> >
> > * config/arm/arm.md (store_minmaxsi):
On 11/27/13 10:58, Vladimir Makarov wrote:
The secondary reload macros/hooks define cases where additional
registers are needed to reload certain forms of rtl. I doubt the use
of LRA completely eliminates the need for secondary reloads.
When I designed LRA I wanted to remove as many hooks a
On 27 November 2013 19:13, Jeff Law wrote:
> On 11/27/13 10:30, Yvan Roux wrote:
>>>
>>> Please include either the patch you are pinging or at the least a link to
>>> it
>>> in the archives.
>>
>>
>> Ok, sorry for that, here is the patch and Changelog
>>
>> Yvan
>>
>>
>> 2013-11-17 Yvan Roux
>>
> -Original Message-
> From: Jason Merrill [mailto:ja...@redhat.com]
> Sent: Wednesday, November 27, 2013 12:43 PM
> To: Iyer, Balaji V; gcc-patches@gcc.gnu.org
> Cc: Jeff Law
> Subject: Re: _Cilk_spawn and _Cilk_sync for C++
>
> On 11/25/2013 10:50 AM, Iyer, Balaji V wrote:
> > I have f
On 11/27/13 10:49, Yvan Roux wrote:
How can that be correct?
The secondary reload macros/hooks define cases where additional registers
are needed to reload certain forms of rtl. I doubt the use of LRA
completely eliminates the need for secondary reloads.
Vladimir explained me that in that cas
On 11/27/13 03:19, Yvan Roux wrote:
Ping.
On 20 November 2013 10:22, Yvan Roux wrote:
Hi,
as Richard said, only a subset of rclass is allowed to be returned by
preferred_reload_class.
I don't think he was quite that definitive. "One reading of the manual
suggests ...". However, Richard's i
On 11/27/2013, 1:20 PM, Jeff Law wrote:
On 11/27/13 10:58, Vladimir Makarov wrote:
The secondary reload macros/hooks define cases where additional
registers are needed to reload certain forms of rtl. I doubt the use
of LRA completely eliminates the need for secondary reloads.
When I design
Something Richi noticed. If the call to thread_block is unsuccessful
when threading a backedge to an interior loop node, then we do not have
to clobber the loop structure. We also need to cleanup in that case.
Bootstrapped and regression tested on x86_64-unknown-linux-gnu.
Installed on the
On Mon, Nov 25, 2013 at 10:08 PM, Sriraman Tallam wrote:
> I have attached a patch to fix this bug :
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58944
>
> A similar problem was also reported here:
> http://gcc.gnu.org/ml/gcc-patches/2013-11/msg01050.html
>
>
> Recently, ix86_valid_targe
On 11/27/2013 01:25 PM, Iyer, Balaji V wrote:
I think a better approach would be to add a cp_build_cilk_spawn that uses
stabilize_call to pre-evaluate the arguments of the call.
I really can't pre-evaluate the calls before I move into the nested function
because all those parts must be in the
On Wed, 27 Nov 2013, Max Woodbury wrote:
> On 11/27/2013 05:46 AM, Marek Polacek wrote:> On Wed, Nov 27, 2013 at
> 05:29:22AM -0500, mtewoodb...@gmail.com wrote:
> >> From: Max TenEyck Woodbury
> >
> > This patch is badly missing a description. You also want to mention
> > the PR number, if this
On 11/27/13 01:02, Markus Trippelsdorf wrote:
On 2013.11.20 at 15:43 +0100, Markus Trippelsdorf wrote:
On 2013.11.20 at 14:41 +0100, Paolo Bonzini wrote:
Note that you need to regenerate all users of libtool.m4. Please post a
patch _with_ the regeneration so that whoever applies it won't screw
On 11/27/13 05:29, Bernd Edlinger wrote:
Hi,
ping...
this patch still open: http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02291.html
Note: it does, as it is, _not_ depend on the keep_aligning patch.
And it would fix some really nasty wrong code generation issues.
Is there a testcase for this p
On 11/27/13 10:06, Jason Merrill wrote:
On 11/25/2013 11:03 PM, Iyer, Balaji V wrote:
On a broad note, I think there's a lot of OpenMP code you could be
reusing here rather than writing it all again. And that way Cilk code
will benefit from improvements to OpenMP handling, and vice versa. It
p
On 11/27/13 00:36, Jakub Jelinek wrote:
AddressSanitizer use-after-return instrumentation:
http://gcc.gnu.org/ml/gcc-patches/2013-11/msg02070.html
+ HOST_WIDE_INT offset, sz;
+ sz = ASAN_RED_ZONE_SIZE;
+ sz = data.asan_vec[0] - prev_offset;
Seems to me like the first
On 11/27/13 00:36, Jakub Jelinek wrote:
Use libbacktrace for libsanitizer's symbolization (will need tweaking,
depending on next libsanitizer merge, whether the corresponding
sanitizer_common changes are upstreamed or not, and perhaps to compile
libbacktrace sources again with renamed function n
> I don't think he was quite that definitive. "One reading of the manual
> suggests ...". However, Richard's interpretation is the same as I've had
> for eons. You can return the original class, a narrower class or NO_REGS.
Yes, I also read the manual and discussed the patch with Richard
offlin
On 11/27/13 00:36, Jakub Jelinek wrote:
Elemental function support (updated version of the earlier patch):
http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03197.html
Richi OK's this earlier today. So it's good to go, right? I thought I
saw some follow-up items that Richi agreed could be done lat
On 27 November 2013 19:27, Jeff Law wrote:
> On 11/27/13 10:49, Yvan Roux wrote:
>>>
>>> How can that be correct?
>>>
>>> The secondary reload macros/hooks define cases where additional registers
>>> are needed to reload certain forms of rtl. I doubt the use of LRA
>>> completely eliminates the n
On further consideration, I'm not convinced there's a bug here at all; I
don't think it's sufficiently defined in the standard what the current
token is for the purposes of line numbering when __LINE__ gets expanded to
be able to say that one or the other value in a directive is wrong. As
far
> -Original Message-
> From: Jeff Law [mailto:l...@redhat.com]
> Sent: Wednesday, November 27, 2013 2:43 PM
> To: Jason Merrill; Iyer, Balaji V; Aldy Hernandez
> Cc: gcc-patches@gcc.gnu.org; r...@redhat.com; Jakub Jelinek
> Subject: Re: [PATCH] _Cilk_for for C and C++
>
> On 11/27/13 10:
mailer added html again...
When trying some of my updated prototype changes on trunk, the code
tripped over this segment in ipa-prop.c :
lhs = gimple_assign_lhs (stmt);
rhs = gimple_assign_rhs1 (stmt);
if (!is_gimple_reg_type (rhs)
Hi!
I'm sorry for the very late answer!
On Thu, 7 Nov 2013 22:10:15 +0400, Evgeny Gavrin wrote:
> I've looked at your changes and in most of front-end parts it looks
> reasonable to me. As you know, we're like-minded with you about how OpenACC's
> front-ends should look like. So, I think it's
Hi!
Again, I'm sorry for the late answer.
On Fri, 8 Nov 2013 13:55:24 +0400, Dmitry Bocharnikov
wrote:
> > > This is in contrast to Samsung's work, who are implementing OpenACC
> > > separately from the existing OpenMP support.
> > I'm not fully agree that this in contrast to things are taking
Hi,
Recently, build.goosList and build.goarchList got fixed in mainline to
be sensible, hard-coded, lists rather than odd autogenerated lists.
This patch updates the 4.8 branch to match.
Cheers,
mwh
diff --git a/libgo/Makefile.am b/libgo/Makefile.am
index 957f23c..199b444 100644
--- a/libgo/Make
On Mon, Nov 25, 2013 at 11:19 PM, Joseph S. Myers
wrote:
> soft-fp patches go to libc-alpha, with glibc code then being imported
> unmodified into GCC.
Now imported from glibc as:
2013-11-27 Uros Bizjak
* soft-fp/op-4.h: Update from glibc.
Tested on x86_64-pc-linux-gnu multilib build a
Hi,
The subject test case uses TRANSFER to do a bitcast from real(16) to
integer(16). For IBM long double, this works OK on big endian, but not
so well on little endian. The implementation of long double as two
64-bit quantities causes the most significant bits of the value to be in
a different
On 11/18/13 14:50, Iyer, Balaji V wrote:
Attached, please find a refreshed patches (one for C and 1 for C++). The trunk
was "diffed" after Aldy's check in of pragma simd was in. So, now this patch is
only dependent on _Cilk_spawn and _Cilk_sync (mostly for execution of tests). They ar
On 11/27/13 15:02, Bill Schmidt wrote:
Hi,
The subject test case uses TRANSFER to do a bitcast from real(16) to
integer(16). For IBM long double, this works OK on big endian, but not
so well on little endian. The implementation of long double as two
64-bit quantities causes the most significan
On Wed, Nov 27, 2013 at 2:01 PM, Michael Hudson-Doyle
wrote:
>
> Recently, build.goosList and build.goarchList got fixed in mainline to
> be sensible, hard-coded, lists rather than odd autogenerated lists.
> This patch updates the 4.8 branch to match.
Thanks. Committed to 4.8 branch.
Ian
On 11/27/13 14:30, Andrew MacLeod wrote:
mailer added html again...
When trying some of my updated prototype changes on trunk, the code
tripped over this segment in ipa-prop.c :
lhs = gimple_assign_lhs (stmt);
rhs = gimple_assign_rhs1 (st
On 11/26/13 23:36, Yury Gribov wrote:
> This patch is supposed to fix PR59063
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063).
>
> The original bug results from libasan providing clock_gettime wrapper
and then trying to call the "real" clock_gettime.
> The "real" symbol is supposed to com
On 11/27/2013 05:16 PM, Jeff Law wrote:
On 11/27/13 14:30, Andrew MacLeod wrote:
mailer added html again...
When trying some of my updated prototype changes on trunk, the code
tripped over this segment in ipa-prop.c :
lhs = gimple_assign_lhs (st
>> Hmm, maybe attack from the other direction? -- could we clear SCHED_GROUP_P
>> for each insn at the start of this loop in sched_analyze?
>>
>> It's not as clean in the sense that SCHED_GROUP_P "escapes" the scheduler,
>> but it might be an option.
>>
>>for (insn = head;; insn = NEXT_INSN (in
On 11/27/2013 05:30 PM, Andrew MacLeod wrote:
On 11/27/2013 05:16 PM, Jeff Law wrote:
On 11/27/13 14:30, Andrew MacLeod wrote:
mailer added html again...
When trying some of my updated prototype changes on trunk, the code
tripped over this segment in ip
On 11/27/13 15:30, Andrew MacLeod wrote:
On 11/27/2013 05:16 PM, Jeff Law wrote:
On 11/27/13 14:30, Andrew MacLeod wrote:
mailer added html again...
When trying some of my updated prototype changes on trunk, the code
tripped over this segment in ipa-pro
This patch brings the recent fix for calling a function or method that
takes or returns an empty struct via reflection to the 4.8 branch.
Cheers,
mwh
diff --git a/libgo/go/reflect/all_test.go b/libgo/go/reflect/all_test.go
index 526f09b..eecc459 100644
--- a/libgo/go/reflect/all_test.go
+++ b/lib
Hi,
This patch brings the recent fix for the generated hash functions of
types that are aliases for structures containing unexported fields to
the 4.8 branch.
Cheers,
mwh
diff --git a/gcc/go/gofrontend/types.cc b/gcc/go/gofrontend/types.cc
index 59247d6..36383de 100644
--- a/gcc/go/gofrontend/ty
On Wed, Nov 27, 2013 at 1:53 AM, Richard Biener wrote:
> On Fri, 22 Nov 2013, Cong Hou wrote:
>
>> Hi
>>
>> Currently in GCC vectorization, some loop invariant may be detected
>> after aliasing checks, which can be hoisted outside of the loop. The
>> current method in GCC may break the information
Hi,
remove_pseudos in lra-spills.c failed to handle REG_CFA_SET_VDRAP note.
This patch changes spill_pseudos to handle REG_XXX notes. It also adds
an assert that regno passed to DWARF_FRAME_REGNUM isn't a pseudo register.
Tested on Linux/x86-64 with -m64 and -m32. g++.dg/cpp1y/vla-initlist1.C
f
On 27-11-13 01:21, Jason Merrill wrote:
On 11/26/2013 03:47 AM, Marc Glisse wrote:
On Tue, 26 Nov 2013, Tom de Vries wrote:
* g++.dg/pr59032.C: New testcase.
* gcc.dg/pr59032.c: Same.
I didn't check very carefully, but they look similar. If they are indeed
the same, could it move to
On 27-11-13 07:20, Jeff Law wrote:
On 11/26/13 14:10, Tom de Vries wrote:
On 26-11-13 11:12, Richard Biener wrote:
On Tue, Nov 26, 2013 at 8:57 AM, Tom de Vries
wrote:
Jason,
This patch prevents creating out-of-bounds BIT_FIELD_REFs in 3
locations.
It fixes a SIGSEGV (triggered by gimple_fo
On 11/27/2013, 6:12 PM, H.J. Lu wrote:
Hi,
remove_pseudos in lra-spills.c failed to handle REG_CFA_SET_VDRAP note.
This patch changes spill_pseudos to handle REG_XXX notes. It also adds
an assert that regno passed to DWARF_FRAME_REGNUM isn't a pseudo register.
Tested on Linux/x86-64 with -m64 a
Hi!
I've committed a bunch of testcases for the reassoc issues that
were fixed by Easwaran or my r203979 changes, so that we can close those
PRs.
2013-11-28 Jakub Jelinek
PR middle-end/57393
PR tree-optimization/58018
PR tree-optimization/58131
* gcc.dg/torture
Hi!
This patch (well, the first hunk) fixes a bug where p_name was
uninitialized for -fopenmp-simd and we were strcating to it.
The second hunk removes a bogus ATTRIBUTE_UNUSED parameter and unused
p_name variable I've noticed while looking around, and the C++ FE hunk
makes sure we don't call keep
On Wed, Nov 27, 2013 at 3:38 PM, Vladimir Makarov wrote:
> On 11/27/2013, 6:12 PM, H.J. Lu wrote:
>>
>> Hi,
>>
>> remove_pseudos in lra-spills.c failed to handle REG_CFA_SET_VDRAP note.
>> This patch changes spill_pseudos to handle REG_XXX notes. It also adds
>> an assert that regno passed to DWA
On 10/16/13, 11:03 AM, Jeff Law wrote:
> On 10/16/13 09:34, Cesar Philippidis wrote:
>> On 10/15/13 12:16 PM, Jeff Law wrote:
>>> On 10/10/13 10:25, Jakub Jelinek wrote:
On Thu, Oct 10, 2013 at 07:26:43AM -0700, Cesar Philippidis wrote:
> This patch addresses an ICE when building qemu for
On Wed, Nov 27, 2013 at 10:05 AM, Alexey Samsonov wrote:
>
> Sorry, I wasn't clear enough. GCC emits DW_AT_low_pc and DW_AT_high_pc for
> *compile unit* DIE, so the consumer doesn't have to iterate over all
> the functions DIEs.
> LLVM doesn't emit DW_AT_low_pc/DW_AT_high_pc for compile unit,
> bu
On 11/27/2013 04:14 PM, Iyer, Balaji V wrote:
I completely agree with you that there are certain parts of Cilk Plus
that is similar to OMP4, namely #pragma simd and SIMD-enabled functions
(formerly called elemental functions). But, the Cilk keywords is almost
completely orthogonal to O
On 11/27/2013 05:59 PM, Iyer, Balaji V wrote:
Well, if I use copy_tree_body_r for C and C++, in lambda functions, it asserts
in varasm.c. The main reason I see that, the copy_tree_body_r walks into the
closure and then maps the variables from the lambda function from the spawner
to the helper
On 11/27/2013 06:23 PM, Tom de Vries wrote:
Also ok for 4.8 branch? It's a 4.8/4.9 regression.
Yes.
Jason
Hi,
This patch back ports revision 200103 from mainline to
ARM/Embedded-4_8-branch. It is tested on arm cortex-m3 and no regressions.
Thanks,
binIndex: gcc/tree-ssa-uncprop.c
===
--- gcc/tree-ssa-uncprop.c (revision 205471)
+++
Terry, this is a bug fix to pic register. I feel it should also be in
gcc-4_8-branch.
- Joey
On Wed, Nov 27, 2013 at 11:45 AM, Terry Guo wrote:
> Hi,
>
> This patch back ported trunk fix at r205391 to arm/embedded-4_8-branch.
>
> BR,
> Terry
>
> gcc/ChangeLog.arm
> 2013-11-27 Terry Guo
>
>
> -Original Message-
> From: Joey Ye [mailto:joey.ye...@gmail.com]
> Sent: Thursday, November 28, 2013 10:56 AM
> To: Terry Guo
> Cc: gcc-patches
> Subject: Re: [arm-embedded] Backport trunk thumb1 pic fix to embedded-
> 4_8-branch
>
> Terry, this is a bug fix to pic register. I feel it
Hi,
This patch back ported a trunk M4 CPU pipeline tuning to
embedded-4_8-branch.
BR,
Terry
gcc/ChangeLog.arm
2013-11-28 Terry Guo
Backport mainline r198021
2013-04-17 Terry Guo
* config/arm/cortex-m4.md: Add a new bypass.
The patch itself:
Index: gcc/config/arm/
On 11/27/2013 04:10 PM, Joseph S. Myers wrote:
On further consideration, I'm not convinced there's a bug here at all; I
don't think it's sufficiently defined in the standard what the current
token is for the purposes of line numbering when __LINE__ gets expanded to
be able to say that one or the
Hi,
This patch back ported a trunk cortex-m4 FPU tuning to embedded-4_8-branch.
BR,
Terry
gcc/ChangeLog.arm
2013-11-28 Terry Guo
Backport mainline r198084
2013-04-19 Terry Guo
* config/arm/cortex-m4-fpu.md (cortex_m4_v): Delete cpu unit.
Replace with ...
> -Original Message-
> From: Jason Merrill [mailto:ja...@redhat.com]
> Sent: Wednesday, November 27, 2013 8:24 PM
> To: Iyer, Balaji V; gcc-patches@gcc.gnu.org
> Cc: Jeff Law
> Subject: Re: _Cilk_spawn and _Cilk_sync for C++
>
> On 11/27/2013 05:59 PM, Iyer, Balaji V wrote:
> > Well, if
Ok
> * config/msp430/msp430.c
> (msp430_attr): Use wide-int interfaces.
>
>
> diff --git a/gcc/config/msp430/msp430.c b/gcc/config/msp430/msp430.c
> index daff4ae..e3f6712 100644
> --- a/gcc/config/msp430/msp430.c
> +++ b/gcc/config/msp430/msp430.c
> @@ -1085,7 +1085,7 @@ msp430_att
> -Original Message-
> From: Richard Earnshaw
> Sent: Tuesday, November 26, 2013 5:44 PM
> To: Terry Guo
> Cc: Ramana Radhakrishnan; gcc-patches@gcc.gnu.org
> Subject: Re: [Patch, ARM] Fix ICE when high register is used as pic base
> register for thumb1 target
>
> On 26/11/13 04:18, Terr
> Wasn't that already checked in?
No, that's an addition to previous commit.
-Y
>> This patch is supposed to fix PR59063
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063).
>
> This is fine. I'm assuming that the minor changes to the sanitizer
libraries are in bits that are maintained within GCC rather than
upstream in LLVM. Right?
Yes. This patch only touches Autotoo
>>> This patch is supposed to fix PR59063
(http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59063).
>>
>> This is fine. I'm assuming that the minor changes to the sanitizer
libraries are in bits that are maintained within GCC rather than
upstream in LLVM. Right?
>
> Yes. This patch only touches A
> -Original Message-
> From: Jeff Law [mailto:l...@redhat.com]
> Sent: Thursday, November 28, 2013 12:43 AM
> To: Zhenqiang Chen; gcc-patches@gcc.gnu.org
> Cc: Ramana Radhakrishnan; Richard Earnshaw
> Subject: Re: [PING] [PATCH, ARM, testcase] Skip target arm-neon for
> lp1243022.c
>
> O
On Wed, Nov 27, 2013 at 04:51:46PM -0800, Ian Lance Taylor wrote:
> OK to commit in stage 3?
Yes, thanks.
> 2013-11-27 Ian Lance Taylor
>
> * dwarf.c (find_address_ranges): New static function, broken out
> of build_address_map.
> (build_address_map): Call it.
> * btes
On Wed, Nov 27, 2013 at 01:26:54PM -0700, Jeff Law wrote:
> On 11/27/13 00:36, Jakub Jelinek wrote:
> >
> >Elemental function support (updated version of the earlier patch):
> >http://gcc.gnu.org/ml/gcc-patches/2013-11/msg03197.html
> Richi OK's this earlier today. So it's good to go, right? I
>
On Wed, Nov 27, 2013 at 01:11:59PM -0700, Jeff Law wrote:
> On 11/27/13 00:36, Jakub Jelinek wrote:
> >
> >Use libbacktrace for libsanitizer's symbolization (will need tweaking,
> >depending on next libsanitizer merge, whether the corresponding
> >sanitizer_common changes are upstreamed or not, and
101 - 168 of 168 matches
Mail list logo
