On 31 Aug 2013, at 22:42, Gabriel Dos Reis wrote:
> Eric Botcazou writes:
>
> | > This patch fixes that by introducing GXX_LINK which is GCC_LINK except
> | > that CXX (e.g. xg++) instead of CC (e.g. xgcc) is invoked.
> | >
> | > Eric, are there other executables that need to be linked with GX
Hi Kirill,
On 23 Aug 2013, at 08:35, Kirill Yukhin wrote:
> On 22 Aug 12:06, Richard Henderson wrote:
>> Ok.
>
> I've updated ChangeLog (thanks, HJ!) and
> checked in to main trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-08/msg00545.html
This patch [actually the change at 201915] also broke X86 Da
On Sat, Aug 31, 2013 at 04:04:03PM +0100, Iain Sandoe wrote:
> OK for trunk?
Ok with the suggested s/sanitise/sanitize/g change.
> gcc:
> * config/darwin.h (LINK_COMMAND_SPEC_A): Revise sanitiser specs to
> include sanitise(undefined).
>
> libsanitiser:
>
> * ubsan/Makefile.am
On Sat, Aug 31, 2013 at 05:15:55PM +0200, Marek Polacek wrote:
> I see, sorry. Will commit the following fix as obvious in a bit.
>
> 2013-08-31 Marek Polacek
>
> * ubsan.c: Include tm_p.h.
You need to add $(TM_P_H) to ubsan.o dependencies in Makefile.in too.
Ok with that change.
> --
On Sun, Sep 01, 2013 at 01:36:37PM +0200, Jakub Jelinek wrote:
> On Sat, Aug 31, 2013 at 05:15:55PM +0200, Marek Polacek wrote:
> > I see, sorry. Will commit the following fix as obvious in a bit.
> >
> > 2013-08-31 Marek Polacek
> >
> > * ubsan.c: Include tm_p.h.
>
> You need to add $(T
On Fri, 30 Aug 2013 11:47:21, Richard Biener wrote:
> On Tue, Jul 2, 2013 at 7:33 PM, DJ Delorie wrote:
>>
>>> The choice appears to be to continue to have broken volatile bitfields
>>> on ARM with no way for users to make them conform to the ABI, or to
>>> change things so that they conform to th
Hi,
this patch implement speculative devirtualization. It is a trivial pass that
asks for targets of every polymorphic call in a program and if the list
contains one likely target, it produces an speculative call. No context
sensitive analysis is done at the moment. This call may or may not survi
Missing test cases?
Have you tested the optimization with SPEC2k and SPEC06? There are a
couple of benchmarks benefit greatly from devirtualization, such as
eon, povray etc. I believe astar will probably improve with this
optimization at O2 (it has hot virtual functions that are not
overridden a
> The test still fails with the following change:
>
> --- /opt/gcc/_clean/gcc/testsuite/gnat.dg/specs/linker_alias.ads2013-08-18
> 17:39:22.0 +0200 +++
> /opt/gcc/work/gcc/testsuite/gnat.dg/specs/linker_alias.ads2013-08-19
> 08:52:35.0 +0200 @@ -5,5 +5,6 @@ package Linker_Alias
> Missing test cases?
Good point. The testcases I have needs rest of the patches from the series to
hit the mainline. For now I have added the following to test the basic scenario
* g++.dg/ipa/devirt-15.C: New testcase.
Index: g++.dg/ipa/devirt-15.C
==
Hello,
> This patch [actually the change at 201915] also broke X86 Darwin
> bootstrap/ABI: pr59269
> - ISTM that SSE_REGNO_P() now returns true for a different set of registers
> than before the patch,
> I've attached a starting-point to fix to the PR - but would welcome any
> additional inputs
> It seems the patch needs a couple of minor amendments to work with Darwin -
> and I've added an updated version to the PR which passes bootstrap and make
> check-ada on x86_64-darwin12. Iain
Thanks, here is the final patch I just installed.
2013-09-01 Eric Botcazou
Iain Sandoe
Hi Janne,
I have tried to answer your points in the attached patch.
> The unformatted sequential format part looks Ok, but the special files
> section is lacking. E.g. what about
>
> - The REWIND statement?
Not supported.
> - The ENDFILE statement?
Not supported.
> - ACCESS='stream' and the
Hi,
this patch resolves the bug by implementing the resolution of DR 565
(DRWP): for template functions, in order to establish that the program
is ill-formed we want to check return type and template parameter list
too. Thus, in practice, we want to accept the reduced testcase which
came with
This pattern fixes the following ICE found compiling ArabicShaping.cpp:
g++-4.7 -D_REENTRANT -I. -I./.. -I../common -I./unicode -
DU_ATTRIBUTE_DEPRECATED= -DU_LAYOUT_IMPLEMENTATION -D__NO_MATH_INLINES
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -DNDEBUG -
D_FORTIFY_SOURCE=2 -DGCC_HIDDEN_VISIBI
Eric Botcazou writes:
| > It seems the patch needs a couple of minor amendments to work with Darwin -
| > and I've added an updated version to the PR which passes bootstrap and make
| > check-ada on x86_64-darwin12. Iain
|
| Thanks, here is the final patch I just installed.
Thanks!
-- Gaby
Hi,
once simple pass has grown up into set of not-so-trivial profile based
transformations.
This patch merely moves it to separate file. I intend to add toplevel comment
and fix
interesting problem Martin Liska noticed in followup patch.
I did some basic housekeeping of include files and depend
.. consider this withdrawn, I have to fix another very similar check
elsewhere.
Paolo.
Mike Stump writes:
> @@ -643,12 +653,14 @@ equality. */
> __FUNCTION__); \
> &_rtx->u.hwint[_n]; }))
>
> -#define XHWIVEC_ELT(HWIVEC, I) __extension__ \
> -(*({ __typeof (HWIVEC) const _hwivec = (HWIVEC); const
On 08/19/2013 10:01 AM, Jan Hubicka wrote:
+ /* All equivalent types, if more than one. */
+ vec *types;
+ /* Set of all equivalent types, if NON-NULL. */
+ pointer_set_t * GTY((skip)) types_set;
Why do you need both a vector and a pointer set? Can't you drop the
vector and use poin
> On 08/19/2013 10:01 AM, Jan Hubicka wrote:
> >+ /* All equivalent types, if more than one. */
> >+ vec *types;
> >+ /* Set of all equivalent types, if NON-NULL. */
> >+ pointer_set_t * GTY((skip)) types_set;
>
> Why do you need both a vector and a pointer set? Can't you drop the
> vec
On 08/27/2013 03:42 PM, Adam Butcher wrote:
Unfortunately, due to errors being thrown 'early' in grokdeclarator, I
haven't been able to get 'auto...' (or reference/qualified variants)
working yet. I think I need to defer processing the parameter pack
internals of grokdeclarator until I have the
On 08/27/2013 03:42 PM, Adam Butcher wrote:
+ vec_safe_push (argvec, arg);
I bet we want convert_from_reference in the non-generic lambda case, too.
OK with that change.
Jason
On 08/27/2013 03:42 PM, Adam Butcher wrote:
+ if (FUNCTION_PARAMETER_PACK_P (src))
+ return;
Yeah, in the case of a parameter pack we want to pass a pack expansion.
Jason
On 08/27/2013 03:42 PM, Adam Butcher wrote:
+ else // extend current template parameter list
+ // pop the innermost template parms into tparms
Most comments should start with a capital letter and end with a period.
+ for (size_t n = 0, end = TREE_VEC_LENGTH (inner_vec); n < end; ++
On 09/01/2013 03:50 PM, Jan Hubicka wrote:
I was mostly worried about ordering issues. pointer_set is not stable
across memory layout changes and we may end up outputting warnings/merging
binfos in random orders. This seemed uncool. Maybe the final form is quite
safe, since we do everything with
Hi again,
... thus Take 2: the extended comparison per DR 565 must be used in
push_overloaded_decl_1 too, in order to handle correctly the case of an
using decl *followed* by a decl. I added a testcase too.
Again booted and tested x86_64-linux.
Thanks,
Paolo.
//
/cp
2013
On 30-Aug-13, at 6:38 AM, Eric Botcazou wrote:
Let's avoid trying to do something general since this seems to be
really a
corner case. Can't we simply deal with hard registers specially?
/* Move floating point as parts if splitting is easy. */
if (GET_MODE_CLASS (mode) == MODE_COMPLEX_FLO
My main discomfort is the double-int part. At this point in time the
only usage of double-int left on the branch is because the fixed ints
use it as their underlying representation. I think that rather than
doing a lot of work to accommodate this, it would be better to just
change the fixed r
On Sep 1, 2013, at 7:37 PM, Kenneth Zadeck wrote:
> On 09/01/2013 03:21 PM, Richard Sandiford wrote:
>>
>> * As Richard requested, the main wide int class is parameterised by the
>> storage:
>>
>> template
>> class GTY(()) generic_wide_int : public storage
Careful of GTY, it gets fin
OK.
Jason
Hi,
The gimple-ssa-strength-reduction pass handles CAND_REFs in order to find
different MEM_REFs sharing common part in addressing expression. If such
MEM_REFs are found, the pass rewrites MEM_REFs, and produces more efficient
addressing expression during the RTL passes.
The pass analyzes address
32 matches
Mail list logo
