On Tue, 13 Nov 2012, Richard Henderson wrote:
> As discussed elsewhere. Tested on x86_64-linux.
> +2012-11-13 Richard Henderson
> +
> + * configure.ac: Move libsanitizer logic to subdirectory.
> + * configure: Regenerate.
> +
Thanks and sorry for copypasting the wrong pattern in the f
I've committed another installment of my copy-edits to extend.texi. For this
patch, I went through the table at
http://gcc.gnu.org/codingconventions.html#Spelling
and tried to check off everything I haven't already addressed by prior patches
in this series. I also took care of a few of the li
Based on advice from Dmitry Vyukov, this patch fixes the use of
__atomic_compare_exchange_n in libgo. The Go library assumes a strong
compare-exchange operation, and always assumes sequential consistency
around the operation. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu, where, a
On Fri, Nov 16, 2012 at 3:08 PM, Peter Bergner wrote:
> Attached below is an initial port of ASAN to powerpc*-linux.
> With the patch below along with Jakub's ASAN testsuite patch:
>
> http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01365.html
>
> ASAN not only builds, but seems to be working.
Out
On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo wrote:
> 2012-11-15 Diego Novillo
>
> Adjust for new vec API
> (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
>
> * c-common.c: Use new vec API in vec.h.
> * c-common.h: Likewise.
> * c-gimplify.c: Likewise.
>
On 11/16/2012 04:19 PM, Ian Lance Taylor wrote:
On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo wrote:
2012-11-15 Diego Novillo
Adjust for new vec API (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
* config/bfin/bfin.c: Use new vec API in vec.h.
I think that changes li
On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo wrote:
> 2012-11-15 Diego Novillo
>
> Adjust for new vec API
> (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
>
> * config/bfin/bfin.c: Use new vec API in vec.h.
I think that changes like this count as obvious and do not require
On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo wrote:
> 2012-11-15 Diego Novillo
>
> Adjust for new vec API
> (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
>
> * go-lang.c: Use new vec API in vec.h.
> * gofrontend/expressions.cc: Likewise.
> * gofrontend/gogo
On Thu, Nov 15, 2012 at 1:53 PM, Diego Novillo wrote:
>
> 2012-11-15 Diego Novillo
>
> * vec.c (register_overhead): Convert it into
> member function of vec_prefix.
> (release_overhead): Likewise.
> (calculate_allocation): Likewise.
> (vec_heap_free): Rem
Hi,
Is it ok for the trunk?
Thanks,
Wei.
On Tue, Nov 13, 2012 at 5:06 PM, Wei Mi wrote:
> Thanks for catching this. I update the patch.
>
> Regards,
> Wei.
>
> On Tue, Nov 13, 2012 at 4:54 PM, Richard Henderson wrote:
>> On 11/13/2012 04:08 PM, Wei Mi wrote:
>>> +extern void tsan_finish_file (
Attached below is an initial port of ASAN to powerpc*-linux.
With the patch below along with Jakub's ASAN testsuite patch:
http://gcc.gnu.org/ml/gcc-patches/2012-11/msg01365.html
ASAN not only builds, but seems to be working. The lone ASAN
test case does fail, but it seems to be related to us
It seems like your new code is a generalization of the old code for
handling substitution of a pack for itself (arg_from_parm_pack and such)
and the code for handling other packs with a single pack expansion
argument, and should replace those rather than adding on.
The solution that if at a ce
Hi,
On 11/16/2012 10:58 PM, François Dumont wrote:
We can see that inserting the same elements again, that is to say
detecting the collisions, is slower in the new implementation. It is
the problem I had already signaled in bugzilla entry. In the new
implementation when we need to look for
Attached patch applied.
2012-11-16 François Dumont
* include/bits/hashtable_policy.h (_Prime_rehash_policy): Remove
automatic shrink.
(_Prime_rehash_policy::_M_bkt_for_elements): Do not call
_M_next_bkt anymore.
(_Prime_rehash_policy::_M_next_bkt): Move usage of
_S_gro
The included small change to g++.dg/tm/pr51516.C fixes the remaining TM
regression. With two code paths, there are two instances of the
constructor (a clone and an uninstrumented version), so that part of the
test is no longer relevant.
However... in fixing this, I noticed that for some odd r
Hi,
The previous patch was incomplete because the front-end strips off
invalid target attributes which I did not consider. The attached
updated patch handles this with updated test cases.
Thanks,
-Sri.
On Thu, Nov 15, 2012 at 2:08 PM, Sriraman Tallam wrote:
> Hi,
>
>Currently, function m
On 11/15/2012 01:14 PM, Jakub Jelinek wrote:
Hi!
On the following testcase we ICE, because tree_ssa_dominator_optimize
modifies the bitmap while iterating it. In particular, it contained just
a single bit in it, bitmap_clear_bit turned the bitmap into empty bitmap and
bitmap_set_bit on a lower
> This patch addresses the bogus "Invocation mismatch" messages seen in parallel
> profiledbootstrap builds of gcc. See PR bootstrap/55051 for a discussion of
> why this is occurring and why this checking is inaccurate.
>
> Profilebootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk?
This patch implements the standard pattern bswaphi2 for AArch64.
Regression tests all pass.
OK for trunk and backport to arm/aarch64-4.7-branch?
Cheers,
Ian
2012-11-16 Ian Bolton
* gcc/config/aarch64/aarch64.md (bswaphi2): New pattern.
* gcc/testsuite/gcc.target/aarch64/bu
On Fri, Nov 16, 2012 at 11:32 AM, Tom Tromey wrote:
> Diego>= build_utf8_ref (unmangle_classname
> (IDENTIFIER_POINTER (sig),
> Diego> -IDENTIFIER_LENGTH
> (sig)));
> Diego> -e = &VEC_index (constructor_elt, v, idx-
On 11/15/2012 03:08 PM, Eric Botcazou wrote:
> For a naked call to memcpy? I don't think so, this is a call so there will
> be
> an abnormal edge at the Tree level. The problem arises only when the
> abnormal
> edge is created during RTL expansion.
Well, there's a call at the tree level for
On 11/16/2012 03:21 AM, Michael Zolotukhin wrote:
> Maybe we should introduce a
> new gen-utility which should be used only for dumping md-files and add
> a new target into makefile to invoke it,
This is exactly what I suggested somewhere up-thread.
r~
On 16 November 2012 18:05, Tom Tromey wrote:
>> "Tom" == Tom Tromey writes:
>
> Jonathan> Please go ahead and commit, thanks, Tom!
>
> Tom> Thanks. The needed gdb patches are still pending, so I plan to wait
> Tom> until those go in before committing to libstdc++. I hope it will be
> Tom> ne
Hi all,
I have just committed an obvious patch for a recent OOP regression:
http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=193568
Thanks to Damian for reporting ...
Cheers,
Janus
On 11/16/2012 05:34 PM, Matthias Klose wrote:
> this is an update of zlib from 1.2.5 to 1.2.7, the compressed changes are
> attached. No merge glitches. Ok for the trunk?
Fine by me, because I guess we should keep up with supported zlib,
as long as it all still works.
Andrew.
> "Tom" == Tom Tromey writes:
Jonathan> Please go ahead and commit, thanks, Tom!
Tom> Thanks. The needed gdb patches are still pending, so I plan to wait
Tom> until those go in before committing to libstdc++. I hope it will be
Tom> next week sometime.
Hahaha, "next week". The patches wen
A commit I did earlier in the week got truncated somehow, leading
to a broken testcase for AArch64 target.
I've just commited this fix as obvious on trunk and the
arm/aarch64-4.7-branch.
Cheers
Ian
Index: gcc/testsuite/gcc.target/aarch64/csinc-2.c
===
Hi,
that's not really a bugfix, but I don't see the point in keeping this flag,
which was already dead in 4.7, one more release.
Tested on x86_64-suse-linux, OK for the mainline?
2012-11-16 Eric Botcazou
* tree.h (DECL_RESTRICTED_P): Delete.
(DECL_READ_P): Add comment.
On Fri, Nov 16, 2012 at 08:55:52PM +0400, Alexander Potapenko wrote:
> >> Also, Alexander Potapenko is the best person to ask about asan-darwin.
> >
> > here.
> >
> >> Maybe we can add him to the list of sanitizer maintainers?
> >
> > Seconded. At least for libsanitier/Darwin.
> >
> > Cheers.
>> Also, Alexander Potapenko is the best person to ask about asan-darwin.
>
> here.
>
>> Maybe we can add him to the list of sanitizer maintainers?
>
> Seconded. At least for libsanitier/Darwin.
>
> Cheers.
I can take this, but I'll be busy within the several upcoming days
(till mid-next-wee
On Fri, Nov 16, 2012 at 05:00:22PM +0100, Jakub Jelinek wrote:
> On Fri, Nov 16, 2012 at 10:57:04AM -0500, Jack Howarth wrote:
> > +case "$host" in
> > + *-*-darwin*) MACH_OVERRIDE=true ;;
> > + *) MACH_OVERRIDE=false ;;
> > +esac
> > +AM_CONDITIONAL(USING_MACH_OVERRIDE, $MACH_OVERRIDE)
> > +
>
The following patch fixes
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55330
The problem was in a complex interaction of inheritance and
constraint passes. The test would be compiled successfully if we
increased the number constraint passes. But for such pass interaction,
it hard to know
> "Diego" == Diego Novillo writes:
Diego> 2012-11-15 Diego Novillo
Diego> Adjust for new vec API (http://gcc.gnu.org/wiki/cxx-conversion/cxx-vec)
Diego> * boehm.c: Use new vec API in vec.h.
Diego> * class.c: Likewise.
Diego> * constants.c: Likewise.
Diego> * decl.c: Likewise.
Diego>
This patch to the Go frontend fixes the case of float64(float32(1.0)).
Here float32(1.0) is a typed constant. The compiler was translating
that to float64(1.0), which is not correct. Bootstrapped and ran Go
testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7
branch.
Ian
Index:
This patch adds + to gcc/go/Make-lang.in as needed so that the make -j
option will work correctly when doing an LTO bootstrap. This
corresponds to similar patches to other frontends. Bootstrapped on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
Index: gcc/go/Make-lang.in
===
Hi,
I'm sending this on behalf of Sofiane Naci who
is currently on holiday.
Code generation for C++ thunks previously spat out assembly code,
which affects scheduling of the generated code.
This patch rewrites the code to generate RTL patterns.
A full aarch64-none-elf regression run shows no i
On Fri, Nov 16, 2012 at 10:57:04AM -0500, Jack Howarth wrote:
> +case "$host" in
> + *-*-darwin*) MACH_OVERRIDE=true ;;
> + *) MACH_OVERRIDE=false ;;
> +esac
> +AM_CONDITIONAL(USING_MACH_OVERRIDE, $MACH_OVERRIDE)
> +
Shouldn't AM_CONDITIONAL follow AM_INIT_AUTOMAKE? I'd say move it
before AC_CO
On Fri, Nov 16, 2012 at 09:27:26AM +0100, Dodji Seketeli wrote:
> Jack Howarth writes:
>
> > The Google branch is missing the required
> > interception/mach_override/mach_override.h and
> > interception/mach_override/mach_override.c files from compiler-rt svn
> > for darwin. I have posted wha
On Fri, Nov 16, 2012 at 8:51 AM, Jason Merrill wrote:
> Would it work to just do a cp_parser_commit_to_tentative_parse after we see
> the '='?
I like that solution better.
-- Gaby
Hello!
2012-11-16 Uros Bizjak
* lib/gcc-dg.exp (cleanup-saved-temps): Add .mii to suffixes.
* gcc.dg/hoist-register-pressure-1.c: Cleanup hoist rtl dump.
* gcc.dg/hoist-register-pressure-2.c: Ditto.
* gcc.dg/hoist-register-pressure-3.c: Ditto.
* gcc.dg/t
On 16/11/12 15:35, Ian Lance Taylor wrote:
On Fri, Nov 16, 2012 at 1:54 AM, Andrew Haley wrote:
On 11/14/2012 01:49 PM, Richard Earnshaw wrote:
Please please don't get into the habit of calling it ARM32 and ARM64,
you're just sowing confusion; there are good reasons why those names
weren't ad
On Thu, Nov 15, 2012 at 5:48 PM, Richard Sandiford
wrote:
>> @@ -4013,11 +4013,9 @@ struct mips_multi_member {
>> typedef struct mips_multi_member mips_multi_member;
>>
>> /* Vector definitions for the above. */
>> -DEF_VEC_O(mips_multi_member);
>> -DEF_VEC_ALLOC_O(mips_multi_member, heap);
>
On Fri, Nov 16, 2012 at 1:54 AM, Andrew Haley wrote:
> On 11/14/2012 01:49 PM, Richard Earnshaw wrote:
>
>> Please please don't get into the habit of calling it ARM32 and ARM64,
>> you're just sowing confusion; there are good reasons why those names
>> weren't adopted (some technical, some not) an
> 2012-11-15 Diego Novillo
>
> * gengtype-lex.l (VEC): Remove.
> Add characters in the set [\!\>\.-].
> * gengtype-parse.c (token_names): Remove "VEC".
> (require_template_declaration): Remove handling of VEC_TOKEN.
> (type): Likewise.
> Call creat
Tom Tromey a écrit:
> My build of gcc today tried to run autoconf in libsanitizer.
> However, I had the wrong version in my path, so the build died.
>
> In gcc it is normal to use AM_MAINTAINER_MODE to avoid this problem.
> I think this is missing from libsanitizer/configure.ac due to an
> oversi
On Fri, Nov 16, 2012 at 07:57:59AM -0700, Tom Tromey wrote:
> My build of gcc today tried to run autoconf in libsanitizer.
> However, I had the wrong version in my path, so the build died.
>
> In gcc it is normal to use AM_MAINTAINER_MODE to avoid this problem.
> I think this is missing from libsa
My build of gcc today tried to run autoconf in libsanitizer.
However, I had the wrong version in my path, so the build died.
In gcc it is normal to use AM_MAINTAINER_MODE to avoid this problem.
I think this is missing from libsanitizer/configure.ac due to an
oversight.
This patch adds the missing
Would it work to just do a cp_parser_commit_to_tentative_parse after we
see the '='?
Jason
On Fri, Nov 16, 2012 at 2:06 AM, Jakub Jelinek wrote:
>
> Haven't studied libbacktrace too much, can it provide right now
> just function name or function name and file:line info too?
Yes. backtrace_syminfo reads the symbol table to map a PC value to a
symbol name. backtrace_pcinfo reads the de
Ah, I see. OK.
Jason
Jason Merrill writes:
> On 11/16/2012 07:43 AM, Dodji Seketeli wrote:
>> So I guess that condition should be changed to TREE_CODE
>> (template_type) == ENUMERAL_TYPE, to specifically detect the member
>> enum of a class template case.
>
> Why does that help? What is template_type in the alias te
Hi,
I seem to have uncovered what seems to be a bug with handling SUBREG (MEM) with
the MEM having side-effects while testing aarch64-4.7. This bug seems to be
latent on trunk.
Here is a test case reduced from gcc.c-torture/execute/scal-to-vec1.c.
#define vector(elcount, type) \
__attribute__
On Fri, Nov 16, 2012 at 09:27:26AM +0100, Dodji Seketeli wrote:
> Jack Howarth writes:
>
> > The Google branch is missing the required
> > interception/mach_override/mach_override.h and
> > interception/mach_override/mach_override.c files from compiler-rt svn
> > for darwin. I have posted wha
On 11/16/2012 07:43 AM, Dodji Seketeli wrote:
So I guess that condition should be changed to TREE_CODE
(template_type) == ENUMERAL_TYPE, to specifically detect the member
enum of a class template case.
Why does that help? What is template_type in the alias template case?
Jason
* testsuite/gcc.dg/torture/pr55238.c: New test.
Does this PR rely on hidden visibility? The new testcase uses
attribute visibility hidden, but does not check DejaGNU
dg-require-visibility and fails on AIX.
void __attribute__ ((visibility ("hidden"))) gz_error (gz_state
*state, int err, c
I am friendly pinging the patch below ...
Dodji Seketeli a écrit:
> Hello,
>
> Consider this invalid example given in the PR, where T is not defined:
>
> 1template
> 2struct X {
> 3using type = T;
> 4};
>
> g++ yields the confusing diagnost
I am friendly pinging the patch below ...
Dodji Seketeli a écrit:
> Hello,
>
> Consider this example:
>
> 1template struct List {};
> 2template struct Z {static const int value = T;};
> 3template using LZ = List...>;
> 4
> 5template
>
Hello,
Consider this short example:
1 template
2 using AddConst = T const;
3
4 enum FwdEnum : int;
5
6 int main() {
7AddConst *ptr = nullptr;
8 }
At line 7, when we build the type for AddConst in
lookup_template_class_1, the resulting type is the
> -Original Message-
> From: Matthew Gretton-Dann [mailto:matthew.gretton-d...@linaro.org]
> Sent: Friday, November 16, 2012 6:30 PM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH ARM]Define LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target
>
> On 16 November 2012 05:37, B
On 16-11-2012 12:13, Diego Novillo wrote:
> On Fri, Nov 16, 2012 at 5:23 AM, Pedro Alves wrote:
>
>> Was this considered?
>
> Yup. I just did not implement it. Would be a good follow up, though.
Ah, good to know. Thanks.
For the archives, cause gcc-patches@ bounced my mail for not being sub
On Fri, Nov 16, 2012 at 5:23 AM, Pedro Alves wrote:
> Was this considered?
Yup. I just did not implement it. Would be a good follow up, though.
Diego.
On Thu, Nov 15, 2012 at 4:53 PM, Diego Novillo wrote:
>
> [ The patch is too big for e-mail. It's available at
> http://www.airs.com/~dnovillo/pub/vec-rewrite/06vec.diff ]
This is the wrong link. The correct one is
http://www.airs.com/~dnovillo/pub/vec-rewrite/06cp.diff
Diego.
2012/11/16 Martin Jambor :
> Hi,
>
> On Thu, Nov 15, 2012 at 04:53:25PM -0500, Diego Novillo wrote:
>> [ The patch is too big for e-mail. It's available at
>> http://www.airs.com/~dnovillo/pub/vec-rewrite/06vec.diff ]
>
> when I tried to wget it, I got ERROR 404: Not Found. (I downloaded
> 01vec
Hello!
2012-11-16 Uros Bizjak
* g++.dg/mv1.C (dg-options): Add -march=x86-64.
* g++.dg/mv6.C (dg-options): Ditto.
Tested on x86_64-linux-gnu {,-m32} configured --with-arch=corei7-avx
and committed to mainline SVN.
Uros.
Index: g++.dg/mv1.C
===
>> Yeah, one or other way to being able to debug what exactly has been
>> performed during the iterator expansion is certainly desirable for the
>> future.
> We actually have internal machinery for dumping MDs with expanded
> iterators and substs, but this looks really kinda hack now.
> We're going
On 16 November 2012 05:37, Bin Cheng wrote:
> Hi,
> This patch defines LOGICAL_OP_NON_SHORT_CIRCUIT for ARM target and prefers
> short circuit for armv6-m and Thumb2+Os.
>
> ===
> --- gcc/config/arm/arm.h (revision 193494)
> +++ gcc/
Hi,
On Thu, Nov 15, 2012 at 04:53:25PM -0500, Diego Novillo wrote:
> [ The patch is too big for e-mail. It's available at
> http://www.airs.com/~dnovillo/pub/vec-rewrite/06vec.diff ]
when I tried to wget it, I got ERROR 404: Not Found. (I downloaded
01vec.diff fine.)
Thanks,
Martin
>
> 201
On Thu, Nov 15, 2012 at 05:01:49PM -0800, Andrew Pinski wrote:
> All of these tests failure for me because my addr2line is too old and
> does not support dwarf4. Can we move over to using libbacktrace in
> libsanitzer instead of depending on addr2line here?
The problem is that addr2line doesn't f
On 11/14/2012 01:49 PM, Richard Earnshaw wrote:
> Please please don't get into the habit of calling it ARM32 and ARM64,
> you're just sowing confusion; there are good reasons why those names
> weren't adopted (some technical, some not) and I'm not about to rehash
> them all now. AArch32 and AA
On Fri, Nov 16, 2012 at 9:52 AM, Jakub Jelinek wrote:
>> 2012-11-16 Uros Bizjak
>>
>> * lib/target_suports.exp
>> (check_effective_target_has_w_floating_suffix): New procedure.
>> (check_effective_target_has_q_floating_suffix): Ditto.
>> * g++.dg/cpp0x/gnu_fext-numeric-
Hi!
- PR54921 invalidate sp in cselib on fp setter insn
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg02035.html
(perhaps in light of PR54402 the single_succ (ENTRY_BLOCK_PTR)
from the patch should be nuked)
- PR55188 testcase fix for targets with different branch cost
http://gcc.gnu.org/m
On Fri, Nov 16, 2012 at 08:53:54AM +0100, Uros Bizjak wrote:
> 2012-11-16 Uros Bizjak
>
> * lib/target_suports.exp
> (check_effective_target_has_w_floating_suffix): New procedure.
> (check_effective_target_has_q_floating_suffix): Ditto.
> * g++.dg/cpp0x/gnu_fext-numeric-
Jack Howarth writes:
> The Google branch is missing the required
> interception/mach_override/mach_override.h and
> interception/mach_override/mach_override.c files from compiler-rt svn
> for darwin. I have posted what I believe to be the final patch which
> eanbles libsanitizer on darwin...
David Miller writes:
> From: Dodji Seketeli
> Date: Thu, 15 Nov 2012 11:56:40 +0100
>
>> David Miller wrote
>>
>>> From: Dodji Seketeli
>>> Date: Wed, 14 Nov 2012 14:26:40 +0100
>>>
>>> > I guess we could do that. That would build libsanitizer, but asan will
>>> > still not be available on
74 matches
Mail list logo
