On Thu, Aug 14, 2014 at 11:29 PM, Sebastian Pop seb...@gmail.com wrote:
Bin.Cheng wrote:
The overflow check can be improved by using deeper inspection to prove the
equality. This patch deals with that by making below two improvements:
a) Handles constant cases.
b) Uses affine
On Thu, Aug 14, 2014 at 8:53 PM, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
In order for your work to be generally useful, I'd ask several things:
- Update
https://code.google.com/p/address-sanitizer/wiki/CompileTimeOptimizations
with examples that will be handled
Done
On Fri, Aug 8, 2014 at 2:43 PM, Dmitry Vyukov dvyu...@google.com wrote:
Similar optimization could be used for tsan builtins, or some of the ubsan
builtins (which is the reason why the pass is called sanopt).
That would be great.
Note that tsan optimizations must be based around a different
On Thu, 14 Aug 2014, Jan Hubicka wrote:
And this fixes the thinko in zlib use.
du /tmp/*ltrans* | awk '{ sum += $1 } END { print sum }'
stage3 cc1 WPA ltrans file size 178740 (patched)
stage3 cc1 WPA ltrans file size 460068 (unpatched)
The patch works now with Firefox. For
Hi!
We need OMP_CLAUSE_SHARED even for global vars if they are mapped
in an outer target region, because then the parallel region can't
access the global, but has to go through the mapping.
Testcase already in the testsuite (but failed only with non-shared
address space obviously).
On Thu, 14 Aug 2014, Sebastian Pop wrote:
Richard Biener wrote:
The following fixes missing dominator computation before fixing loops.
Rather than doing even more such weird stuff in a pass gate function
this puts this into a new pass scheduled before the loop passes gate.
Ok.
Hi!
For these two, we need to actually map 3 things, the reference
(pointer-assign), the pointer (pointer-assign) and what the pointer points
to.
Testcase already in the testsuite, just needs non-shared address space.
Bootstrapped/regtested on x86_64-linux and i686-linux, committed to
patched:
real6m12.437s
user51m18.829s
sys 4m30.809s
WPA is 129s, stream in 29.23s, stream out 12.16s.
Patched + fast compression
real6m4.383s
user49m15.123s
sys 4m31.166s
WPA is 124s, stream in 29.39, stream out 7.33s.
So I guess the
On Fri, 15 Aug 2014, Jan Hubicka wrote:
patched:
real6m12.437s
user51m18.829s
sys 4m30.809s
WPA is 129s, stream in 29.23s, stream out 12.16s.
Patched + fast compression
real6m4.383s
user49m15.123s
sys 4m31.166s
WPA is 124s, stream
The following makes tree_swap_operands_p put all constants 2nd place,
also looks through sign-changes when considering further canonicalzations
and removes the odd -Os guard for those. That was put in with
https://gcc.gnu.org/ml/gcc-patches/2003-10/msg01208.html just
motivated by CSiBE numbers -
points-to is one place that needs special handling of us
allowing string as operands (rather than having a
CONST_DECL for each string literal). The following patch
realizes that readonly_id is only used for string literals
and adjusts code accordingly.
Bootstrap and regtest running on
On 15 August 2014 11:07, Richard Biener rguent...@suse.de wrote:
- if (TREE_CODE (arg1) == INTEGER_CST)
+ if (CONSTANT_CLASS_P (arg1) == INTEGER_CST)
Huh?
/* Nonzero if NODE represents a constant. */
#define CONSTANT_CLASS_P(NODE)\
(TREE_CODE_CLASS (TREE_CODE (NODE)) == tcc_constant)
On Fri, 15 Aug 2014, Manuel López-Ibáñez wrote:
On 15 August 2014 11:07, Richard Biener rguent...@suse.de wrote:
- if (TREE_CODE (arg1) == INTEGER_CST)
+ if (CONSTANT_CLASS_P (arg1) == INTEGER_CST)
Huh?
Eh ;)
/* Nonzero if NODE represents a constant. */
#define
On Thu, Aug 14, 2014 at 8:45 PM, Sebastian Pop seb...@gmail.com wrote:
Steve Ellcey wrote:
I understand the desire not to add optimizations just for benchmarks but
we do know other compilers have added this optimization for coremark
(See
On Thu, Aug 14, 2014 at 8:25 PM, Steve Ellcey sell...@mips.com wrote:
On Thu, 2014-08-14 at 10:21 -0600, Jeff Law wrote:
On 08/14/14 10:12, David Malcolm wrote:
On Thu, 2014-08-14 at 09:56 -0600, Jeff Law wrote:
On 08/14/14 04:32, Richard Biener wrote:
You'll note in a separate thread
Hi!
As discussed in the PR, nerrs(i,io) isn't allocatable var even
when nerrs is allocatable, thus a reasonable reading of the new OpenMP 4.0
requirement is that the testcase is still valid.
I went ahead and committed the change suggested by Tobias. For the rest
mentioned in the PR, more
Hi,
This patch adds myself to the MAINTAINERS file. Commmitted as 214012.
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 87fb9dd..a40a537 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -555,6 +555,7 @@ Dinar Temirbulatov
On Fri, Aug 15, 2014 at 12:13 PM, Richard Biener
richard.guent...@gmail.com wrote:
On Thu, Aug 14, 2014 at 8:25 PM, Steve Ellcey sell...@mips.com wrote:
On Thu, 2014-08-14 at 10:21 -0600, Jeff Law wrote:
On 08/14/14 10:12, David Malcolm wrote:
On Thu, 2014-08-14 at 09:56 -0600, Jeff Law
I've attached the patch, which should eliminate CLooG library
installation dependency from GCC. The CLooG AST generator is still the
main code generator, but the isl ast generator will be chosen in case
of nonavailability of CLooG library.
However, I've found out a problem. Almost all the
Hello,
This patch introduces AVX-512BW's psadbw insn pattern.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_mode_iterator VI8_AVX2_AVX512BW): New.
(define_insn sse2_avx2_psadbw): Add evex
Hello,
This patch splits VI48_AVX512F iterator into two.
It extends vcvtps2udq,vpbroadcastmb2d patterns as well.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_mode_iterator VI48_AVX512F): Delete.
On Fri, Aug 15, 2014 at 12:42 PM, Ilya Tocar tocarip.in...@gmail.com wrote:
Hi,
This patch adds myself to the MAINTAINERS file. Commmitted as 214012.
Please keep this list sorted alphabetically.
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)
diff --git a/MAINTAINERS
On Fri, Aug 15, 2014 at 1:13 PM, Roman Gareev gareevro...@gmail.com wrote:
I've attached the patch, which should eliminate CLooG library
installation dependency from GCC. The CLooG AST generator is still the
main code generator, but the isl ast generator will be chosen in case
of
I checked that zeroing destination operand for unary bit-manipulation
instruction is helpful for 64- and 32-bit mode only. So the patch was
changed.
Is it OK for trunk?
gcc/ChangeLog
2014-08-15 Yuri Rumyantsev ysrum...@gmail.com
PR target/62011
* config/i386/i386-protos.h
On Fri, Aug 15, 2014 at 01:43:55PM +0200, Richard Biener wrote:
On Fri, Aug 15, 2014 at 12:42 PM, Ilya Tocar tocarip.in...@gmail.com wrote:
Hi,
This patch adds myself to the MAINTAINERS file. Commmitted as 214012.
Please keep this list sorted alphabetically.
Well, Ilya did, just
Hello,
This patch extends pattern for vpbroadcastmb2q insn
pattern.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_mode_iterator VI8_AVX512VL): New.
(define_insn avx512cd_maskb_vec_dupmode):
This patch adds myself to the MAINTAINERS file. Commmitted as 214012.
Please keep this list sorted alphabetically.
Sorry attached wrong version of the patch.
Actually commited vesrion (rev 214012), has alphabetical order
Index: ChangeLog
Hello,
This patch introduces new patterns to support
AVX-512Vl,DQ broadcast insns.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_mode_iterator VI4F_BRCST32x2): New.
(define_mode_attr
On Fri, Aug 15, 2014 at 03:45:33PM +0400, Yuri Rumyantsev wrote:
gcc/ChangeLog
2014-08-15 Yuri Rumyantsev ysrum...@gmail.com
PR target/62011
* config/i386/i386-protos.h (ix86_avoid_false_dep_for_bm): New function
prototype.
* config/i386/i386.c (ix86_avoid_false_dep_for_bm): New
Hello,
This patch extends shift pattern to support AVX-512
new insn.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_mode_iterator VI248_AVX2): Add V32HI mode.
(define_insn
Richard!
Here is updated patch with the following changes:
1. Any restrictions on phi-function were eliminated for extended conversion.
2. Put predicate for critical edges to 'aux' field of edge, i.e.
negate_predicate was deleted.
3. Deleted splitting of critical edges, i.e. both outgoing edges
Just committed this patch, fixing the bootstrap.
2014-08-13 Andrew Sutton andrew.n.sut...@gmail.com
Fix regression in bootstrap.
* gcc/cp/call.c (get_temploid): Removed. No longer called.
(joust): Remove unused variable declarations.
Andrew
On Wed, Aug 13, 2014 at 9:50 PM, Andrew Sutton
Jakub,
Is it important to have correct value for length attribute for Big Cores?
As I new this attribute is used for code layout alignment.
2014-08-15 15:54 GMT+04:00 Jakub Jelinek ja...@redhat.com:
On Fri, Aug 15, 2014 at 03:45:33PM +0400, Yuri Rumyantsev wrote:
gcc/ChangeLog
2014-08-15
Ping.
On 29 Jul 18:45, Ilya Tocar wrote:
Hi,
As discussed here in https://gcc.gnu.org/ml/gcc/2014-01/msg00189.html
Gcc should complain about pragma omp declare target without
corresponding pragma omp end declare target. This patch adds a warning
for those cases.
Bootstraps/passes
This removes the deprecated APIs and inlines them into their single
caller. This exposes publically the 'code_helper' class which will
also be needed for manually written simplifiers (that's next).
Bootstrapped on x86_64-unknown-linux-gnu, applied.
Richard.
2014-08-15 Richard Biener
Hi,
This patch adds myself to the MAINTAINERS file. Commmitted as 214017.
Index: MAINTAINERS
===
--- MAINTAINERS (revision 214014)
+++ MAINTAINERS (working copy)
@@ -561,6 +561,7 @@
David Ung
Am 15.08.2014 um 11:32 schrieb Manuel López-Ibáñez:
On 15 August 2014 11:07, Richard Biener rguent...@suse.de wrote:
- if (TREE_CODE (arg1) == INTEGER_CST)
+ if (CONSTANT_CLASS_P (arg1) == INTEGER_CST)
Huh?
/* Nonzero if NODE represents a constant. */
#define CONSTANT_CLASS_P(NODE)\
The following introduces manually written patterns. That is,
part of the matching and the transform are fully manual. An
example where this is necessary is when the result isn't really
an expression but a series of statements.
For example take simplifications of the memset builtin. With
the
On 15 August 2014 14:43, Franz Sirl franz.sirl-ker...@lauterbach.com wrote:
Am 15.08.2014 um 11:32 schrieb Manuel López-Ibáñez:
On 15 August 2014 11:07, Richard Biener rguent...@suse.de wrote:
- if (TREE_CODE (arg1) == INTEGER_CST)
+ if (CONSTANT_CLASS_P (arg1) == INTEGER_CST)
Huh?
/*
On 08/15/14 04:07, Richard Biener wrote:
On Thu, Aug 14, 2014 at 8:45 PM, Sebastian Pop seb...@gmail.com wrote:
Steve Ellcey wrote:
I understand the desire not to add optimizations just for benchmarks but
we do know other compilers have added this optimization for coremark
(See
Am 06.08.2014 18:50, schrieb Manuel López-Ibáñez:
This is the first step for moving Fortran to use the common
diagnostics machinery. This patch makes Fortran use the common
machinery for those warnings that don't have a location or a
controlling option.
Bootstrapped and regression tested on
Hi Joseph,
I seem to address what you've said.
Except for I still left non_bionic check for three tests
(Namely: builtins-67.c, strlenopt-14g.c, strlenopt-14gf.c)
because checks for the presence of mempcpy, stpcpy and rintl (sorry, I
didn't mention it last time)
seem to be very narrow, I don't
Hi,
This patch moves the location of various Wformat warnings from
pointing to the first character of the function name (such as printf)
to the actual format string. This is specially useful when you have
something like
printf (cond ? format 1 : format 2);
It also moves (some) of the warnings
On Fri, Aug 15, 2014 at 2:26 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Is it important to have correct value for length attribute for Big Cores?
As I new this attribute is used for code layout alignment.
2014-08-15 15:54 GMT+04:00 Jakub Jelinek ja...@redhat.com:
On Fri, Aug 15, 2014 at
On 08/14/2014 12:04 PM, Jakub Jelinek wrote:
No, this should be if, not else if, and be after the } below.
We really can't handle it otherwise.
Generally, the bitfield COMPONENT_REFs should have
DECL_BIT_FIELD_REPRESENTATIVE which is not a bitfield, therefore the common
case will be handled.
On Fri, Aug 15, 2014 at 06:53:25PM +0400, Yury Gribov wrote:
2014-08-15 Yury Gribov y.gri...@samsung.com
gcc/
PR sanitizer/62089
* asan.c (instrument_derefs): Fix bitfield check.
gcc/testsuite/
PR sanitizer/62089
*
I should have pointed out that this is based on a preliminary patch by
Steven attached to PR 52952. I will update the Changelog to reflect
this when committing.
Cheers,
Manuel.
On 15 August 2014 15:51, Manuel López-Ibáñez lopeziba...@gmail.com wrote:
Hi,
This patch moves the location of
On 17 Jul 03:36, Jeff Law wrote:
On 04/16/14 05:52, Ilya Enkovich wrote:
diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index b8ca17e..d868129 100644
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -4333,6 +4333,13 @@ This hook returns the va_list type of the calling
convention specified by
Rewritten to meet the C++11 spec, not the 2009 draft I used when
adding nested_exception.
I've used the builtin traits directly, to avoid making libsupc++
depend on type_traits, which shouldn't be necessary because
type_traits should be present for freestanding implementations, but
it isn't (PR
Hi Vladimir,
The following testcase fails when compiled with -O2 -mips32r2:
long long a[];
long long b, c, d, k, m, n, o, p, q, r, s, t, u, v, w;
int e, f, g, h, i, j, l, x;
fn1() {
for (; x; x++)
if (x 1)
s = h | g;
else
s = f | e;
l = ~0;
m = 1 | k;
n = i;
o =
This seems to fix PR 62159, in that I can now include all the required
headers in a C++11 program using a compiler built with
--disable-libstdcxx-hosted, but 'make check' doesn't do anything for
freestanding builds - should I be doing anything else to test it?
commit
On 14/08/14 19:25, Steve Ellcey wrote:
On Thu, 2014-08-14 at 10:21 -0600, Jeff Law wrote:
On 08/14/14 10:12, David Malcolm wrote:
On Thu, 2014-08-14 at 09:56 -0600, Jeff Law wrote:
On 08/14/14 04:32, Richard Biener wrote:
You'll note in a separate thread Steve and I discussed this during
I committed the below as obvious in r214026.
Since the condition is already controlled by extra_warnings, the only
different is that we now print [-Wextra] and that -Werror=extra now
works.
Cheers,
Manuel.
Index: gcc/cp/ChangeLog
.. unfortunately something is wrong with this commit, thus c++/62072.
For the time being I'm simply reverting it and adding the new testcase.
By the way, more generally, I don't understand at the moment how we can
safely use complete_type in the middle of tsubst co: it can emit hard
errors
I tripped at a typo that goes undetected because the macro NAMED_PARAM
doesn't apply in the absence of designated initializers.
Since struct scale_addr_mode_cost has the cost for DI, but not for QI, the
instances of struct cpu_addrcost_table are not initialized as intended due
to the different
On Thu, Aug 14, 2014 at 2:51 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
On 14 Aug 13:45, Uros Bizjak wrote:
Please update the above entry.
Whoops. Updated ChangeLog:
gcc/
* config/i386/i386.c
(ix86_expand_special_args_builtin): Handle avx512vl_storev8sf_mask,
On 14/08/2014 20:48, Manuel López-Ibáñez wrote:
--- a/gcc/fortran/options.c
+++ b/gcc/fortran/options.c
@@ -693,6 +693,10 @@ gfc_handle_option (size_t scode, const char *arg,
int value,
gfc_option.warn_line_truncation = value;
break;
+case OPT_Wmissing_return:
+
It turned out that the CRITICAL patch had two issues:
a) The lock variable was named __lock_var@0. That's unambiguous but
the linker didn't like the file name. As the variable is unused (only
the associated token gets used), the assembler error only occurred with
-O0 and hence not in the test
On Fri, Aug 15, 2014 at 05:24:58PM +0100, Evandro Menezes wrote:
I tripped at a typo that goes undetected because the macro NAMED_PARAM
doesn't apply in the absence of designated initializers.
Since struct scale_addr_mode_cost has the cost for DI, but not for QI, the
instances of struct
On 08/14/2014 12:28 PM, Jason Merrill wrote:
On 08/14/2014 05:07 AM, Richard Biener wrote:
So - can you take over this C++ frontend issue?
OK.
Here's what I'm applying to trunk:
commit 76cce1d62861b6c99e3ecd97bcd607cd242d1efa
Author: Jason Merrill ja...@redhat.com
Date: Wed Aug 13
On Aug 15, 2014, at 6:49 AM, Alexander Ivchenko aivch...@gmail.com wrote:
* lib/target-supports.exp (error_h): New check.
(libc_has_complex_functions): Ditto.
(tgmath_h): Ditto.
* gcc.dg/builtins-59.c: Add libc_has_complex_functions check.
* gcc.dg/builtins-61.c:
On 08/15/2014 12:17 PM, Paolo Carlini wrote:
By the way, more generally, I don't understand at the moment how we can
safely use complete_type in the middle of tsubst co: it can emit hard
errors about, eg, incompleteness (see c++/62072) irrespective of the
tsubst_flags_t argument...
Yes, it
can you file bugs against bionic for stuff like this? use
b.android.com (and feel free to mail me to ensure that they get
noticed).
one thing we'd like to do is get to a point where we're building
gcc/gdb et cetera without any local hacks, and when we've got to that
point, we're going to have to
On Aug 15, 2014, at 10:32 AM, enh e...@google.com wrote:
can you file bugs against bionic for stuff like this?
I suspect you meant to ask Alexander this question… but just in case you did
intend to ask me, no. I don’t have any visibility into android local patches
or procedures.
In the end,
On Fri, Aug 15, 2014 at 10:56 AM, Mike Stump mikest...@comcast.net wrote:
On Aug 15, 2014, at 10:32 AM, enh e...@google.com wrote:
can you file bugs against bionic for stuff like this?
I suspect you meant to ask Alexander this question… but just in case you did
intend to ask me, no. I
On Fri, Aug 15, 2014 at 1:37 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello,
This patch introduces AVX-512BW's psadbw insn pattern.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
On Fri, Aug 15, 2014 at 5:45 PM, Robert Suchanek wrote:
gcc/
* rtlanal.c (get_base_term): Accept HIGH as the base term.
diff --git gcc/rtlanal.c gcc/rtlanal.c
index 82cfc1bf..2bea2ca 100644
--- gcc/rtlanal.c
+++ gcc/rtlanal.c
@@ -5624,6 +5624,7 @@ get_base_term (rtx *inner)
Hi,
On 08/15/2014 07:30 PM, Jason Merrill wrote:
On 08/15/2014 12:17 PM, Paolo Carlini wrote:
By the way, more generally, I don't understand at the moment how we can
safely use complete_type in the middle of tsubst co: it can emit hard
errors about, eg, incompleteness (see c++/62072)
Hi,
This adds a macro to indicate the order in which vector elements appear
in a register on PowerPC. Elements may appear in right-to-left order
for little endian, or in left-to-right order for big endian and when
-maltivec=be is selected for little endian. The same macro is being
implemented
On Fri, Aug 15, 2014 at 1:42 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello,
This patch splits VI48_AVX512F iterator into two.
It extends vcvtps2udq,vpbroadcastmb2d patterns as well.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
On Fri, Aug 15, 2014 at 1:47 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello,
This patch extends pattern for vpbroadcastmb2q insn
pattern.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
On Fri, Aug 15, 2014 at 1:52 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello,
This patch introduces new patterns to support
AVX-512Vl,DQ broadcast insns.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
On Thu, Aug 14, 2014 at 11:55 PM, Yuri Gribov tetra2...@gmail.com wrote:
On Thu, Aug 14, 2014 at 8:53 PM, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
In order for your work to be generally useful, I'd ask several things:
- Update
On Fri, Aug 15, 2014 at 1:56 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Hello,
This patch extends shift pattern to support AVX-512
new insn.
Bootstrapped.
New tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
Yeah, the 2MB was just a guess, I'll change it to 64k blocks. Note
the original code exponentially increased block size to not have
too many blocks (for whatever reason). A 800MB compressed decl section
would need 12800 64k blocks. But in the end it matters only that
the block
... in practice, something like the below.
Paolo.
///
Index: cp/pt.c
===
--- cp/pt.c (revision 214027)
+++ cp/pt.c (working copy)
@@ -162,7 +162,7 @@ static tree tsubst_friend_class (tree, tree);
static int
On 08/15/2014 03:16 PM, Paolo Carlini wrote:
+ bool in_function = (TREE_TYPE (tparms)
+ DECL_FUNCTION_TEMPLATE_P (TREE_TYPE (tparms)));
Huh? There's no such thing as a template parameter of function type.
Jason
Hi,
On 08/15/2014 09:22 PM, Jason Merrill wrote:
On 08/15/2014 03:16 PM, Paolo Carlini wrote:
+ bool in_function = (TREE_TYPE (tparms)
+ DECL_FUNCTION_TEMPLATE_P (TREE_TYPE (tparms)));
Huh? There's no such thing as a template parameter of function type.
Works fine, in fact, I
Thanks for the review.
--
Evandro Menezes Austin, USA
e.mene...@samsung.com +1-512-425-3365
-Original Message-
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On Behalf Of James Greenhalgh
Sent: Friday, August 15,
2014-08-15 21:32 GMT+04:00 enh e...@google.com:
can you file bugs against bionic for stuff like this? use
b.android.com (and feel free to mail me to ensure that they get
noticed).
Sure, I will do that.
one thing we'd like to do is get to a point where we're building
gcc/gdb et cetera
On August 15, 2014 7:25:55 PM CEST, Jason Merrill ja...@redhat.com wrote:
On 08/14/2014 12:28 PM, Jason Merrill wrote:
On 08/14/2014 05:07 AM, Richard Biener wrote:
So - can you take over this C++ frontend issue?
OK.
Here's what I'm applying to trunk:
Thanks Jason.
Richard.
On Fri, Aug 15, 2014 at 1:05 PM, Alexander Ivchenko aivch...@gmail.com wrote:
2014-08-15 21:32 GMT+04:00 enh e...@google.com:
can you file bugs against bionic for stuff like this? use
b.android.com (and feel free to mail me to ensure that they get
noticed).
Sure, I will do that.
one thing
This test is failing with LTO because in the LTRANS phase (DCE) we
realize that the call to f() is useless, so we don't generate it. This
leads to an uncalled f() which also gets deleted. We end up with an
empty main(), and rightly so, gdb has nothing good to print.
Marking `a1' as used
On Fri, 2014-08-15 at 22:58 +0200, Oleg Endo wrote:
On Mon, 2014-08-04 at 08:19 +0200, Oleg Endo wrote:
On Aug 4, 2014, at 6:00 AM, Gerald Pfeifer ger...@pfeifer.com wrote:
On Wed, 23 Jul 2014, Hans-Peter Nilsson wrote:
The page https://gcc.gnu.org/simtest-howto.html is
Hi,
Testing 'mpx' branch after merge with trunk I got a segfault in ipa-devirt.c.
It appears that cgraph_node cloning with indirect edge causes call to
get_polymorphic_call_info which uses current_function_decl. It happens in
IPA pass and therefore current_function_decl is NULL which
Hi,
I get a segafult in decl_maybe_in_construction_p during function versioning.
We have following steps in clone creation (e.g. as in
create_version_clone_with_body):
1. Create function decl
2. Create clone of cgraph node
3. Copy function body
After the first step there is no body
Hello world,
I committed the attached patch as obvious to trunk after regression
testing. It fixes the regression in the test case by adding
a NULL check to a pointer. Will commit to 4.9 soon.
Regards
Thomas
2014-08-15 Thomas Koenig tkoe...@gcc.gnu.org
PR fortran/62142
On 08/06/14 11:22, David Malcolm wrote:
gcc/
* target.def (reorder): Strengthen param ready of this DEFHOOK
from rtx * to rtx_insn **.
(reorder2): Likewise.
(dependencies_evaluation_hook): Strengthen params head, tail
from rtx to rtx_insn *.
*
This patch fixes two minor issues
a) The argument issue mentioned in
https://gcc.gnu.org/ml/fortran/2014-08/msg7.html
The main issue is that the decl uses void as argument; the FE passes
IARG() alias gfc_array_index_type while the library expects a
GFC_INTEGER_4. As n_dim and ts-kind are
On 08/06/14 11:22, David Malcolm wrote:
This patch updates params of 22 of the target hooks to pass an rtx_insn *
rather than an rtx, as appropriate.
Known to compile on:
alpha
arc
arm
bfin
c6x
epiphany
ia64
m32c
m32r
m68k
mep
microblaze
mips
pa
pdp11
picochip
rs6000
s390
sh
sparc
spu
tilegx
On 08/06/14 11:23, David Malcolm wrote:
gcc/
* final.c (get_attr_length_1): Replace GET_CODE check with a
dyn_cast, introducing local seq and the use of methods of
rtx_sequence.
(shorten_branches): Likewise, introducing local body_seq.
Strengthen local
On 08/06/14 11:23, David Malcolm wrote:
gcc/
* function.c (contains): Introduce local seq for PATTERN (insn),
with a checked cast, in the region for where we know it's a
SEQUENCE. Use methods of rtx_sequence.
OK. As is #206.
Jeff
On 08/06/14 11:23, David Malcolm wrote:
gcc/
* reorg.c (redundant_insn): In two places in the function, replace
a check of GET_CODE with a dyn_cast, introducing local seq, and
usings methods of rtx_sequence to clarify the code.
Some concerns here with the dynamic cast.
On 06/30/2014 02:49 PM, Jason Merrill wrote:
decl_mangling_context was failing to recognize a lambda in template
context as a lambda.
It turns out that was far from the only issue with a lambda in a member
template
Tested x86_64-pc-linux-gnu, applying to trunk.
commit
The runtime_gc function in libgo invokes the garbage collector proper on
the g0 thread (a thread with a large stack that is not involved in
scheduling). This is done via runtime_mcall. Upon return from
runtime_mcall, the caller may be running on a different thread.
Unfortunately, the runtime_gc
On Fri, Aug 15, 2014 at 04:24:49PM -0600, Jeff Law wrote:
On 08/06/14 11:23, David Malcolm wrote:
gcc/
* final.c (get_attr_length_1): Replace GET_CODE check with a
dyn_cast, introducing local seq and the use of methods of
rtx_sequence.
(shorten_branches): Likewise,
Hi,
The attached patch updates the SH options documentation.
Tested with 'make info dvi pdf'. Committed to trunk, 4.9 and 4.8
branches.
Cheers,
Oleg
gcc/ChangeLog:
* doc/invoke.texi (SH options): Document missing processor variant
options. Remove references to Hitachi.
On Thu, 2014-08-14 at 23:30 -0600, Jeff Law wrote:
On 08/06/14 11:22, David Malcolm wrote:
/
* rtx-classes-status.txt: Update.
---
rtx-classes-status.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/rtx-classes-status.txt b/rtx-classes-status.txt
On Fri, 2014-08-15 at 16:03 -0600, Jeff Law wrote:
On 08/06/14 11:22, David Malcolm wrote:
gcc/
* target.def (reorder): Strengthen param ready of this DEFHOOK
from rtx * to rtx_insn **.
(reorder2): Likewise.
(dependencies_evaluation_hook): Strengthen params head, tail
This adds a new output modifier e that prints an 's' for things like
xoris, and changes u to work for both xoris and xori. With that, both
SI and DI can simply use an n constraint, where previously they needed
K,L resp. K,J (and it used JF in fact, but the F doesn't do anything
there).
1 - 100 of 108 matches
Mail list logo
