Hi,
this patch moves DECL_ARGUMENTS into FUNCTION_DECL. For this I needed to solve
reuse in C++ and objC frontends. In C++ I think I actually run out of places
to use for TEMPLATE_DECL (well, perhaps DECL_ATTRIBUTES but I would definitely
run out of pointers fro DECL_RESULT I want to move next) s
> Hi!
>
> Fallout for frv, see eg.
> http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272377
>
> g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
> -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wwrite-strings -Wcast-qual
> -Wmissing-format-attribute -Wove
> Hi!
>
> This is the xtensa fallout, see
> http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272418
>
> g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
> -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing
> -Wwrite-strings -Wcast-qual -Wmissing-
> Hi!
>
> I've been away for holidays and other stuff, so this is a bit late,
> but nevermind... The stringification of DECL_SECTION_NAME had some
> fallout, see eg.
> http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272454
>
> g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE
> On 06/24/2014 02:18 PM, Jan Hubicka wrote:
> >>On 06/23/2014 04:25 PM, Jan Hubicka wrote:
> >>> * class.c (check_methods, create_vtable_ptr, determine_key_method,
> >>> add_vcall_offset_vtbl_entries_1): Guard VINDEX checks by FUNCTION_DECL
> >>> check.
> >>
> >>These changes are unnecessary:
OK. Thank you very much for your review, Richard!
thanks,
Cong
On Tue, Jun 24, 2014 at 4:19 AM, Richard Biener
wrote:
> On Tue, Dec 3, 2013 at 2:06 AM, Cong Hou wrote:
>> Hi Richard
>>
>> Could you please take a look at this patch and see if it is ready for
>> the trunk? The patch is pasted as
Hi!
Fallout for frv, see eg.
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272377
g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
-fno-rtti -fasynchronous-unwind-tables -W -Wall -Wwrite-strings -Wcast-qual
-Wmissing-format-attribute -Woverloaded-virtual
Hi!
This is the xtensa fallout, see
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272418
g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
-fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings
-Wcast-qual -Wmissing-format-attribute
Hi!
I've been away for holidays and other stuff, so this is a bit late,
but nevermind... The stringification of DECL_SECTION_NAME had some
fallout, see eg.
http://toolchain.lug-owl.de/buildbot/show_build_details.php?id=272454
g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions
On Tue, Jun 24, 2014 at 11:06:33PM +0200, Kai Tietz wrote:
> Hi Trevor,
>
> your recent commits have broken bootstrap for cygwin/mingw i386 targets with:
should be fixed in r211962. Sorry I didn't realize the const
qualification thing matter in other places, it didn't show up in core
gcc so I di
This patch from Peter Collingbourne adds some benchmark input files that
were missing from the gofrontend repository. Bootstrapped and ran Go
testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
The patch adds binary files. The size of the patch is too large to
attach, and the files ar
Part 4 of 4, split from the full patch. This is the MSP430-specific
use of the new intN framework to enable true 20-bit pointers. Since
I'm one of the MSP430 maintainers, this patch is being posted for
reference, not for approval.
gcc/config/msp430
* config/msp430/msp430-protos.h (msp4
Part 3 of 4, split from the full patch. Additional optimization
opportunity, since the MSP430 does a lot of conversions between HImode
and PSImode.
gcc/
* expr.c (convert_move): If the target has an explicit converter,
use it.
Index: gcc/expr.c
==
Part 1 of 4, split from the full patch. The purpose of this set of
changes is to remove assumptions in GCC about type sizes. Previous to
this patch, GCC assumed that all types were powers-of-two in size, and
used naive math accordingly.
Old:
POINTER_SIZE / BITS_PER_UNIT
TYPE_SIZ
On 2014-06-24 23:22, Paolo Carlini wrote:
On 06/24/2014 01:40 AM, Adam Butcher wrote:
+// { dg-do compile { target c++1y } }
I don't think this is a C++1y specific issue...
You're right. I'm so used to creating pr testcases in that
g++.dg/cpp1y dir, I automatically added the test there with
Hi,
On 06/24/2014 01:40 AM, Adam Butcher wrote:
+++ b/gcc/testsuite/g++.dg/cpp1y/pr61537.C
@@ -0,0 +1,24 @@
+// PR c++/61537
+// { dg-do compile { target c++1y } }
I don't think this is a C++1y specific issue...
+// { dg-options "" }
Also, likely minor detail, could you please explain why you
Matthew Fortune writes:
>> I suppose we'll need a way of specifying an isa_rev range, say
>> "isa_rev=2-5". That should be a fairly localised change though.
>
> There appear to be about 9 tests that are not fixed by educating mips.exp
> about flags which are not supported on R6. Steve has initial
OK for google-4_8 and google-4_9
Thanks,
Dehao
On Tue, Jun 24, 2014 at 3:09 PM, Yi Yang wrote:
> Hi,
>
> This patch removes unnecessary edge probability calculations in
> afdo_propagate_circuit() that would eventually be overridden by
> afdo_calculate_branch_prob().
>
> This would pave the way f
... in order to handle correctly in C++98 mode (*) the case of
references in a member, I think we have to add an explicit check of
CLASSTYPE_REF_FIELDS_NEED_INIT, per the below.
Thanks,
Paolo.
(*) For C++11 my previous patch is fine, the TREE_CODE (TREE_TYPE
(field)) == REFERENCE_TYPE check c
Hi,
This patch removes unnecessary edge probability calculations in
afdo_propagate_circuit() that would eventually be overridden by
afdo_calculate_branch_prob().
This would pave the way for my next patch, which compares the
estimated branch probabilities or the branch annotations against the
real
Hi,
this remained unresolved for a long time, but, if I understand correctly
Jason' Comment 1, should be rather easy, just do not complain for
uninitialized const members in aggregates, recursively too (per struct B
in the testcases). Does the below makes sense, then?!? It passes
testing, any
> I think that was on purpose to avoid arithmetics in enum types. As those
> conversions are useless and thus stripped later is it really important
> to retain enum and boolean kind here?
The problem is that convert.c is called by front-ends and the patch also
removed the callback into them that
Please don't ignore this, support for MIPS r5900 was already ignored for ~10
years. Just apply the patch when you don't know what I am talking about. The
fix for mips16 is very important. The patch has impact only on MIPS r5900.
> Gesendet: Sonntag, 15. Juni 2014 um 22:08 Uhr
> Von: "Jürgen Urba
* parser.c (cp_parser_elaborated_type_specifier): Only consider template
parameter lists outside of function parameter scope.
* g++.dg/cpp1y/pr61537.C: New testcase.
---
gcc/cp/parser.c | 28 +++-
gcc/testsuite/g++.dg/cpp1y/pr61
Hi Trevor,
your recent commits have broken bootstrap for cygwin/mingw i386 targets with:
../../gcc/gcc/config/i386/winnt.c: In Funktion »unsigned int
i386_pe_section_type_flags(tree, const char*, int)«:
../../gcc/gcc/config/i386/winnt.c:503:61: Fehler: keine passende
Funktion für Aufruf von »hash
On 06/24/2014 02:18 PM, Jan Hubicka wrote:
On 06/23/2014 04:25 PM, Jan Hubicka wrote:
* class.c (check_methods, create_vtable_ptr, determine_key_method,
add_vcall_offset_vtbl_entries_1): Guard VINDEX checks by FUNCTION_DECL
check.
These changes are unnecessary: TYPE_METHODS is
On Tue, Jun 24, 2014 at 10:26:27PM +0200, FX wrote:
> >> 3. Does the attached updated patch (libgfortran only, without
> >> regenerated files) fix the problem?
> >
> > I'll test it when my regtesting is completed. But, a scan of
> > the configure.host re-arrangement suggests that it should work.
On 06/13/14 04:44, mliska wrote:
Hello,
this is core of IPA ICF patchset. It adds new pass and registers all needed
stuff related to a newly introduced interprocedural optimization.
Algorithm description:
In LGEN, we visit all read-only variables and functions. For each symbol, a
hash v
>> 3. Does the attached updated patch (libgfortran only, without
>> regenerated files) fix the problem?
>
> I'll test it when my regtesting is completed. But, a scan of
> the configure.host re-arrangement suggests that it should work.
OK.
If you have some spare cycles, could you then also check
On Tue, Jun 24, 2014 at 09:43:06PM +0200, FX wrote:
>
> Thanks for testing on a platform I don?t have access to! I try to
> answer to the three main points below:
>
> > I suppose I don't understand the logic in libgfortran/configure.host.
> > It is picking the wrong config/fpu*.h file.
>
> 1. Th
On Mon, Jun 23, 2014 at 03:35:01PM +0200, Bernd Edlinger wrote:
> Hi Martin,
>
> >>
> >> Well actually, I am not sure if we ever wanted to have a race condition
> >> here.
> >> Have you seen any impact of --param allow-store-data-races on any
> >> benchmark?
> >
> > It's trivially to write one.
Sandra Loosemore writes:
> On 05/14/2014 12:49 PM, Richard Sandiford wrote:
>> Jeff Law writes:
>>> On 05/13/14 14:11, Sandra Loosemore wrote:
2014-05-13 Catherine Moore
Sandra Loosemore
gcc/
* config/mips/mips.c (mips_order_regs_for_local_
Am 24.06.2014 22:05, schrieb Paul Gortmaker:
> We see the following on a 32bit gcc installed on 64 bit host:
>
> Reading symbols from ./i586-pokymllib32-linux-gcc...done.
> (gdb) run
> Starting program:
> x86-pokymllib32-linux/lib32-gcc/4.9.0-r0/image/usr/bin/i586-pokymllib32-linux-gcc
>
>
We see the following on a 32bit gcc installed on 64 bit host:
Reading symbols from ./i586-pokymllib32-linux-gcc...done.
(gdb) run
Starting program:
x86-pokymllib32-linux/lib32-gcc/4.9.0-r0/image/usr/bin/i586-pokymllib32-linux-gcc
Program received signal SIGSEGV, Segmentation fault.
0xf
Hi Honza,
On 23 Jun 2014, at 18:36, Jan Hubicka wrote:
>> The tests gcc.dg/globalalias-2.c and gcc.dg/localalias-2.c fail on darwin
>> with
>>
>> /opt/gcc/work/gcc/testsuite/gcc.dg/globalalias-2.c:20:2: warning: alias
>> definitions not supported in Mach-O; ignored
>>
>> I think they should b
On June 24, 2014 9:16:34 PM CEST, Trevor Saunders wrote:
>On Tue, Jun 24, 2014 at 08:23:49PM +0200, Jan Hubicka wrote:
>> >
>> > On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
>> > >From: Trevor Saunders
>> > >
>> > >Hi,
>> > >
>> > >This patch adds a hash_map class so we can consolidate t
On 06/24/14, Andrew Sutton wrote:
Weird. Any chance you're doing a bootstrap build?
There was an earlier bootstrapping issue with this branch. We had
turned on -std=c++1y by default, and it was causing some conversion
errors with lvalue references to bitfields in libasan.
This doesn't *look
On 06/20/14 01:51, Jan Hubicka wrote:
Hi,
IRA initialization shows high in profiles even when building lto objects. This
patch simply
delays RTL backend initialization until we really decide to output a function.
In some cases
this avoids the initialization completely (like in the case of LTO
On Tue, Jun 24, 2014 at 08:34:23PM +0200, Tobias Burnus wrote:
> Steve Kargl wrote:
> > On FreeBSD (and perhaps other *BSD), there is no fpsetsticky(). The
> > function is fpresetsticky().
>
> Solaris has fpsetsticky() (requires ieeefp.h) and BSD has
> fpresetsticky() ? thus, like at other plac
On Tue, Jun 24, 2014 at 08:23:49PM +0200, Jan Hubicka wrote:
> >
> > On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
> > >From: Trevor Saunders
> > >
> > >Hi,
> > >
> > >This patch adds a hash_map class so we can consolidate the boiler plate
> > >around
> > >using hash_table as a map, it al
> In preparation to fix the alias issues on AIX, this patch changes
> ASM_DECLARE_FUNCTION_NAME from a macro to a function.
Thanks, David!
We will also need to introduce ASM_DECLARE_OBJECT_NAME to handle the aliases
of variables. I can look into that probably later this week (it is last week
of
In preparation to fix the alias issues on AIX, this patch changes
ASM_DECLARE_FUNCTION_NAME from a macro to a function.
Bootstrap on powerpc-ibm-aix7.1.0.0 in progress.
* config/rs6000/xcoff.h (ASM_DECLARE_FUNCTION_NAME): Remove definition
and call...
* config/rs6000/rs600
Steve Kargl wrote:
On FreeBSD (and perhaps other *BSD), there is no fpsetsticky(). The function
is fpresetsticky().
Solaris has fpsetsticky() (requires ieeefp.h) and BSD has
fpresetsticky() – thus, like at other places in that file, one needs to
conditionally enable one or the other.
Tobi
> > The problem is that the patch fails testcases that assume we do such
> > folding at parsing
> > time.
> >
> > ./testsuite/gcc/gcc.sum:FAIL: gcc.dg/pr36901-1.c (test for excess errors)
> > ./testsuite/gcc/gcc.sum:FAIL: gcc.dg/pr36901-2.c (test for excess errors)
> > ./testsuite/gcc/gcc.sum:FAI
>
> On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
> >From: Trevor Saunders
> >
> >Hi,
> >
> >This patch adds a hash_map class so we can consolidate the boiler plate
> >around
> >using hash_table as a map, it also allows us to get rid of pointer_map which
> >I
> >do in this patch by conve
> Hello,
>this patch changes IPA REF API to c++ style. Changes were suggested and
> consulted with Honza.
>
> Patch has been pre approved, will be committed if no comments.
> Bootstrapped on x86_64-pc-linux-gnu, no regressions.
>
> Thanks,
> Martin
>
> ChangeLog:
> 2014-06-22 Martin Liska
> On 06/23/2014 04:25 PM, Jan Hubicka wrote:
> > * class.c (check_methods, create_vtable_ptr, determine_key_method,
> > add_vcall_offset_vtbl_entries_1): Guard VINDEX checks by FUNCTION_DECL
> > check.
>
> These changes are unnecessary: TYPE_METHODS is a list of functions.
I just double
On Tue, Jun 24, 2014 at 11:12:38AM -0600, Tom Tromey wrote:
> Trevor> hrm, I know basically nothing about the upcoming changes, but I would
> Trevor> have expected linking c++03 code against c++11 code would be fine
> Trevor> especially when the interface doesn't involve any stl.
>
> https:
On Tue, Jun 24, 2014 at 09:49:36AM -0700, Steve Kargl wrote:
>
> Not yet.
>
> On i386-*-freebsd
>
> In file included from ../../../gcc4x/libgfortran/runtime/fpu.c:29:0:
> ./fpu-target.h: In function 'set_fpu_trap_exceptions':
> ./fpu-target.h:31:3: error: unknown type name 'fp_except'
>fp_ex
Trevor> hrm, I know basically nothing about the upcoming changes, but I would
Trevor> have expected linking c++03 code against c++11 code would be fine
Trevor> especially when the interface doesn't involve any stl.
https://gcc.gnu.org/wiki/Cxx11AbiCompatibility
This warns against mixing wi
This patch avoids multiple definitions of def_stmt in different
blocks, and moves it at
the beginning of gimple_match_and_simplify.
* genmatch.c (decision_tree::gen_gimple): Call fprintf to generate
definition of def_stmt.
(dt_operand::gen_gimple_expr): Adjust call to fprintf to generate
assign
On Tue, Jun 24, 2014 at 10:11:32AM +0200, FX wrote:
> Here?s a patch fixing the diff issue with configure.host and the doc (which
> apparently is only triggered by some versions of texinfo).
> Apart from that, functionnaly identical, so I?ll paste here the ?history? of
> the patch:
>
> -
Weird. Any chance you're doing a bootstrap build?
There was an earlier bootstrapping issue with this branch. We had
turned on -std=c++1y by default, and it was causing some conversion
errors with lvalue references to bitfields in libasan.
This doesn't *look* like a regression caused by concepts
On Tue, Jun 24, 2014 at 9:00 PM, Richard Biener wrote:
>
> This massages things so GENERIC code-gen works (well, is emitted
> and compiles). The GENERIC interface matches that of
> fold_{unary,binary,ternary} with also supporting calls here.
> It's supposed to be called at the start of those func
This fixes some warnings when building the library with -Wpedantic
(some only show up when also building with -Wsystem-headers).
Tested x86_64-linux, committed to trunk.
commit bf385e3ea2d8d10bdaa8e41c7638d32eb26b9bfd
Author: Jonathan Wakely
Date: Tue Jun 24 14:16:28 2014 +0100
* include
I'm not sure the warning is correct in any case...
In i386.h
struct stringop_algs
{
const enum stringop_alg unknown_size;
const struct stringop_strategy {
const int max;
const enum stringop_alg alg;
int noalign;
} size [MAX_STRINGOP_ALGS];
};
in i386.c
---
> On Jun 24, 2014, at 2:08 AM, Kyrill Tkachov wrote:
>
>
>> On 23/06/14 22:12, Andrew Pinski wrote:
>> Hi,
>> When looking at the current conditional compare patch, I find that
>> we don't have a testcase to test that we don't ICE for the case where
>> we have conditional compares and condit
This massages things so GENERIC code-gen works (well, is emitted
and compiles). The GENERIC interface matches that of
fold_{unary,binary,ternary} with also supporting calls here.
It's supposed to be called at the start of those functions
(and if it returns NULL_TREE their remains is executed).
ok.
David
On Tue, Jun 24, 2014 at 5:50 AM, Teresa Johnson wrote:
> Emit the proper module name in LIPO profile-use mode.
>
> Passes regression tests, ok for google branches?
>
> Thanks,
> Teresa
>
> 2014-06-24 Teresa Johnson
>
> * coverage.c (emit_function_name): Emit module name in L
I am still having problems doing a full build.
I get stuck on something that I think can't be a concepts problem in
gcc/config/i386/i386.c:
make[3]: Entering directory `/home/ed/obj_concepts/gcc'
/home/ed/obj_concepts/./prev-gcc/xg++
-B/home/ed/obj_concepts/./prev-gcc/
-B/home/ed/bin_concept
Actually allow member concepts in using shorthand notation.
2014-06-24 Andrew Sutton
* gcc/cp/parser.c (cp_maybe_constrained_type_specifier): Defer
handling the BASELINK check until concept-resolution in order to
allow member conceps.
(cp_parser_nonclass_name): A
On 06/23/2014 04:25 PM, Jan Hubicka wrote:
* class.c (check_methods, create_vtable_ptr, determine_key_method,
add_vcall_offset_vtbl_entries_1): Guard VINDEX checks by FUNCTION_DECL
check.
These changes are unnecessary: TYPE_METHODS is a list of functions.
The rest of the patch
On Mon, Jun 23, 2014 at 11:51 PM, Uros Bizjak wrote:
> On Mon, Jun 23, 2014 at 6:29 PM, H.J. Lu wrote:
>
> > --- gcc/config/i386/driver-i386.c.jj2014-05-14 14:45:54.0
> > +0200
> > +++ gcc/config/i386/driver-i386.c 2014-06-20 18:59:57.805006358
> > +0200
>
This helps improve debug output. When pretty printing template args
including a placeholder, show instead of a dump_expr
error.
2014-06-24 Andrew Sutton
* gcc/cp/error.C (dump_expr): Pretty print placeholder to improve
debug output.
Committed as r211942.
Andrew
Index: gcc/cp/error.c
On Tue, Jun 24, 2014 at 12:54 PM, Eric Botcazou wrote:
> Hi,
>
> https://gcc.gnu.org/ml/gcc-patches/2012-03/msg00491.html changed the old
> signed_type_for/unsigned_type_for functions and made them always return an
> integer type, whereas they would previously leave integral types unchanged.
> I d
This re-orders operands so we can default-initialize them to zero
now that we only have a single autogenerated function.
Richard.
2014-06-24 Richard Biener
* genmatch.c (write_fn_prototype): Remove and inline...
(decision_tree::gen_gimple): ...here.
* gimple-match-hea
OK.
Jason
Hi,
in this old rejects-valid we reject:
struct s
{
typedef void f(void);
f operator();
};
at the beginning of grokdeclarator:
if (((dname && IDENTIFIER_OPNAME_P (dname)) || flags == TYPENAME_FLAG)
&& innermost_code != cdk_function
&& ! (ctype && !declspecs->any_specifiers_
This adds infrastructure to create generic-match.c (empty for now).
Richard.
2014-06-24 Richard Biener
* Makefile.in (OBJS): Add generic-match.o.
(.PRECIOUS): Likewise.
(MOSTLYCLEANFILES): Add generic-match.c.
(generic-match.c): New rule.
(s-match): Am
On Tue, Jun 24, 2014 at 1:29 PM, Prathamesh Kulkarni
wrote:
> This patch factors out checking expression-code in gimple-match-head.c
>
> * gimple-match-head.c (check_gimple_assign): New function.
> (check_gimple_assign_convert): Likewise.
> (check_gimple_call_builtin): Likewise.
>
> * genm
On Tue, Jun 24, 2014 at 03:17:46PM +0200, Martin Liška wrote:
>
> On 06/24/2014 02:40 PM, Trevor Saunders wrote:
> >On Tue, Jun 24, 2014 at 02:29:53PM +0200, Martin Liška wrote:
> >>On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
> >>>From: Trevor Saunders
> >>>
> >>>Hi,
> >>>
> >>>This patc
Bother.
Apologies.
Simplest fix is to unshare the test body - patch attached; ok for trunk?
gcc/testsuite/ChangeLog:
* gcc.target/arm/simd/vexts64_1.c: Remove #include, inline test body.
* gcc.target/arm/simd/vextu64_1.c: Likewise.
* gcc.target/aarch64/simd/ext_s64_1.c:
On 06/24/2014 02:40 PM, Trevor Saunders wrote:
On Tue, Jun 24, 2014 at 02:29:53PM +0200, Martin Liška wrote:
On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
From: Trevor Saunders
Hi,
This patch adds a hash_map class so we can consolidate the boiler plate around
using hash_table as a ma
Emit the proper module name in LIPO profile-use mode.
Passes regression tests, ok for google branches?
Thanks,
Teresa
2014-06-24 Teresa Johnson
* coverage.c (emit_function_name): Emit module name in LIPO mode.
Index: coverage.c
===
On Tue, Jun 24, 2014 at 02:29:53PM +0200, Martin Liška wrote:
>
> On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
> >From: Trevor Saunders
> >
> >Hi,
> >
> >This patch adds a hash_map class so we can consolidate the boiler plate
> >around
> >using hash_table as a map, it also allows us to g
Hi Richard,
I have here part 1 of this patch, which does not use undefined behaviour.
Currently I start to think that also TYPE_SATURATING just cannot happen
here, because only some targets have saturating types, for instance ARM,
and also for those who have it, it is only allowed with fixed poin
On 06/20/2014 12:52 PM, tsaund...@mozilla.com wrote:
From: Trevor Saunders
Hi,
This patch adds a hash_map class so we can consolidate the boiler plate around
using hash_table as a map, it also allows us to get rid of pointer_map which I
do in this patch by converting its users to hash_map.
Committed as r211935. I updated the patch to add a more appropriate
comment and changelog entry.
Andrew
On Tue, Jun 24, 2014 at 8:07 AM, Ed Smith-Rowland <3dw...@verizon.net> wrote:
> I saw this during bootstrap. I've verified that the patch works (I was
> working on similar).
>
> Ed
>
Index: pa
On Tue, Jun 24, 2014 at 09:53:35PM +1000, Kugan wrote:
> 2014-06-24 Kugan Vivekanandarajah
>
> * gcc/calls.c (precompute_arguments: Check is_promoted_for_type
> and set the promoted mode.
> (is_promoted_for_type) : New function.
> (expand_expr_real_1) : Check is_promoted
On Tue, Jun 24, 2014 at 09:51:51PM +1000, Kugan wrote:
> Changes the the SUBREG flags to be able to set promoted for sign
> (SRP_SIGNED), unsigned (SRP_UNSIGNED), sign and unsigned
> (SPR_SIGNED_AND_UNSIGNED) in SUBREG_PROMOTED_VAR_P.
> 2014-06-24 Kugan Vivekanandarajah
>
> * gcc/calls.
I saw this during bootstrap. I've verified that the patch works (I was
working on similar).
Ed
Sets proper flags on the SUBREG based on value
range info and enables elimination of zext/sext when possible.
Thanks,
Kugan
gcc/
2014-06-24 Kugan Vivekanandarajah
* gcc/calls.c (precompute_arguments: Check is_promoted_for_type
and set the promoted mode.
(is_promoted_f
Changes the the SUBREG flags to be able to set promoted for sign
(SRP_SIGNED), unsigned (SRP_UNSIGNED), sign and unsigned
(SPR_SIGNED_AND_UNSIGNED) in SUBREG_PROMOTED_VAR_P.
Thanks,
Kugan
gcc/
2014-06-24 Kugan Vivekanandarajah
* gcc/calls.c (precompute_arguments): Use new SUBREG_PRO
Hi,
This patch series (2) implements zext/sext extension elimination using
value ranges stored in SSA. Implementation is what was suggested in the
thread https://gcc.gnu.org/ml/gcc/2014-05/msg00213.html.
I have broken this into:
Patch 1 - Changes to store zero and sign extended promotions
(SPR_S
Braden,
Did you have a specific test case that causes this breakage? I have a
feeling that if we're missing base-link nodes in one place, we'll miss
them in others too.
Andrew
On Tue, Jun 17, 2014 at 4:54 AM, Braden Obrzut wrote:
> cp_maybe_constrained_type_specifier asserted that the decl pas
Scan-assembler test was running with "dg-do assemble" and not generating any
assembler to scan.
Tested on aarch64-none-elf and aarch64_be-none-elf; 40 * UNRESOLVED->PASS tests
in singleton_intrinsics_1.c
Patch (below) committed as r211934.
--Alan
--
Index: gcc/testsuite/gcc.target/
On Mon, Jun 23, 2014 at 11:49 AM, Bin Cheng wrote:
> Hi,
> For below simplified case:
>
> #define LEN (32000)
> __attribute__((aligned(16))) float a[LEN],b[LEN];
>
> int foo (int M)
> {
> for (int i = 0; i < M; i++)
> a[i+M] = a[i] + b[i];
> }
>
> Compiling it with command like:
> $ aarch64-
oops, didn't set plain text mode.
sorry for re-post.
On Tue, Jun 24, 2014 at 4:59 PM, Prathamesh Kulkarni
wrote:
> This patch factors out checking expression-code in gimple-match-head.c
>
> * gimple-match-head.c (check_gimple_assign): New function.
> (check_gimple_assign_convert): Likewise.
>
On Mon, 16 Jun 2014 12:42:36 +0100
Julian Brown wrote:
> Hi,
>
> As discussed several times previously, support for NEON in ARM
> big-endian mode is quite broken because of differing assumptions about
> lane ordering made by the ARM EABI and the set of NEON intrinsics on
> the one hand, and the
On Thu, 5 Jun 2014 20:23:27 +0100
Julian Brown wrote:
> Hi,
>
> The print-sysroot-suffix.sh script that can be used (via the
> t-sysroot-suffix makefile fragment) to auto-generate
> the SYSROOT_SUFFIX_SPEC macro for non-trivial multilib setups does not
> take into account the MULTILIB_REUSE targ
On Thu, 29 May 2014 11:16:52 +0100
Julian Brown wrote:
> On Thu, 19 Jul 2012 14:47:54 +0100
> Julian Brown wrote:
>
> > On Thu, 19 Jul 2012 13:54:57 +0100
> > Paul Brook wrote:
> >
> > > > But, that means EABI-conformant callers are also perfectly
> > > > entitled to sign-extend half-float va
On Fri, 9 May 2014 17:33:41 +0100
Julian Brown wrote:
> On Wed, 7 May 2014 09:41:27 -0600
> Tom Tromey wrote:
>
> > Tom> The usual approach is some appropriate text somewhere on the
> > Tom> GCC wiki (though I suppose a note in the mail archives would
> > Tom> do in a pinch) along with a URL in
On Tue, Dec 3, 2013 at 2:06 AM, Cong Hou wrote:
> Hi Richard
>
> Could you please take a look at this patch and see if it is ready for
> the trunk? The patch is pasted as a text file here again.
(found it)
The patch is ok for trunk. (please consider re-testing before you commit)
Thanks,
Richar
On Sat, May 3, 2014 at 2:39 AM, Cong Hou wrote:
> On Mon, Apr 28, 2014 at 4:04 AM, Richard Biener wrote:
>> On Thu, 24 Apr 2014, Cong Hou wrote:
>>
>>> Given the following loop:
>>>
>>> int a[N];
>>> short b[N*2];
>>>
>>> for (int i = 0; i < N; ++i)
>>> a[i] = b[i*2];
>>>
>>>
>>> After being ve
Hi,
https://gcc.gnu.org/ml/gcc-patches/2012-03/msg00491.html changed the old
signed_type_for/unsigned_type_for functions and made them always return an
integer type, whereas they would previously leave integral types unchanged.
I don't see any justification for the latter and this has the annoyi
Hi Richard,
Thanks for the review.
Please find attached updated patch after your review comments.
Changelog:
gcc/
* config/mips/mips.md (JOIN_MODE): New mode iterator.
(join2_load_Store): New pattern.
(join2_loadhi): Likewise.
(define_peehole2): Add peephole2 patte
On Mon, Jun 16, 2014 at 10:24:26AM +0100, Jan Hubicka wrote:
> > On Fri, Jun 13, 2014 at 12:14 AM, Jan Hubicka wrote:
> > > Hi,
> > > while updating vect_can_force_dr_alignment_p for section API I noticed the
> > > predicate is bit confused about when it can update the alignment.
> > >
> > > We ne
On Fri, Jun 20, 2014 at 11:33 AM, Eric Botcazou wrote:
> [I'm at last back to this...]
>
>> > With [1, -x + INF] as the resulting range? But it can be bogus if x is
>> > itself equal to +INF (unlike the input range [x + 1, +INF] which is
>> > always correct)
>>
>> Hmm, indeed.
>>
>> > so this doe
On Tue, Jun 24, 2014 at 10:53 AM, Prathamesh Kulkarni
wrote:
> Hi,
>This patch attempts to generate temporaries only when required.
>
> I have changed generation of operand names. All children of an expr-node
> are assigned at expr-node itself.
> Names are generated as follows at expr-node:
>
The following avoids to increase hard register lifetime by not
sinking loads from those.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
I wonder if it would make sense to not treat hardregs as
memory but instead rewrite them into SSA form but with
SSA_NAME_OCCURS_IN_ABNORMAL_PHI
1 - 100 of 110 matches
Mail list logo
