Thanks for doing this. Just a couple of comments about the .md stuff:
Michael Collison writes:
> diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
> index 5adc5ed..c6ae670 100644
> --- a/gcc/config/aarch64/aarch64.md
> +++ b/gcc/config/aarch64/aarch64.md
> @@ -3999,6 +39
On Thu, 18 May 2017, Bernd Edlinger wrote:
> Hi,
>
> unfortunately the first patch was still insufficient. I identified many
> more relatively new places where static tree objects are invisible to
> GC.
>
> Nathan, whatever you are doing, please do it a bit more slowly, thanks.
>
> Bootstrap a
On Fri, 19 May 2017, Marek Polacek wrote:
> extract_muldiv folds
>
> (n * 1 * z) * 50
>
> to
>
> (n * 50) * z
>
> which is a wrong transformation to do, because it may introduce an overflow.
> This resulted in a ubsan false positive. So we should just disable this
> folding altog
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2017-05-19 Richard Biener
PR middle-end/80764
* cfgexpand.c (expand_gimple_cond): Fix loop fixup.
* gcc.dg/torture/pr80764.c: New testcase.
Index: gcc/cfgexpand.c
Hi Daniel,
> Use of .struct in do_test.S causes breakages when gas isn't the
> assembler (e.g., Solaris). I also wasn't including TEST_ALWAYS_FLAGS in
> my CFLAGS resulting in super-ugly log files. Finally, this patch
> eliminates spam of "test unsupported" (limiting it to one printing).
>
> Sig
Hi,
this patch makes edge summaries to use call_summary template and renames them
as they are not inliner specific for few years.
Bootstrapped/regtested x86_64-linux.
Honza
* ipa-inline-analysis.c (cgraph_2edge_hook_list, cgraph_edge_hook_list,
inline_edge_removal_hook, inline_ed
On Tue, May 16, 2017 at 5:26 PM, Nathan Sidwell wrote:
> This patch breaks apart the creation of the pass manager from gcc::context's
> ctor. This will allow us to register additional dumps between the dumpfile
> manager creation and the pass manager. As I described in
> https://gcc.gnu.org/ml/g
On Thu, May 18, 2017 at 5:08 PM, Nathan Sidwell wrote:
> On 05/16/2017 11:26 AM, Nathan Sidwell wrote:
>>
>> This patch breaks apart the creation of the pass manager from
>> gcc::context's ctor. This will allow us to register additional dumps
>
> ... https://gcc.gnu.org/ml/gcc-patches/2017-05/msg
Hi Daniel,
> There are a few issues with my ms-sysv.exp tests:
>
> 1. Use of gas extensions in do_test.S cause breakages on Solaris,
> 2. Parallelization breaks when no make -j flag is passed,
> 3. Builds aren't adding TEST_ALWAYS_FLAGS, so log files filled with
>color escape codes, and
> 4. T
On Thu, May 18, 2017 at 9:05 PM, Martin Sebor wrote:
> The debug_tree() function doesn't print the DECL_READ_P flag
> for a VAR_DECL or PARM_DECL. The attached two like patch makes
> it do so.
Ok.
Richard.
> Martin
PING^2.
Thanks,
Martin
On Fri, May 19, 2017 at 4:32 AM, Jason Merrill wrote:
> A patch I've been putting together ran into strange memory corruption
> issues which turned out to be because the calculation in
> make_tree_vec_stat was overflowing and allocating a small TREE_VEC
> instead of a large one. This assert shoul
> I can guess what is happening here. It's a 40 bits unsigned long long
> field, (s.b-8) will be like:
> _1 = s.b
> _2 = _1 + 0xf8
> Also get_range_info returns value range [0, 0xFF] for _1.
> You'd need to check if _1(with range [0, 0xFF]) + 0xf8
> overflows agains
On Fri, May 19, 2017 at 11:09 AM, Robin Dapp wrote:
>> I can guess what is happening here. It's a 40 bits unsigned long long
>> field, (s.b-8) will be like:
>> _1 = s.b
>> _2 = _1 + 0xf8
>> Also get_range_info returns value range [0, 0xFF] for _1.
>> You'd need to check if _1(with
On Fri, May 19, 2017 at 12:13 PM, Bin.Cheng wrote:
> On Fri, May 19, 2017 at 11:09 AM, Robin Dapp wrote:
>>> I can guess what is happening here. It's a 40 bits unsigned long long
>>> field, (s.b-8) will be like:
>>> _1 = s.b
>>> _2 = _1 + 0xf8
>>> Also get_range_info returns value range
Late response now that I'm finished refreshing the patches.
On Mon, Nov 28, 2016 at 6:20 PM, Jason Merrill wrote:
> On Thu, Nov 24, 2016 at 8:50 AM, Richard Biener wrote:
>>> > + /* ??? We can't annotate types late, but for LTO we may not
>>> > +generate a location early either (gfortr
From: claziss
Hi Andrew,
Please fing a set of 7 small patches which are upgrating/fixing wrong
patterns. Please let me know if you need more info.
Cheers,
Claudiu
Claudiu Zissulescu (7):
[ARC] Make mulsi for A700 pattern commutative.
[ARC] Avoid use of hard registers before reg-alloc.
[
gcc/
2016-12-12 Claudiu Zissulescu
* config/arc/arc.c (arc_conditional_register_usage): Allow r30 to
be used by the reg-alloc.
---
gcc/config/arc/arc.c | 9 -
gcc/config/arc/arc.h | 3 ++-
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/gcc/config/arc/ar
From: Claudiu Zissulescu
2016-12-17 Claudiu Zissulescu
* config/arc/simdext.md (movv2hi_insn): Change predicate to avoid
scaled addresses.
---
gcc/config/arc/simdext.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/arc/simdext.md b/gcc/config/
gcc/
2017-04-10 Claudiu Zissulescu
* config/arc/arc.md (mulsi3): Avoid use of hard registers before
reg-alloc when having mul64 or mul32x16 instructions.
(mulsidi3): Likewise.
(umulsidi3): Likewise.
(mulsi32x16): New pattern.
(mulsi64): Likewise.
gcc/
2016-11-10 Claudiu Zissulescu
* config/arc/arc.md (mulsi3_700): Make it commutative.
---
gcc/config/arc/arc.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
index 71d076c..db5867c 100644
--- a/gcc/config/arc/arc.m
From: Claudiu Zissulescu
If the stack pointer is needed, emit a special barrier that will prevent
the scheduler from moving stores to the frame before the stack adjustment.
2017-01-03 Claudiu Zissulescu
* config/arc/arc.c (arc_expand_prologue): Emit a special barrier
to preve
This is a repost (unchanged) of the simple-object ELF support for
early LTO debug transfer from IL object to a separate debug-only object
file.
Bootstrapped and tested on x86_64-unknown-linux-gnu.
Richard.
2017-05-19 Richard Biener
include/
* simple-object.h (simple_object_
arc_can_eliminate is using arc_frmae_pointer_required() which is wrong
as the frame_pointer_needed can be set on different conditions. Fix it
by calling arc_frame_pointer_needed().
gcc/
2017-01-09 Claudiu Zissulescu
* config/arc/arc.c (arc_can_eliminate): Test against
arc_frame
gcc/
2016-12-20 Claudiu Zissulescu
* config/arc/arc.md (commutative_binary_comparison): Remove 'I'
constraint. It is not valid for the pattern.
(noncommutative_binary_comparison): Likewise.
---
gcc/config/arc/arc.md | 16
1 file changed, 8 insertions(+)
On Fri, May 19, 2017 at 09:58:45AM +0200, Richard Biener wrote:
> On Fri, 19 May 2017, Marek Polacek wrote:
>
> > extract_muldiv folds
> >
> > (n * 1 * z) * 50
> >
> > to
> >
> > (n * 50) * z
> >
> > which is a wrong transformation to do, because it may introduce an overflow.
> >
This is a repost of the main part of the early LTO debug support.
The only changes relative to the last post is in the dwarf2out.c
pieces due to Jasons review and Jakubs introduction of
DW_OP_GNU_variable_value.
I've also adjusted testcases for fallout (the asan backtraces do
give files / line nu
On Fri, 19 May 2017, Marek Polacek wrote:
> On Fri, May 19, 2017 at 09:58:45AM +0200, Richard Biener wrote:
> > On Fri, 19 May 2017, Marek Polacek wrote:
> >
> > > extract_muldiv folds
> > >
> > > (n * 1 * z) * 50
> > >
> > > to
> > >
> > > (n * 50) * z
> > >
> > > which is a wro
On Fri, 19 May 2017, Richard Biener wrote:
> On Fri, 19 May 2017, Marek Polacek wrote:
>
> > On Fri, May 19, 2017 at 09:58:45AM +0200, Richard Biener wrote:
> > > On Fri, 19 May 2017, Marek Polacek wrote:
> > >
> > > > extract_muldiv folds
> > > >
> > > > (n * 1 * z) * 50
> > > >
> > >
Hi Michael,
On 19 May 2017 at 07:12, Michael Collison wrote:
> Hi,
>
> This patch improves code generations for builtin arithmetic overflow
> operations for the aarch64 backend. As an example for a simple test case such
> as:
>
> Sure for a simple test case such as:
>
> int
> f (int x, int y,
Hi Michael,
On 19 May 2017 at 09:21, Richard Sandiford wrote:
> Thanks for doing this. Just a couple of comments about the .md stuff:
>
> Michael Collison writes:
>> diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
>> index 5adc5ed..c6ae670 100644
>> --- a/gcc/config/
Hi!
Ping.
On Thu, 11 May 2017 14:24:05 +0200, I wrote:
> OK for trunk?
>
> commit 0ba48b4faf85420fbe12971afdd6e0afe70778bb
> Author: Thomas Schwinge
> Date: Fri May 5 16:41:59 2017 +0200
>
> Runtime checking of OpenACC parallelism dimensions clauses
>
> libgomp/
>
On 05/19/2017 03:51 AM, Richard Biener wrote:
you are commoning 'hwi' here. Also a bad (very short) name for a global
(even if static).
I'll leave review to Nathan anyway.
IMHO the C FE changes are obvious, with the fixing of the hwi name I
guess. I'll go rename fn1..6 in the C++ FE once
Hi!
Ping.
On Thu, 11 May 2017 14:26:51 +0200, I wrote:
> [...] support the num_gangs, num_workers, vector_length
> clauses for the OpenACC 2.5 kernels construct. OK for trunk?
>
> commit a689c52cde71960bc08ae30c3f88980f66fdd0b8
> Author: Thomas Schwinge
> Date: Thu May 11 13:43:28 2017 +0200
Hi!
Ping.
On Fri, 31 Mar 2017 17:05:29 +0200, I wrote:
> It would appear that ever since the first version of gcc/omp-low.c got
> committed to GCC trunk (more than ten years ago), it extraced OpenMP
> default clause data that it doesn't actually use for anything. OK to
> commit the following cle
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2017-05-19 Richard Biener
PR build/80821
* genmatch.c (dt_node::gen_kids_1): Add missing scope around
predicate evaluation.
Index: gcc/genmatch.c
==
On Wed, May 17, 2017 at 10:45 AM, Martin Liška wrote:
> Hi.
>
> As discussed on IRC some time ago, there are configurations that produce
> -Werror=maybe-uninitialized
> and other warnings that break bootstrap. So that, the patch makes -Werror
> just for default (boostrap-debug)
> configuration.
If we cannot load a constant into a register in one insn, and that
constant is a valid mask (for rotate instructions), we currently
prefer to load -1 and then mask it. This patch makes us not do that if
instead we could use two add or or instructions, since those are
sometimes faster on certain CP
Hi Jan,
> this patch makes edge summaries to use call_summary template and renames them
> as they are not inliner specific for few years.
>
> Bootstrapped/regtested x86_64-linux.
>
> Honza
>
> * ipa-inline-analysis.c (cgraph_2edge_hook_list, cgraph_edge_hook_list,
> inline_edge_removal
When I copied the searchers from std::experimental to std I forgot to
add the new std::search overload.
PR libstdc++/80796
* include/bits/stl_algo.h (search): Add new overload for C++17.
* testsuite/25_algorithms/search/searcher.cc: New.
Tested powerpc64le-linux, committe
Richard Sandiford wrote:
> Insn patterns shouldn't check can_create_pseudo_p, because there's no
> guarantee that the associated split happens before RA. In this case it
> should be safe to reuse operand 0 after RA if you change it to:
The goal is to only create and split this pattern before reg
On Thu, May 18, 2017 at 01:00:39PM +0200, Richard Biener wrote:
>
> The following reverts an earlier change of mine (2008) to explicitely
> warn about accessing alias-set zero memory with alias-set non-zero.
> That was supposed to catch the case in g++.dg/warn/Wstrict-aliasing-6.C
> which is
>
>
On 11/05/2017 14:54:37, Prakhar Bahuguna wrote:
> tls-disable-literal-pool.c should only be run if the toolchain and target
> support native thread-local storage rather than emulated TLS. This patch also
> improves the matching of the error message.
>
> testsuite/ChangeLog:
>
> 2017-05-11 Prakha
On Fri, 19 May 2017, Marek Polacek wrote:
> On Thu, May 18, 2017 at 01:00:39PM +0200, Richard Biener wrote:
> >
> > The following reverts an earlier change of mine (2008) to explicitely
> > warn about accessing alias-set zero memory with alias-set non-zero.
> > That was supposed to catch the case
>
> and way way more (C, Ada, ...). Assuming that the ipa-predicate.[ch]
> patch was really only cosmetic, this only leaves your patch in this rev
> range.
>
> Please fix.
Hi,
I apologize for the breakage. I already commited a fix before lunch but seems my
email did not get through. I am attach
Hi Prakhar,
On 19/05/17 13:29, Prakhar Bahuguna wrote:
On 11/05/2017 14:54:37, Prakhar Bahuguna wrote:
tls-disable-literal-pool.c should only be run if the toolchain and target
support native thread-local storage rather than emulated TLS. This patch also
improves the matching of the error messa
On Mon, May 15, 2017 at 5:58 PM, Bin.Cheng wrote:
> On Thu, May 11, 2017 at 12:02 PM, Richard Biener
> wrote:
>> On Tue, Apr 18, 2017 at 12:54 PM, Bin Cheng wrote
>>> Hi,
>>> Based on vect_peeling algorithm, we know for sure that vectorized loop must
>>> iterates at least once.
>>> This patch s
On 05/19/2017 01:24 PM, Richard Biener wrote:
> On Wed, May 17, 2017 at 10:45 AM, Martin Liška wrote:
>> Hi.
>>
>> As discussed on IRC some time ago, there are configurations that produce
>> -Werror=maybe-uninitialized
>> and other warnings that break bootstrap. So that, the patch makes -Werror
> > struct D: B {
> > char buf[32];
> > virtual void* f (unsigned n) {
> > if (n < 32)
> > return n <= 32 ? buf : B::f (n);
> > }
> >
> > Breaking foo's attribute malloc constraint.
> >
> > In other words, I think virtual functions need to be excluded
> > from the list (
On Fri, May 19, 2017 at 2:56 PM, Martin Liška wrote:
> On 05/19/2017 01:24 PM, Richard Biener wrote:
>> On Wed, May 17, 2017 at 10:45 AM, Martin Liška wrote:
>>> Hi.
>>>
>>> As discussed on IRC some time ago, there are configurations that produce
>>> -Werror=maybe-uninitialized
>>> and other war
Hi again,
I wrote:
== The attached patch to libbacktrace is intended to fix a memory
== allocation bug involving reading of line table information.
I've revised my previous patch to include a new test ("edtest") that
verifies the fix.
Thanks, Than
---
Patch (take 2):
diff --git a/Makefile.am
>
> * LTO and memory management
> This is a general question about LTO and memory management.
> IIUC the following sequence takes place during normal LTO:
> LGEN: generate_summary, write_summary
> WPA: read_summary, execute ipa passes, write_opt_summary
>
> So I assumed it was OK in LGEN to alloc
Hi!
On Sun, 9 Apr 2017 18:22:00 +0200, Martin Jambor wrote:
> On Wed, Apr 05, 2017 at 10:11:34AM +0200, Thomas Schwinge wrote:
> > I ran into a case where the libgomp hsa plugin wouldn't load. The
> > following patch helped me to quickly diagnose and then fix that. OK for
> > trunk?
>
> Yes, t
Original post: https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01030.html
The attached patch fixes an issue for the msp430 target where the TYPE_SIZE of
the __int20 type was set using the precision (20 bits) instead of the in-memory
size (32 bits) of the type. This bug caused an ICE as reported in P
Hi!
On Wed, 29 Mar 2017 00:19:30 +0200, Jakub Jelinek wrote:
> On Tue, Mar 28, 2017 at 11:35:35AM +0200, Thomas Schwinge wrote:
> > I noticed that in C++, for a directive like:
> >
> > #pragma acc parallel default (hello)
> >
> > ... we currently print two error messages for the same locati
Hi!
On Wed, 10 May 2017 17:48:36 +0200, Jakub Jelinek wrote:
> On Fri, Apr 07, 2017 at 05:08:55PM +0200, Thomas Schwinge wrote:
> > OpenACC 2.5 added a default (present) clause, which "causes all arrays or
> > variables of aggregate data type used in the compute construct that have
> > implicitly
Hi!
On Wed, 10 May 2017 17:48:36 +0200, Jakub Jelinek wrote:
> On Fri, Apr 07, 2017 at 05:08:55PM +0200, Thomas Schwinge wrote:
> > OpenACC 2.5 added a default (present) clause, which "causes all arrays or
> > variables of aggregate data type used in the compute construct that have
> > implicitly
On 05/19/17 09:51, Richard Biener wrote:
> On Thu, 18 May 2017, Bernd Edlinger wrote:
>
>> Hi,
>>
>> unfortunately the first patch was still insufficient. I identified many
>> more relatively new places where static tree objects are invisible to
>> GC.
>>
>> Nathan, whatever you are doing, please
On 05/19/2017 10:05 AM, Bernd Edlinger wrote:
hwi cannot be the root cause of the problem,
because it can only be long_integer_type_node
or long_long_integer_type_node, otherwise
an error would be triggered.
that's the error I made with the static fns. PCH moves things around,
so anything th
This patch gets the remaining stragglers outside of name-lookup.
Nothing particularly exciting here.
Remaining patches will be primarily in name-lookup.c, with a few in
cp-tree.h, tree.c and the parser. These will be the patches that are
the significant changes (all this lot was just ground w
On 05/19/17 16:10, Nathan Sidwell wrote:
> On 05/19/2017 10:05 AM, Bernd Edlinger wrote:
>
>> hwi cannot be the root cause of the problem,
>> because it can only be long_integer_type_node
>> or long_long_integer_type_node, otherwise
>> an error would be triggered.
>
> that's the error I made with
Hello.
Following mechanical change moves the function to symbol_table where one would
expect such a function.
Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
Ready to be installed?
Martin
>From f12a6fc831eb0d78d57572b8fcbb9f170d84d773 Mon Sep 17 00:00:00 2001
From: mar
Hello.
We repeat all over following patterns:
fprintf (dump_file, "Removing speculative call %s/%i => %s/%i\n",
xstrdup_for_dump (edge->caller->name ()),
edge->caller->order,
xstrdup_for_dump (e2->callee->name ()),
> Hello.
>
> Following mechanical change moves the function to symbol_table where one would
> expect such a function.
>
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Ready to be installed?
OK,
thanks!
Honza
> Martin
> >From f12a6fc831eb0d78d57572b8fcbb9f170d84
On 18/05/17 19:10 +0800, Xi Ruoyao wrote:
This UB has been hiding so long...
Indeed! Thanks for the patch.
2017-03-11 Xi Ruoyao
PR libstdc++/67214
* include/bits/locale_facets.tcc (_M_extract_int):
Add explicit conversion to avoid signed overflow.
---
libstdc++-
> Hello.
>
> We repeat all over following patterns:
>
> fprintf (dump_file, "Removing speculative call %s/%i => %s/%i\n",
> xstrdup_for_dump (edge->caller->name ()),
> edge->caller->order,
> xstrdup_for_dump (e2->callee->n
[Adding Thomas]
On 05/08/2017 04:30 PM, Eli Zaretskii wrote:
> When compiling libiberty (as part of GDB) with MinGW on MS-Windows, I
> see the following warning:
>
> gcc -c -DHAVE_CONFIG_H -O2 -gdwarf-4 -g3 -D__USE_MINGW_ACCESS -I.
> -I./../include -W -Wall -Wwrite-strings -Wc++-compat -
On Fri, May 19, 2017 at 01:57:24PM +0300, Alexander Monakov wrote:
> On Fri, 19 May 2017, Richard Biener wrote:
>
> > On Fri, 19 May 2017, Marek Polacek wrote:
> >
> > > On Fri, May 19, 2017 at 09:58:45AM +0200, Richard Biener wrote:
> > > > On Fri, 19 May 2017, Marek Polacek wrote:
> > > >
> >
On 05/08/2017 04:27 PM, Eli Zaretskii wrote:
> When compiling libiberty (as part of GDB) with MinGW on MS-Windows, I
> see the following warning:
>
> gcc -c -DHAVE_CONFIG_H -O2 -gdwarf-4 -g3 -D__USE_MINGW_ACCESS -I.
> -I./../include -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototype
> Cc: gdb-patc...@sourceware.org, Thomas Schwinge
> From: Pedro Alves
> Date: Fri, 19 May 2017 16:22:55 +0100
>
> But then, xstrndup.c has at the top:
>
> #ifdef HAVE_STRING_H
> #include
> #else
> # ifdef HAVE_STRINGS_H
> # include
> # endif
> #endif
>
> So I would expect your build to pick
On Fri, 19 May 2017, Marek Polacek wrote:
> > I think it's possible to keep this folding, note that it's valid to
> > transform to
> >
> > (n * 1 * z) * 50
> >
> > (i.e. accumulate multiplications on the outermost factor)
(to be precise, if the multiplication is done in a signed type an
On 05/08/2017 04:25 PM, Eli Zaretskii wrote:
> When compiling libiberty (as part of GDB) with MinGW on MS-Windows, I
> see the following warning:
>
> gcc -c -DHAVE_CONFIG_H -O2 -gdwarf-4 -g3 -D__USE_MINGW_ACCESS -I.
> -I./../include -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototype
On 05/18/2017 07:06 PM, Steve Kargl wrote:
On Thu, May 18, 2017 at 05:16:45PM -0700, Jerry DeLisle wrote:
2017-05-18 Paul Thomas
PR fortran/80333
* trans-io.c (nml_get_addr_expr): If we are dealing with class
type data set tmp tree to get that address.
(trans
Hi,
Revision 247881 possibly exposes bug in RTL or ARM backend, while the change
itself
may be incomplete too. Given a TRUNCATE rtx with complicated sub-rtx, backend
may
want to know the complicated sub-rtx happens in context of TRUNCATE in order to
give
different costs. This patch adds restri
On 05/19/2017 04:40 PM, Eli Zaretskii wrote:
>> Cc: gdb-patc...@sourceware.org, Thomas Schwinge
>> From: Pedro Alves
>> Date: Fri, 19 May 2017 16:22:55 +0100
>>
>> But then, xstrndup.c has at the top:
>>
>> #ifdef HAVE_STRING_H
>> #include
>> #else
>> # ifdef HAVE_STRINGS_H
>> # include
>> # e
On Fri, May 19, 2017 at 6:18 AM, Than McIntosh wrote:
>
> I wrote:
> == The attached patch to libbacktrace is intended to fix a memory
> == allocation bug involving reading of line table information.
>
> I've revised my previous patch to include a new test ("edtest") that
> verifies the fix.
Than
Hi,
while looking into some bugs (eg, c++/71464) I noticed a few more of
those consecutive errors, which I propose to adjust per the below
patchlet. The second case in add_method is a bit tricky because in
principle we'd really like to be more specific (eg, clang talks about
constructors whic
On May 19, 2017 5:47:10 PM GMT+02:00, Alexander Monakov
wrote:
>On Fri, 19 May 2017, Marek Polacek wrote:
>> > I think it's possible to keep this folding, note that it's valid to
>transform to
>> >
>> > (n * 1 * z) * 50
>> >
>> > (i.e. accumulate multiplications on the outermost factor)
-mno-gen-cell-microcode is a pain to handle correctly: it causes
different code generation for some very basic patterns, even patterns
specific to that option. It also requires marking up many patterns,
which is a pain when adding new patterns or modifying existing ones
(first non-trivial step is
--enable-languages=all confused me, because as RichardB said, 'it really
means default'. So this patch does 2 things
1) allow --enable-languages=default to mean what =all does now.
2) change =all to mean all available languages.
2.1) jit is included in =all if -enable-host-shared is given. If y
Hi Bernd,
Your patches are causing troubles when I try to do "incremental updates »:
After revision r248290 it fails with
../../work/gcc/cp/init.c:4916:10: fatal error: gt-cp-init.h: No such file or
directory
#include "gt-cp-init.h »
and after r248242 with
../../work/gcc/c-family/c-format.c:
> Cc: gdb-patc...@sourceware.org
> From: Pedro Alves
> Date: Fri, 19 May 2017 16:36:46 +0100
>
> So I wonder whether we could just unconditionally remove the waitpid
> replacement instead.
That's probably the best path forward.
On 05/19/17 19:05, Dominique d'Humières wrote:
> Hi Bernd,
>
> Your patches are causing troubles when I try to do "incremental updates »:
>
> After revision r248290 it fails with
>
> ../../work/gcc/cp/init.c:4916:10: fatal error: gt-cp-init.h: No such file or
> directory
> #include "gt-cp-ini
> Cc: gdb-patc...@sourceware.org
> From: Pedro Alves
> Date: Fri, 19 May 2017 16:51:30 +0100
>
> So again there's a system header that defines the symbol
> but for some reason libiberty still wants to declare/define
> it is if it weren't?
Yes. AFAICS, libiberty's configure script doesn't check
> Le 19 mai 2017 à 19:26, Bernd Edlinger a écrit :
>
> No, unfortunately it looks like incremental does not work in this case.
>
> You can try to remove gcc/s-gtyp-input, maybe it un-breaks your
> build. However, I gave up at that point and did a full bootstrap
> instead.
>
> Bernd.
Quite b
On Fri, 19 May 2017, Alexander Monakov wrote:
> On Fri, 19 May 2017, Marek Polacek wrote:
> > > I think it's possible to keep this folding, note that it's valid to
> > > transform to
> > >
> > > (n * 1 * z) * 50
> > >
> > > (i.e. accumulate multiplications on the outermost factor)
>
>
> Just so I am clear, gcc/testsuite/ChangeLog needs to be
> checked into the patch (but gcc/ChangeLog is not)?
This 4/4 commit needs to contain gcc/config/sparc/sparc.c, gcc/ChangeLog,
gcc/testsuite/gcc.target/sparc/niagara7-align.c and gcc/testsuite/ChangeLog.
The rule is that you put the Chang
> The rule is that you put the ChangeLog entry into the ChangeLog file of the
> directory where the change is made, or that of the parent directory if there
> is none, recursively. The filename in that ChangeLog file must be relative
> to the directory when the ChangeLog file is.
...where...
--
Hi
It looks alright. No questions to ask, if you need me to upstream it
please ping me.
Cheers,
Claudiu
On Mon, May 15, 2017 at 7:07 PM, Graham Markall
wrote:
> Hi Claudiu,
>
> I ran the gcc testsuite with EZsim for NPS-400:
>
> $ ./EZsim_linux_x86_64 --version
> NPS-400 EZsim - Version 1.9
On Fri, May 19, 2017 at 5:14 AM, Richard Biener
wrote:
> On Fri, May 19, 2017 at 4:32 AM, Jason Merrill wrote:
>> A patch I've been putting together ran into strange memory corruption
>> issues which turned out to be because the calculation in
>> make_tree_vec_stat was overflowing and allocating
On 05/19/2017 02:45 PM, Eric Botcazou wrote:
This 4/4 commit needs to contain gcc/config/sparc/sparc.c, gcc/ChangeLog,
gcc/testsuite/gcc.target/sparc/niagara7-align.c and gcc/testsuite/ChangeLog.
The rule is that you put the ChangeLog entry into the ChangeLog file of the
directory where the ch
On Tue, May 16, 2017 at 5:39 PM, Martin Sebor wrote:
> On 05/16/2017 01:41 PM, Jason Merrill wrote:
>
>> I'm still not convinced we need to consider standard-layout at all.
>
> I agree. The patch doesn't make use of is_standard_layout_p().
> It defines its own helper called almost_std_layout_p()
On Thu, Apr 27, 2017 at 12:59 PM, Jonathan Wakely wrote:
> I also tried to add a warning like EDG's (see the PR) but it gave a
> false positive for direct-list-init of scoped enums (P0138R2, r240449)
> because that code goes through build_c_cast to perform the conversion,
> so looks like a cast to
It doesn't make sense to warn about a switch case not being an
enumerator if the enum type has no enumerators; such a type is being
used as an opaque typedef rather than a true enumerated type. This
usage makes more sense with C++11 enums with explicitly specified
underlying type, but I don't thin
Hi
On 15/05/2017 21:31, Marc Glisse wrote:
The __fill_bvector part of the fill overload for vector could do
with some improvements as well. Looping is unnecessary, one just needs
to produce the right mask and and or or with it, that shouldn't take
more than 4 instructions or so.
I have impl
On Fri, 19 May 2017, Joseph Myers wrote:
> On Fri, 19 May 2017, Alexander Monakov wrote:
> > (to be precise, if the multiplication is done in a signed type and the
> > middle
> > constant factor was a negated power of two, the sign change needs to remain:
> >
> > a * -4 * b * 2
> >
> > needs
On May 19, 2017, at 9:59 AM, Nathan Sidwell wrote:
>
> --enable-languages=all confused me, because as RichardB said, 'it really
> means default'. So this patch does 2 things
> 1) allow --enable-languages=default to mean what =all does now.
> 2) change =all to mean all available languages.
> 2.1
On 05/19/2017 01:07 PM, Jason Merrill wrote:
On Tue, May 16, 2017 at 5:39 PM, Martin Sebor wrote:
On 05/16/2017 01:41 PM, Jason Merrill wrote:
I'm still not convinced we need to consider standard-layout at all.
I agree. The patch doesn't make use of is_standard_layout_p().
It defines its o
Christophe,
I had a type in the two test cases: "addcs" should have been "adcs". I caught
this previously but submitted the previous patch incorrectly. Updated patch
attached.
Okay for trunk?
-Original Message-
From: Christophe Lyon [mailto:christophe.l...@linaro.org]
Sent: Friday, Ma
On Fri, May 19, 2017 at 4:07 PM, Martin Sebor wrote:
> On 05/19/2017 01:07 PM, Jason Merrill wrote:
>>
>> On Tue, May 16, 2017 at 5:39 PM, Martin Sebor wrote:
>>>
>>> On 05/16/2017 01:41 PM, Jason Merrill wrote:
>>>
I'm still not convinced we need to consider standard-layout at all.
>>>
>>>
1 - 100 of 113 matches
Mail list logo