Have you compared the latency of the multiply instructions to the
overhead of saving those registers in the interrupt handler? What
about the case where performance is priority, and the developer knows
that the interrupt handlers don't use the multiply registers?
Also, your code doesn't properly
Hi,
Please find attached a patch which disables interrupts during inline hardware
multiplication routines.
These routines use up several control registers which are not saved/restored in
interrupt routines.
This causes corruption of result in case multiplication/division registers are
used in ma
On Tue, 2 Jun 2015, Jan Hubicka wrote:
> this patch adds a short info about Cauldron to the news page. OK?
Sure thing!
(Not sure whether you can use A HREF, or just the lower-case version,
to be on the safe side I'd opt for the latter. Have you considered
mentioning Czech Republic, or will every
Symbols are normally PSImode, and the MSP430 has PSImode registers and
PSImode moves to reg/mem. But sometimes gcc uses an SImode move
instead, if the result will later be used in SImode (as in getP() in
gcc/testsuite/gcc.dg/torture/pr65077.c). Committed.
* config/msp430/msp430.md (movs
On 06/04/2015 05:34 PM, Jason Merrill wrote:
On 06/03/2015 10:42 AM, Aldy Hernandez wrote:
- if (decl && (DECL_ABSTRACT_P (decl) || declaration || old_die ==
NULL))
+ if (decl && (DECL_ABSTRACT_P (decl)
+ || !old_die || is_declaration_die (old_die)))
Do we still need DECL_ABSTRACT_
On 06/04/2015 02:39 PM, Jason Merrill wrote:
On 06/04/2015 02:23 PM, Jason Merrill wrote:
On 06/03/2015 10:42 AM, Aldy Hernandez wrote:
+ /* Die was generated early via dwarf2out_early_global_decl. */
+ BOOL_BITFIELD dumped_early : 1;
Missed a "dump".
Actually, why do we need this flag?
I'm away on PTO for the next couple weeks, which likely means that patch
review times will suffer. Normally I'd ask Richard Henderson to help
cover, but he's going to be on PTO as well.
It's probably safe to assume that when I return there will be a bit of a
patch backlog and I'll have a hi
On 06/03/2015 10:42 AM, Aldy Hernandez wrote:
- if (decl && (DECL_ABSTRACT_P (decl) || declaration || old_die == NULL))
+ if (decl && (DECL_ABSTRACT_P (decl)
+ || !old_die || is_declaration_die (old_die)))
Do we still need DECL_ABSTRACT_P? I wouldn't expect to get here when
emi
On Thu, Jun 4, 2015 at 10:05 AM, Richard Henderson wrote:
> On 06/04/2015 09:54 AM, Sriraman Tallam wrote:
>> + DECL_ATTRIBUTES (SYMBOL_REF_DECL (XEXP(fnaddr, 0)
>
> Spacing.
>
>> {
>> use_reg (&use, gen_rtx_REG (Pmode, REAL_PIC_OFFSET_TABLE_REGNUM));
>> if (
On Thu, 4 Jun 2015, Marek Polacek wrote:
> This patch fixes two issues in locations with offsets that these two
> new tests revealed:
> 1) we should check the return value of location_get_source_line;
> 2) linemap_position_for_loc_and_offset can be confused if a location
>is in a different map
On Wed, 3 Jun 2015, Marek Polacek wrote:
> > Well, ok then. This new version incorporates Richard S.'s suggestion,
> > and warns even for 1 << 31 in C99/C11 (also in C90 when -Wshift-overflow
> > is explicitely specified). For C++, it warns about 1 << 31 by default
> > only in C++11 mode, in C++
> There was a significant amount of bitrot. Apparently right after I
> submitted the patch, many macros were adjusted by Adacore hitting the
> same files and copyrights as I did. I updated the patch set by hand and
> tested it against the 31 may snapshot. I also fixed the
> TOOLS_TARGET_PAIRS is
Hello world,
the attached patch fixes the PR by adding a charlen to the
function expression of adjustl and adjustr. Committed
as obvious after regression-testing.
Regards
Thomas
2015-06-04 Thomas Koenig
PR fortran/58749
* iresolve.c (gfc_resolve_adjustl): If string
On 06/04/2015 02:23 PM, Jason Merrill wrote:
On 06/03/2015 10:42 AM, Aldy Hernandez wrote:
+ /* Die was generated early via dwarf2out_early_global_decl. */
+ BOOL_BITFIELD dumped_early : 1;
Missed a "dump".
Actually, why do we need this flag? The uses I see are
* to avoid declaring proto
On 06/03/2015 10:42 AM, Aldy Hernandez wrote:
+ /* Die was generated early via dwarf2out_early_global_decl. */
+ BOOL_BITFIELD dumped_early : 1;
Missed a "dump".
Actually, why do we need this flag? The uses I see are
* to avoid declaring prototype parameters multiple times
Can't we just
On 6/3/2015 11:03, Arnaud Charlet wrote:
>> Is a new submission necessary or can this be handled by committer?
> Preferably yes.
>> A new submission will take me a few days because I'm traveling ATM.
There was a significant amount of bitrot. Apparently right after I
submitted the patch, many ma
OK.
Jason
On 06/04/2015 09:54 AM, Sriraman Tallam wrote:
> + DECL_ATTRIBUTES (SYMBOL_REF_DECL (XEXP(fnaddr, 0)
Spacing.
> {
> use_reg (&use, gen_rtx_REG (Pmode, REAL_PIC_OFFSET_TABLE_REGNUM));
> if (ix86_use_pseudo_pic_reg ())
> @@ -25598,7 +25603,31 @@ ix86_expand_ca
ping? or do you want to leave this dormant until 325 is resolved (which doesn't
seem to be any time soon ...)
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02892.html
On 05/27/15 16:05, Nathan Sidwell wrote:
this patch fixes 52595, which I'd not noticed was actually suspended. anyway,
when p
> Patch attached with those changes.
Is this patch alright to commit?
* c-family/c-common.c (noplt): New attribute.
(handle_noplt_attribute): New handler.
* calls.c (prepare_call_address): Check for noplt attribute.
* config/i386/i386.c (ix86_function_ok_for_sibcall): Check
for noplt attribute.
On 06/04/2015 07:19 AM, James Greenhalgh wrote:
> 2015-06-04 James Greenhalgh
>
> * builtins.c (expand_builtin_atomic_compare_exchange): Call
> emit_cmp_and_jump_insns with the mode of target.
Ok.
r~
On 22/04/15 09:36, Richard Biener wrote:
On Tue, 21 Apr 2015, Thomas Schwinge wrote:
Hi!
On Tue, 25 Nov 2014 12:22:02 +0100, Tom de Vries wrote:
On 24-11-14 11:56, Tom de Vries wrote:
On 15-11-14 18:19, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
I'm submitting a patch seri
Ping.
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02405.html
Thanks,
Kyrill
On 27/05/15 11:25, Kyrill Tkachov wrote:
Ping.
Here is the rebased (and retested) patch after Christian's series.
Thanks,
Kyrill
On 18/05/15 11:26, Kyrill Tkachov wrote:
Hi all,
When using the short Thumb2 IT bloc
On 03/06/15 13:20, Richard Biener wrote:
On Wed, 3 Jun 2015, Tom de Vries wrote:
On 22/04/15 09:39, Richard Biener wrote:
Ehm. So why not simply add a flag to struct loop instead and set it
during OMP region parsing/lowering?
Attached patch adds an in_oacc_kernels_region flag to struct loop
On Thu, 4 Jun 2015, Ilya Enkovich wrote:
> 2015-06-03 18:59 GMT+03:00 Joseph Myers :
> > You could, for example, have
> >
> > #if SOMETHING
> > #define MSG "%nsome msg"
> > #else
> > #define MSG ""
> > #endif
> >
> > and have another spec using MSG - that should work.
>
> In this case I should de
Hi Jonathan,
I have committed it on trunk.
Is it Okay for me to backport it to branch 5.0, 4.9 to remove failing noise?
Regards,
Renlin Li
On 01/06/15 16:58, Jonathan Wakely wrote:
On 06/05/15 09:09 +0100, Renlin Li wrote:
Hi all,
This is a simple patch to add _GLIBCXX_HAVE_LIMIT_FSIZE to g
Hi,
currently GCC generates an assembly file when run on C/C++ header files with
the -fdump-ada-spec switch. This is unexpected since -fdump-ada-spec is used
only to output an Ada binding.
This patch fixes the specs for C/C++ header files in order to pass "-o %g.s"
to cc1/cc1plus in this cas
Jeff Law writes:
> I'm pleased to announce that Richard Sandiford has been appointed as a
> maintainer for all the programs which processes the machine description
> to generate code (ie the gen* programs).
>
> Richard, can you please add your new role in the MAINTAINERS file?
Thanks Jeff! Her
Hi Uros,
As discussed here https://gcc.gnu.org/ml/gcc/2015-06/msg00043.html
I am going to install the following patch to trunk.
GCC bootstrap and regressions tests passed.
Regards,
Venkat.
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index ab5c004..2fa6e96 100644
--- a/gcc/ChangeLog
+++ b/gcc/C
On 22/05/15 11:27, Richard Biener wrote:
On Thu, May 21, 2015 at 5:36 PM, Thomas Schwinge
wrote:
Hi!
It's just been a year. ;-P
In early March, I (hopefully correctly) adapted Tom's patch to apply to
then-current GCC trunk sources; posting this here. Is the general
approach OK?
On Tue, 20
On 04/06/15 15:20, Tom de Vries wrote:
Hi,
[ posted earlier as part of "Don't dump low gimple functions in gimple
dump", https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01586.html,
currently discussed at:
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02076.html ]
This patch adds a debug msg to du
On 06/04/2015 08:27 AM, Jan Kratochvil wrote:
On Thu, 04 Jun 2015 16:00:18 +0200, Jeff Law wrote:
But my assertion is that stuff like what you've shown above simply isn't
important to handle. What we need to look at are the common cases and I
haven't seen a strong argument that the common case
On Thu, 04 Jun 2015 16:00:18 +0200, Jeff Law wrote:
> But my assertion is that stuff like what you've shown above simply isn't
> important to handle. What we need to look at are the common cases and I
> haven't seen a strong argument that the common cases can't be handled by
> gdb.
If we target
Hi,
This patch adds a set of tests for worker-single predication (added
by Bernd in https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00094.html)
and worker-partitioned mode for OpenACC.
Results generally look good, though support for synchronisation after
worker loops is currently missing, so the co
This patch fixes two issues in locations with offsets that these two
new tests revealed:
1) we should check the return value of location_get_source_line;
2) linemap_position_for_loc_and_offset can be confused if a location
is in a different map than it expects it to be. Proper fix is unknown
Hi,
I was playing with some changes to costs for some immediate values in
AArch64, and ended up tripping an ICE in some builtin expansion code.
The ICE looked like this (some pruning of the boring bits...)
format.c: In function '_gfortran_caf_atomic_cas':
format.c:13:3: internal compiler error:
2015-06-03 18:59 GMT+03:00 Joseph Myers :
> You could, for example, have
>
> #if SOMETHING
> #define MSG "%nsome msg"
> #else
> #define MSG ""
> #endif
>
> and have another spec using MSG - that should work.
In this case I should define SOMETHING in configure of gcc, not in
configure of libmpx, ri
On 06/04/2015 01:36 AM, Jan Kratochvil wrote:
On Thu, 04 Jun 2015 09:24:36 +0200, Manuel López-Ibáñez wrote:
On 3 June 2015 at 22:58, Jan Kratochvil wrote:
In general parsing LHS vs. RHS is not so trivial:
*array1@10
expression wrapped into ->
(*array2+"a@c"[1]+'@'+
Hi,
Il 04 giugno 2015 11:15:33 CEST, Andreas Schwab ha
scritto:
>/opt/gcc/gcc-20150604/gcc/testsuite/obj-c++.dg/tls/diag-5.mm:4:1:
>error: '__thr\
>ead' can only be specified for objects and functions^M
Thanks Andreas. Looks like I missed a trivial testcase adjus
On 29/04/15 11:58, Marcus Shawcroft wrote:
On 29 April 2015 at 01:24, Andrew Pinski wrote:
On Tue, Dec 16, 2014 at 2:19 AM, Renlin Li wrote:
Hi all,
This is a simple patch to add another two ACLE 2.0 predefined macros into
aarch64 backend.
They are __ARM_ALIGN_MAX_PWR and __ARM_ALIGN_MAX_S
On Thu, 04 Jun 2015 10:55:59 +0200, Jakub Jelinek wrote:
> (gdb) p *(int (*)[4])&a[0]
> $1 = {1, 2, 3, 4}
> (gdb) p *(char (*)[4])&b[1]
> $2 = "bcde"
>
> Though, admittedly that is more typing than a[0]@4 or b[1]@4 .
I forgot during this discussion about the C style cast, you are right.
For some
Hi,
[ posted earlier as part of "Don't dump low gimple functions in gimple
dump", https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01586.html,
currently discussed at:
https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02076.html ]
This patch adds a debug msg to dump_file in cgraph_node::add_new_func
Ping ~
Can somebody review it? Thank you!
Regards,
Renlin Li
On 16/04/15 10:32, Renlin Li wrote:
Ping~
Regards,
Renlin Li
On 16/04/15 10:09, wrote:
Ping~
Anybody has time to review it?
Regards,
Renlin Li
On 06/02/15 17:48, Renlin Li wrote:
Hi all,
This is a backport patch for branch 4.
Hi,
we would like to add a new query for -fdump-ada-spec corresponding to the
any_dependent_template_arguments_p predicate of the C++ front-end. This makes
it possible for -fdump-ada-spec to accept a few more patterns.
Tested on x86_64-suse-linux, OK for the mainline?
2015-06-04 Pierre-Mari
Committed with said change r223982.
Is patch ok for fsf-5 backport?
Alex
OK if no regressions.
Ramana
On 01/06/15 10:50, Ramana Radhakrishnan wrote:
On 01/06/15 10:48, Alex Velenko wrote:
Hi,
This patch fix thumb-ltu.c to pass excess error test.
Without default -std=gnu90 flag, this testcase started failing
as some functions were called before being predefined.
Is patch ok?
gcc/testsuite
2
On 03/06/15 21:56 +0200, François Dumont wrote:
Hi
Here is a patch to add heterogeneous lookup to alternative modes.
Excellent.
To do so I had to expose __is_transparent as __has_is_transparent to
avoid confilct with existing __is_transparent.
That's a better name, thanks.
Should I p
On 01/06/15 20:51 -0700, Tim Shen wrote:
On Tue, Nov 25, 2014 at 12:46 AM, Tim Shen wrote:
On Wed, Nov 12, 2014 at 11:45 PM, Tim Shen wrote:
Committed with comment fix and slight change on testcase
(VERIFY(false) at end of the try block -- must throw).
Is it possible to backport this pa
On 06/03/2015 09:36 AM, Andrew MacLeod wrote:
On 06/03/2015 07:47 AM, Richard Biener wrote:
On Tue, Jun 2, 2015 at 4:19 PM, Andrew MacLeod
wrote:
On 06/02/2015 09:30 AM, Richard Biener wrote:
On Tue, Jun 2, 2015 at 2:34 PM, Andrew MacLeod
wrote:
On 06/02/2015 04:26 AM, Richard Biener wrote:
Hi all,
This is a simple patch to change the align variable (used in
emit_local()) from type "int" to type "unsigned int".
It should be defined as "unsigned int" which is the same data type
returned by symtab_node::get (decl)->definition_alignment ()
For the maximum alignment allowed by GCC(w
This fixes an undefined symbol at link time in specific cases for functions
that return a discriminated record type with default discriminant. It's a
regression present on the mainline and 5 branch.
Tested on x86_64-suse-linux, applied on the mainline and 5 branch.
2015-06-04 Pierre-Marie de
Hi Steve,
The patch did last from gcc-4.2, apparently without anybody detecting
a problem :-)
Anyway, the patch looks good to me too. Thanks for getting to it so fast, FX.
Cheers
Paul
On 4 June 2015 at 10:24, FX wrote:
>
>> 2015-06-03 Russell Whitesides
>>Steven G. Kargl
>>
>>
2015-06-04 Uros Bizjak
PR target/66369
* config/i386/sse.md (_pmovmsk): Merge from avx2_pmovmskb
and sse2_pmovmskb using VI1_AVX2 mode iterator.
(*_movmsk_zext): New insn pattern.
(*_pmovmskb_zext): Ditto.
Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.
Com
On Thu, Jun 04, 2015 at 09:27:25AM +0100, Richard Earnshaw wrote:
> On 03/06/15 20:44, James Greenhalgh wrote:
> > On Wed, Jun 03, 2015 at 08:28:12PM +0100, Sandra Loosemore wrote:
> >> On 06/03/2015 12:05 PM, James Greenhalgh wrote:
> >>> Basically the problem occurs when a target which is not OK
Hi Shiva,
On 04/06/15 10:57, Shiva Chen wrote:
Hi, Kyrill
Thanks for the tips of syntax.
It seems that correct syntax for
ldrb with condition code is ldreqb
ldab with condition code is ldabeq
So I modified the pattern as follow
{
enum memmodel model = (enum memmodel) INTVA
Hi, Kyrill
Thanks for the tips of syntax.
It seems that correct syntax for
ldrb with condition code is ldreqb
ldab with condition code is ldabeq
So I modified the pattern as follow
{
enum memmodel model = (enum memmodel) INTVAL (operands[2]);
if (model == MEMMODEL_RELAXED
Hello world,
front-end optimization and FORALL do not appear to mix well.
This patch fixes an ICE resulting from an attempt by front-end
optimization to use a BLOCK inside a FORALL statement.
I will commit this as obvious in a day or so unless somebody objects.
I will also backport (time to inst
On 29/05/15 20:40, Jason Merrill wrote:
On 05/29/2015 09:18 AM, Ramana Radhakrishnan wrote:
+static tree
+build_atomic_load_byte (tree src, HOST_WIDE_INT model)
This function needs a comment. The C++ changes are OK with that.
Jason
I'm assuming your review and rth's review constitute an
ARMv8.1 is a set of optional architectural extensions to ARMv8. Support, added
by other patches, is enabled in binutils for ARMv8.1 and for the individual
extensions by using architechure name "armv8.1-a" or by adding the extension
name to "armv8-a".
This patch adds support to gcc for using "armv
/opt/gcc/gcc-20150604/gcc/testsuite/obj-c++.dg/tls/diag-5.mm:4:1: error: '__thr\
ead' can only be specified for objects and functions^M
FAIL: obj-c++.dg/tls/diag-5.mm -fgnu-runtime (test for errors, line 4)
FAIL: obj-c++.dg/tls/diag-5.mm -fgnu-runtime (test for excess errors)
Excess er
On Thu, Jun 04, 2015 at 10:36:46AM +0200, Manuel López-Ibáñez wrote:
> On 4 June 2015 at 09:36, Jan Kratochvil wrote:
> > On Thu, 04 Jun 2015 09:24:36 +0200, Manuel López-Ibáñez wrote:
> >> On 3 June 2015 at 22:58, Jan Kratochvil wrote:
> > These two expressions are equivalent for all operations
On Thu, 04 Jun 2015 10:36:46 +0200, Manuel López-Ibáñez wrote:
> except for printing a memory
> region, and for that purpose one only needs to parse LHS@RHS and only
> one @ makes sense within the same print command.
Yes, just LHS or RHS can be pretty complicated containing the '@' character at
le
On 04/06/15 09:17, Kyrill Tkachov wrote:
> Hi Shiva,
>
> On 04/06/15 04:13, Shiva Chen wrote:
>> Hi, Ramana
>>
>> Currently, I work for Marvell and the company have copyright assignment on
>> file.
>>
>> Hi, all
>>
>> After adding the attribute and rebuild gcc, I got the assembler error message
>
On 4 June 2015 at 09:36, Jan Kratochvil wrote:
> On Thu, 04 Jun 2015 09:24:36 +0200, Manuel López-Ibáñez wrote:
>> On 3 June 2015 at 22:58, Jan Kratochvil wrote:
> These two expressions are equivalent for all operations except of sizeof():
> pointer
> (*pointer@ANYTHING)
>
Sure,
On 26/05/15 12:39, Richard Biener wrote:
On Thu, 14 May 2015, Tom de Vries wrote:
On 20-04-15 14:25, Richard Biener wrote:
On Wed, 15 Apr 2015, Tom de Vries wrote:
On 03-04-15 14:39, Tom de Vries wrote:
On 27-03-15 15:10, Tom de Vries wrote:
Hi,
this patch fixes PR65443, a todo in the par
On 03/06/15 20:44, James Greenhalgh wrote:
> On Wed, Jun 03, 2015 at 08:28:12PM +0100, Sandra Loosemore wrote:
>> On 06/03/2015 12:05 PM, James Greenhalgh wrote:
>>>
>>> This has caused some issues for my multilib testing. Summarised below,
>>> with some help from Alan Lawrence.
>>>
>>> Basically t
On 03/06/15 17:03, Kyrill Tkachov wrote:
> Hi all,
>
> This patch is part of a few prerequisites to get target attribute support in
> AArch64.
> This registers the fma steering pass unconditionally but gates its execution
> instead.
> This way the pass will be available if during the compilation
> 2015-06-03 Russell Whitesides
>Steven G. Kargl
>
>PR fortran/40958
>PR fortran/60780
>PR fortran/66377
>* module.c (load_equiv): Add check for loading duplicate EQUIVALENCEs
>from different modules. Eliminate the pruning of unused
>equivalence-objects
>
>
Hi Shiva,
On 04/06/15 04:13, Shiva Chen wrote:
Hi, Ramana
Currently, I work for Marvell and the company have copyright assignment on file.
Hi, all
After adding the attribute and rebuild gcc, I got the assembler error message
load_n.s:39: Error: bad instruction `ldrbeq r0,[r0]'
When i look i
On Thu, 04 Jun 2015 09:24:36 +0200, Manuel López-Ibáñez wrote:
> On 3 June 2015 at 22:58, Jan Kratochvil wrote:
>
> > In general parsing LHS vs. RHS is not so trivial:
> > *array1@10
> > expression wrapped into ->
> > (*array2+"a@c"[1]+'@'+'\''@(*array1@10)[5])[2]
> >
>
>
On 3 June 2015 at 22:58, Jan Kratochvil wrote:
> In general parsing LHS vs. RHS is not so trivial:
> *array1@10
> expression wrapped into ->
> (*array2+"a@c"[1]+'@'+'\''@(*array1@10)[5])[2]
>
Is this a real case? I cannot understand what this means, but it could
simply be
71 matches
Mail list logo
