On Wed, Nov 01, 2017 at 10:30:29AM -0600, Jeff Law wrote:
> On 10/31/2017 08:47 PM, Trevor Saunders wrote:
> > On Tue, Oct 31, 2017 at 11:38:48AM -0600, Jeff Law wrote:
> > > On 10/31/2017 11:22 AM, Eric Botcazou wrote:
> > > > > I don't see a reason not to other than a pretty small amount of work
The attached patch fixes a regression where gfortran was
issuing an error for an entity in a common block within
a module when it appears in equivalence, and the entity
*is not* use associated in a pure subprogram. OK to
commit?
2017-11-01 Steven G. Kargl
PR fortran/82796
*
The following patch aims to removing the attribute only tf_waring flag is on.
diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index d1c846ecf44..8ff46498fd6 100644
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -7410,7 +7410,10 @@ canonicalize_type_argument (tree arg,
tsubst_flags_t complain)
if (!arg || arg ==
On Wed, 01 Nov 2017 15:07:59 PDT (-0700), jos...@codesourcery.com wrote:
> On Wed, 1 Nov 2017, Palmer Dabbelt wrote:
>
>> On Wed, 01 Nov 2017 14:59:43 PDT (-0700), jos...@codesourcery.com wrote:
>> > On Tue, 31 Oct 2017, Palmer Dabbelt wrote:
>> >
>> >> +between absolute addresses -2 GiB and +2 GiB
Hi,
On 1 November 2017 at 03:12, Jim Wilson wrote:
> On Tue, 2017-10-31 at 14:35 +1100, Kugan Vivekanandarajah wrote:
>> Ping ?
>>
>> I see that Jim has clarified the comments from Andrew.
>
> Andrew also suggested that we add a testcase to the testsuite. I
> didn't do that. I did put a testcas
I of course don't want to break anything. I didn't see any fallout
in my testing and I normally test all the front ends, including Ada,
but let me check to make sure I tested it this time (I had made some
temporary changes to my build script and may have disabled it.) Let
me double check it afte
On Wed, 1 Nov 2017, Palmer Dabbelt wrote:
> On Wed, 01 Nov 2017 14:59:43 PDT (-0700), jos...@codesourcery.com wrote:
> > On Tue, 31 Oct 2017, Palmer Dabbelt wrote:
> >
> >> +between absolute addresses -2 GiB and +2 GiB. Programs can be statically
> >> or
> >
> > That should be @minus{}2 GiB, to g
On Wed, 01 Nov 2017 14:59:43 PDT (-0700), jos...@codesourcery.com wrote:
> On Tue, 31 Oct 2017, Palmer Dabbelt wrote:
>
>> +between absolute addresses -2 GiB and +2 GiB. Programs can be statically or
>
> That should be @minus{}2 GiB, to get a minus sign instead of a hyphen.
Oops, I already committ
On Tue, 31 Oct 2017, Palmer Dabbelt wrote:
> +between absolute addresses -2 GiB and +2 GiB. Programs can be statically or
That should be @minus{}2 GiB, to get a minus sign instead of a hyphen.
--
Joseph S. Myers
jos...@codesourcery.com
Hi Carl,
On Tue, Oct 31, 2017 at 08:46:09AM -0700, Carl Love wrote:
> gcc/ChangeLog:
See Jakub's comments (thanks!)
> --- a/gcc/config/rs6000/rs6000.c
> +++ b/gcc/config/rs6000/rs6000.c
> @@ -14286,6 +14286,34 @@ swap_selector_for_mode (machine_mode mode)
>return force_reg (V16QImode, gen_rt
> Oops, I thought I've removed it, but apparently I haven't; thanks for
> catching it. Here is an updated patch I'll bootstrap/regtest soon.
>
> 2017-11-01 Jakub Jelinek
>
> PR rtl-optimization/82778
> PR rtl-optimization/82597
> * compare-elim.c (struct comparison): Add in_
On 11/1/2017 1:02 PM, Jason Merrill wrote:
On Tue, Oct 31, 2017 at 12:17 PM, Mukesh Kapoor
wrote:
On 10/25/2017 6:44 PM, Mukesh Kapoor wrote:
On 10/25/2017 4:20 AM, Nathan Sidwell wrote:
On 10/25/2017 12:03 AM, Mukesh Kapoor wrote:
Thanks for pointing this out. Checking in the front end wil
On Wed, Nov 1, 2017 at 2:29 PM, Nathan Sidwell wrote:
> This is the last of the overloaded operator reworking. Now we can cheaply
> go from identifier->overload info, there's no need to specially mark
> identifiers as new or delete operators themselves. Thus we can reorder the
> cp_identifier_ki
On Tue, Oct 31, 2017 at 12:17 PM, Mukesh Kapoor
wrote:
> On 10/25/2017 6:44 PM, Mukesh Kapoor wrote:
>>
>> On 10/25/2017 4:20 AM, Nathan Sidwell wrote:
>>>
>>> On 10/25/2017 12:03 AM, Mukesh Kapoor wrote:
>>>
Thanks for pointing this out. Checking in the front end will be
difficult becau
On Wed, Nov 01, 2017 at 01:31:57PM -0500, Segher Boessenkool wrote:
> Hi Mike,
>
> On Fri, Oct 27, 2017 at 06:39:21PM -0400, Michael Meissner wrote:
> > The power9 (running PowerPC ISA 3.0) has a round to integer instruction
> > (XSRQPI) that does various flavors of round an IEEE 128-bit floating
On Wed, Nov 1, 2017 at 3:13 PM, Alexandre Oliva wrote:
> Hi, Jason,
>
> Apologies for the delay in responding; somehow I missed your reply when
> it arrived (probably because I had a cold back then :-), and only saw it
> today.
>
> On Oct 24, 2017, Jason Merrill wrote:
>
>> On Sat, Sep 30, 2017 a
On Wed, Nov 01, 2017 at 03:24:56PM -0400, Nathan Sidwell wrote:
> Thomas Koenig noticed -fdump-lang-raw gives an immediate ICE. I broke it
> with the DECL_ASSEMBLER_NAME changes last month. Fixed thusly, applied as
> obvious.
Shouldn't there be a testcase with that option?
> 2017-11-01 Nathan
Thomas Koenig noticed -fdump-lang-raw gives an immediate ICE. I broke
it with the DECL_ASSEMBLER_NAME changes last month. Fixed thusly,
applied as obvious.
nathan
--
Nathan Sidwell
2017-11-01 Nathan Sidwell
* tree-dump.c (dequeue_and_dump): Use HAS_DECL_ASSEMBLER_NAME_P.
Index: t
On 10/31/2017 12:11 PM, David Edelsohn wrote:
With your recent removal of SDB and -gcoff support, I would appreciate
your advice about my patch to incrementally add some preliminary LTO
support for AIX to collect2.c:
OK. I can take a look. I started a new job this week, so I'm a bit
disorgan
On 10/30/2017 12:13 PM, Jeff Law wrote:
On 10/13/2017 12:04 PM, David Edelsohn wrote:
This patch adds the basic LTO scanning pass to the COFF support in
collect2. I don't believe that this change should affect other COFF
targets adversely (do they even use collect2?), but I wanted to give
peopl
Hi, Jason,
Apologies for the delay in responding; somehow I missed your reply when
it arrived (probably because I had a cold back then :-), and only saw it
today.
On Oct 24, 2017, Jason Merrill wrote:
> On Sat, Sep 30, 2017 at 5:08 AM, Alexandre Oliva wrote:
>> diff --git a/gcc/cp/constexpr.c
On Oct 31, 2017, Jeff Law wrote:
>> @@ -5691,6 +5691,15 @@ expand_gimple_basic_block (basic_block bb, bool
>> disable_tail_calls)
>> goto delink_debug_stmt;
>> else if (gimple_debug_begin_stmt_p (stmt))
>> val = gen_rtx_DEBUG_MARKER (VOIDmode);
>> + else if (gimple_debug_inline_entry_p
Hi Mike,
On Fri, Oct 27, 2017 at 06:39:21PM -0400, Michael Meissner wrote:
> The power9 (running PowerPC ISA 3.0) has a round to integer instruction
> (XSRQPI) that does various flavors of round an IEEE 128-bit floating point to
> integeral values. This patch adds the support to the machine indep
This is the last of the overloaded operator reworking. Now we can
cheaply go from identifier->overload info, there's no need to specially
mark identifiers as new or delete operators themselves. Thus we can
reorder the cp_identifier_kind enumeration and leave space in it for
udlit operators.
Dear All,
This patch is adequately described by the comment in the second chunk
applied to resolve.c.
Note, however, that the 'unconditionally' is promptly undermined by
the subsequent conditions. I will change the adjective appropriately.
In writing this, I have just realised that access=private
On Oct 31, 2017, Jeff Law wrote:
> On 09/30/2017 03:08 AM, Alexandre Oliva wrote:
>> This API change will enable final_start_function() to "consume"
>> initial insns, and choose the first insn to be passed to final().
>>
>> Many ports call final_start_function() and final() when creating
>> thun
On 30/10/2017 19:15, Jonathan Wakely wrote:
@@ -1227,7 +1227,12 @@ class Printer(object):
def add_version(self, base, name, function):
self.add(base + name, function)
if _versioned_namespace:
- self.add(base + _versioned_namespace + name, function)
+ vbase
Committed.
On Tue, 31 Oct 2017 23:37:04 PDT (-0700), Andrew Waterman wrote:
> Thanks for caring enough about our patches to even notice the grammar :)
>
> On Tue, Oct 31, 2017 at 11:09 PM, Sandra Loosemore
> wrote:
>> On 10/31/2017 10:27 PM, Andrew Waterman wrote:
>>>
>>> I have to disagree. It'
Hi Carl,
On Fri, Oct 27, 2017 at 09:49:04AM -0700, Carl Love wrote:
> > > +;; Swap all bytes in each element of vector
> > > +(define_expand "revb_"
> > > + [(set (match_operand:VEC_A 0 "vsx_register_operand")
> > > + (bswap:VEC_A (match_operand:VEC_A 1 "vsx_register_operand")))]
> > > + "TARGET
On Tue, Oct 31, 2017 at 05:07:54PM +, Wilco Dijkstra wrote:
> The AArch64 backend currently doesn't set MALLOC_ABI_ALIGNMENT, so
> add this to enable alignment optimizations on malloc pointers.
>
> OK for commit?
As far as I understand it, because we have 128-bit types, a malloc of
anything g
This fixes a bug in the implementation of the normalization algo. I
messed up a case with adjacent ../.. elements, because it's written to
work in a single pass, rather than applying each of the rules in turn,
creating loads of temporaries and reallocations.
PR libstdc++/82777
* s
On Wed, Nov 1, 2017 at 9:32 AM, Rich Felker wrote:
> On Sun, Oct 15, 2017 at 06:16:57AM -0700, H.J. Lu wrote:
>> crt1.o is used to create dynamic and non-PIE static executables. Static
>> PIE needs to link with Pcrt1.o, instead of crt1.o, to relocate static PIE
>> at run-time. When -pg is used w
When we have a REG_DEAD note for a reg that is set in the new I2, we
drop the note on the floor (we cannot find whether to place it on I2
or on I3). But the code I added to do this has a bug and does not
always actually drop it. This patch fixes it.
But that on its own is too pessimistic, it tur
On Sun, Oct 15, 2017 at 06:16:57AM -0700, H.J. Lu wrote:
> crt1.o is used to create dynamic and non-PIE static executables. Static
> PIE needs to link with Pcrt1.o, instead of crt1.o, to relocate static PIE
> at run-time. When -pg is used with -static-pie, gPcrt1.o should be used.
>
> Tested on
On 10/31/2017 08:47 PM, Trevor Saunders wrote:
On Tue, Oct 31, 2017 at 11:38:48AM -0600, Jeff Law wrote:
On 10/31/2017 11:22 AM, Eric Botcazou wrote:
I don't see a reason not to other than a pretty small amount of work
each time we make a release.
I'm not sure it would be so small an amount o
This patch shrinks the lang_fn_decl's operator_code field from 16 to 6
bits, by storing the compressed operator code instead of the tree_code.
I rename DECL_OVERLOADED_OPERATOR_CODE to
DECL_OVERLOADED_OPERATOR_CODE_RAW, to remind that it's not a tree_code.
DECL_OVERLOADED_OPERATOR_CODE_IS rema
This patch adds a new enum for the overloaded operators, we only
enumerate the codes we need, and thus can shrink the ovl_op_info array.
We do need to create a mapping array to go from tree_code to ovl_op_code.
More significantly is the use of the base.u.bits.address_space field,
which is othe
On Wed, Nov 01, 2017 at 02:49:45PM +0100, Eric Botcazou wrote:
> > At least, the only spot that used to mention df_link in compare-elim.c was:
> > struct df_link *ref_chain;
> > ref_chain = DF_REF_CHAIN (use);
> > if (!ref_chain || !ref_chain->ref
> >
> > || !DF_REF_INSN_INFO (ref_chai
> At least, the only spot that used to mention df_link in compare-elim.c was:
> struct df_link *ref_chain;
> ref_chain = DF_REF_CHAIN (use);
> if (!ref_chain || !ref_chain->ref
>
> || !DF_REF_INSN_INFO (ref_chain->ref) || ref_chain->next != NULL)
>
> return false;
>
> rtx_insn
On Wed, Nov 01, 2017 at 02:32:02PM +0100, Eric Botcazou wrote:
> > 2017-10-31 Jakub Jelinek
> >
> > PR rtl-optimization/82778
> > PR rtl-optimization/82597
> > * compare-elim.c (struct comparison): Add in_a_setter field.
> > (find_comparison_dom_walker::before_dom_children): Rem
> Here is an untested patch (only tried the cmpelim_mult_uses_1.c testcase in
> aarch64 cross) that does that. I don't have aarch64 boxes around for easy
> trunk testing, can bootstrap/regtest it on x86_64-linux/i686-linux and
> maybe powerpc64le-linux though.
>
> If the memset (last_setter, 0, s
-- Forwarded message --
From: Aldy Hernandez
Date: Mon, Oct 16, 2017 at 9:52 AM
Subject: [patch] implement generic debug() for vectors and hash sets
To: gcc-patches
We have a generic mechanism for dumping types from the debugger with:
(gdb) call debug(some_type)
However, even
Hello,
just a little tweak to that transformation. There is some overlap between
the 2 versions, but it seemed easier to handle the NOP case (including the
case without convert and the vector case) separately from the
narrowing / sign-extending scalar integer case.
At some point it would be
On Fri, Oct 27, 2017 at 12:46:12PM +0200, Richard Biener wrote:
> On Fri, 27 Oct 2017, Jakub Jelinek wrote:
>
> > On Fri, Oct 27, 2017 at 12:31:46PM +0200, Richard Biener wrote:
> > > I fear it doesn't work at all with LTO (you'll always get the old ABI
> > > if I read the patch correctly). This
Hi all,
during development on OpenCoarrays I found these three issues in gfortran:
- caf_send: Did not treat char arrays as arrays when calling caf_send.
- gfc_trans_assignment_1: Conversion of character kind creates a loop variant
temporary which must not be put before the loop body, but withi
Committed to 7-branch as revision 254293. I will close the PR now.
Cheers
Paul
On 30 October 2017 at 22:16, Paul Richard Thomas
wrote:
> Dear Andre,
>
> Committed to trunk as revision 254244.
>
> In order to debug the code, I was forced to use 7-branch for
> development since there were depende
Hi.
As shown from examples with -O2 optimization level, previous approach was not
viable.
A more complex approach is necessary and can be shortly described as follows:
1) each function has assigned start and end line in a source file
2) group functions are these functions that begin on a line an
The testcase for an alpha glibc bug [1] that bit us again.
2017-11-01 Uros Bizjak
* gcc.target/alpha/sqrt.c: New test.
Tested on alphaev68-linux-gnu, committed to mainline SVN.
[1] https://sourceware.org/ml/libc-alpha/2017-04/msg00256.html
Uros.
Index: gcc.target/alpha/sqrt.c
==
48 matches
Mail list logo
