> I've looked at the C++ testcase
>
> int foo (int &x)
> {
> try {
> return x;
> }
> catch (...)
> {
> return 0;
> }
> }
>
> which exhibits exactly the behavior you quote - return x is considered
> throwing an exception. The C++ FE doesn't arrange for TREE_THIS_NOTRAP to
> be s
> The attached patch fixes the regression in gcc.dg/attr-ifunc-4.c (PR 58398).
>
> The problem is that the resolver function just looks like an alias, but it
> actually is
> something completely different. So inlining the resolver function has to be
> avoided.
>
> The patch was bootstrapped and
Hello Jan,
the MAINTAINERS file reveals that you are the right person to contact
for profile feedback related changes.
This is the third iteration of the caller instrumentation patch
originally posted and explained here:
http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01593.html
The hooks now co
2013-07-01 Paul Woegerer
Caller instrumentation with -finstrument-calls.
* gcc/builtins.def: Add call-hooks __gnu_profile_call_before and
__gnu_profile_call_after.
* gcc/libfuncs.h (enum libfunc_index): Likewise.
* gcc/optabs.c (init_optabs): Likewise.
On Mon, 16 Sep 2013, Jakub Jelinek wrote:
> Hi!
>
> This patch fixes two issues I found on the pr58392.c testcase:
> 1) we weren't copying decl attributes, so e.g. inside #pragma omp parallel
> "omp simd array" temporary arrays lost their attribute and weren't
> adjusted because of that
> 2) DR_A
Hi Guys,
I am applying the patch below to add support for interrupt handlers to
the MSP430 backend. The patch also adds a couple of MSP430 specific
builtin functions intended to be used inside interrupt handlers. In
addition the patch adds support for naked functions, critical
function
On Mon, Sep 16, 2013 at 8:36 PM, Teresa Johnson wrote:
> Yep, looked too quickly every time and thought the newline after "be
> zero" was applying. Here is the patch with the fix. Ok for trunk
> pending regression testing?
Ok.
Thanks,
Richard.
> 2013-09-16 Teresa Johnson
>
> * covera
Hi,
Here is a patch introducing new type and mode for bounds. It is a part of MPX
ISA support patch (http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01094.html).
Bootstrapped and tested on linux-x86_64. Is it OK for trunk?
Thanks,
Ilya
--
gcc/
2013-09-16 Ilya Enkovich
* mode-classes.
On Mon, Sep 16, 2013 at 10:24 PM, Xinliang David Li wrote:
> On Mon, Sep 16, 2013 at 3:13 AM, Richard Biener
> wrote:
>> On Fri, Sep 13, 2013 at 5:16 PM, Xinliang David Li
>> wrote:
>>> On Fri, Sep 13, 2013 at 1:30 AM, Richard Biener
>>> wrote:
On Thu, Sep 12, 2013 at 10:31 PM, Xinliang D
On Tue, Sep 17, 2013 at 9:03 AM, Eric Botcazou wrote:
>> I've looked at the C++ testcase
>>
>> int foo (int &x)
>> {
>> try {
>> return x;
>> }
>> catch (...)
>> {
>> return 0;
>> }
>> }
>>
>> which exhibits exactly the behavior you quote - return x is considered
>> throwing an e
Hi,
Let me suggest to remove the section "Macros to control conditional
execution" in GCC internals. I assume the section is obsolete given
that it is empty.
Best,
Nicklas
2013-09-17 Nicklas Bo Jensen
* doc/tm.texi (Macros to control conditional execution):
Remove empty section.
Inde
Hi,
On 09/15/2013 03:45 AM, Tim Shen wrote:
...finally.
This patch complete flags specifed in [28.5]. However, `optimize` and
`match_any` are ignored. `format_*` are unimplemented yet.
regex_iterator and regex_token_iterator should work now, but need more
testcases.
Great. Tim, please complete
On Tue, Sep 17, 2013 at 10:20 AM, Richard Biener
wrote:
> On Mon, Sep 16, 2013 at 10:24 PM, Xinliang David Li
> wrote:
>> On Mon, Sep 16, 2013 at 3:13 AM, Richard Biener
>> wrote:
>>> On Fri, Sep 13, 2013 at 5:16 PM, Xinliang David Li
>>> wrote:
On Fri, Sep 13, 2013 at 1:30 AM, Richard B
On 16 Sep 11:24, Uros Bizjak wrote:
> On Fri, Sep 13, 2013 at 12:18 PM, Ilya Enkovich
> wrote:
> > 2013/9/11 Uros Bizjak :
> >>
> >
> > Hi Uros,
> >
> > Thanks a lot for the review!
> >
> >> The x86 part looks mostly OK (I have a couple of comments bellow), but
> >> please first get target-indepe
Here is a final patch with fixed commentary.
2013/9/16 Uros Bizjak :
> On Mon, Sep 16, 2013 at 5:01 PM, Yuri Rumyantsev wrote:
>
>> Does this comment looks good to you:
>>
>> if (start != NULL_RTX)
>> {
>> bb = BLOCK_FOR_INSN (start);
>> if (start != BB_HEAD (bb))
>> /* Initiali
> Yeah, I thought testing for a PARM_DECL should be sufficient? For
> nested functions references to outer parms should have been lowered via the
> static chain at the point tree-inline.c sees them.
OK for the latter point, but are you sure for the former? My understanding is
that we're already
On Fri, Sep 13, 2013 at 04:29:48PM +0200, Eric Botcazou wrote:
> @@ -4748,6 +4774,8 @@ copy_gimple_seq_and_replace_locals (gimp
>id.transform_call_graph_edges = CB_CGE_DUPLICATE;
>id.transform_new_cfg = false;
>id.transform_return_to_modify = false;
> + id.transform_parameter = false;
On Tue, Sep 17, 2013 at 10:42 AM, Eric Botcazou wrote:
>> Yeah, I thought testing for a PARM_DECL should be sufficient? For
>> nested functions references to outer parms should have been lowered via the
>> static chain at the point tree-inline.c sees them.
>
> OK for the latter point, but are you
Installed as obvious.
Andreas.
* gcc.dg/tree-ssa/ldist-22.c (main): Return zero.
diff --git a/gcc/testsuite/gcc.dg/tree-ssa/ldist-22.c
b/gcc/testsuite/gcc.dg/tree-ssa/ldist-22.c
index f6fff77..afc792f 100644
--- a/gcc/testsuite/gcc.dg/tree-ssa/ldist-22.c
+++ b/gcc/testsuite/gcc.dg/tree-
Hi Martin,
On Tue, 17 Sep 2013 01:09:45, Martin Jambor wrote:
> Hi,
>
> On Sun, Sep 15, 2013 at 06:55:17PM +0200, Bernd Edlinger wrote:
>> Hello Richard,
>>
>> attached is my second attempt at fixing PR 57748. This time the
>> movmisalign path is completely removed and a similar bug in the read
>>
On Tue, Sep 17, 2013 at 10:35:22AM +0200, Nicklas Bo Jensen wrote:
> Hi,
>
> Let me suggest to remove the section "Macros to control conditional
> execution" in GCC internals. I assume the section is obsolete given
> that it is empty.
>
> Best,
> Nicklas
>
> 2013-09-17 Nicklas Bo Jensen
>
On Sun, Sep 15, 2013 at 6:55 PM, Bernd Edlinger
wrote:
> Hello Richard,
>
> attached is my second attempt at fixing PR 57748. This time the movmisalign
> path is completely removed and a similar bug in the read handling of
> misaligned
> structures with a non-BLKmode is fixed too. There are sever
On Mon, Sep 16, 2013 at 08:35:35PM +0200, Jakub Jelinek wrote:
> On Fri, Sep 13, 2013 at 08:01:36PM +0200, Marek Polacek wrote:
> I'd say the above is going to be a maintainance nightmare, with all the code
> duplication. And you are certainly going to miss cases that way,
> e.g.
> void
> foo (voi
On Tue, Sep 17, 2013 at 12:00 PM, Richard Biener
wrote:
> On Sun, Sep 15, 2013 at 6:55 PM, Bernd Edlinger
> wrote:
>> Hello Richard,
>>
>> attached is my second attempt at fixing PR 57748. This time the movmisalign
>> path is completely removed and a similar bug in the read handling of
>> misali
> That's true... so you can only simplify is_parameter_of by dropping
> the context check.
OK, thanks, installed with this modification and the fix for the oversight
spotted by Jakub, after retesting on x86-64/Linux.
--
Eric Botcazou
This is no-op for usual GCC targets, because we don't pass any string to
CreateSemaphore anyway. However this trivial change will help
mingw-w64's efforts to support WinRT, where only unicode variant is
available.
libgcc/Changelog:
config/i386/gthr-win32.c: CreateSemaphoreW instead of CreateSe
2013/9/17 Jacek Caban :
> This is no-op for usual GCC targets, because we don't pass any string to
> CreateSemaphore anyway. However this trivial change will help
> mingw-w64's efforts to support WinRT, where only unicode variant is
> available.
>
> libgcc/Changelog:
> config/i386/gthr-win32.c:
> Hasn't this been already removed by
> http://gcc.gnu.org/ml/gcc-patches/2013-09/msg01231.html
> ?
Yes. Okay. Please ignore then.
Best,
Nicklas
On 09/17/13 13:41, Kai Tietz wrote:
> 2013/9/17 Jacek Caban :
>> This is no-op for usual GCC targets, because we don't pass any string to
>> CreateSemaphore anyway. However this trivial change will help
>> mingw-w64's efforts to support WinRT, where only unicode variant is
>> available.
>>
>> libgc
Hi Jacek,
I applied patch at rev. 202648 with following ChangeLog
2013-09-17 Jacek Caban
* config/i386/gthr-win32.c: CreateSemaphoreW instead of
CreateSemaphoreA.
* config/i386/gthr-win32.h: Likewise.
The wide-variant is in general ok due we don't support any windows-OS
anymore, wh
Hello,
On 16 Sep 16:36, Uros Bizjak wrote:
> The patch with a fixed comment is OK otherwise.
Checked into main trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-09/msg00512.html
--
Thanks, K
On Tue, 17 Sep 2013 12:45:40, Richard Biener wrote:
>
> On Tue, Sep 17, 2013 at 12:00 PM, Richard Biener
> wrote:
>> On Sun, Sep 15, 2013 at 6:55 PM, Bernd Edlinger
>> wrote:
>>> Hello Richard,
>>>
>>> attached is my second attempt at fixing PR 57748. This time the movmisalign
>>> path is complet
This teaches loop distribution to distribute nested loops. I plan
to commit the trivial bits of it but not the rest of the patch
until I have an idea how to best limit the loop nest walk
(it tries distributing nests from outer to inner loops, re-doing
dependence analysis and RDG build).
At this
On Mon, Sep 16, 2013 at 03:59:12PM +, Joseph S. Myers wrote:
> On Mon, 16 Sep 2013, Marek Polacek wrote:
>
> > On Fri, Sep 13, 2013 at 07:18:24PM +, Joseph S. Myers wrote:
> > > On Fri, 13 Sep 2013, Marek Polacek wrote:
> > >
> > > > This is kind of fugly, but don't have anything better a
OK.
Jason
Hi Mike,
Ok, I assume that the changes to hppa and return 0 are intentional and good…
- || [istarget hppa64-hp-hpux11.23] } {
- return 0;
+|| [istarget hppa64-hp-hpux11.23] } {
+ return 0;
Sorry - yes - they are just whitespace adjustments so that the entries
line
On 17/09/13 03:16, bin.cheng wrote:
>
>
>> -Original Message-
>> From: Richard Earnshaw
>> Sent: Thursday, September 12, 2013 11:24 PM
>> To: Bin Cheng
>> Cc: gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more comparison
>> instructions
>>
>> On 18/04/13 0
On 09/09/13 10:56, Kyrylo Tkachov wrote:
> [Resending, since I was away and not pinging it]
>
>
> Hi all,
>
> In PR58088 the constant folder goes into an infinite recursion and runs out of
> stack space because of two conflicting optimisations:
> (X * C1) & C2 plays dirty when nested inside an I
Hi all,
This is a resubmission of my previous demangler fix [1] rewritten
to avoid using hashtables and other libiberty features.
>From the above referenced email:
d_print_comp maintains a certain amount of scope across calls (namely
a stack of templates) which is used when evaluating references
Hello,
Has anyone had a chance to look at this. The C++ part is only a week
old, but the C part has been in review for ~3 weeks. I would greatly appreciate
if someone could review this and approve for trunk if it is Ok for trunk.
Thanks,
Balaji V. Iyer.
> -Original Message-
> F
This patch adds the no_sanitize_undefined attribute, so the user can tell
that a particular function should be ignored by ubsan.
Ran ubsan testsuite/bootstrap-ubsan on x86_64-linux, ok for trunk?
2013-09-17 Marek Polacek
PR sanitizer/58411
* doc/extend.texi: Document no_saniti
Hi,
this patch should fix HP-PA bootstrap issue where we create local aliases but
the target
has no support for them.
Bootstrapped/regtested x86_64-linux (with aliases disabled) and commited.
PR middle-end/58329
* ipa-devirt.c (ipa_devirt): Be ready for symtab_nonoverwritable_ali
On Tue, Sep 17, 2013 at 05:24:22PM +0200, Marek Polacek wrote:
> This patch adds the no_sanitize_undefined attribute, so the user can tell
> that a particular function should be ignored by ubsan.
Does this correspond to some llvm attribute?
> --- gcc/builtins.c.mp22013-09-17 16:13:26.6231
On Tue, Sep 17, 2013 at 05:37:51PM +0200, Jakub Jelinek wrote:
> On Tue, Sep 17, 2013 at 05:24:22PM +0200, Marek Polacek wrote:
> > This patch adds the no_sanitize_undefined attribute, so the user can tell
> > that a particular function should be ignored by ubsan.
>
> Does this correspond to some
Hi,
this patch makes inliner to not inline functions with -O0 optimization attribute
and also to not inline into functions.
Bootstrapped/regtested x86_64-linux, comitted.
PR middle-end/58332
* gcc.c-torture/compile/pr58332.c: New testcase.
* cif-code.def (FUNCTION_NOT_OPTIM
On 09/17/2013 08:50 AM, Iyer, Balaji V wrote:
Hello, Has anyone had a chance to look at this. The C++ part is only
a week old, but the C part has been in review for ~3 weeks. I would
greatly appreciate if someone could review this and approve for trunk
if it is Ok for trunk.
Obviously not yet.
On Tue, Sep 17, 2013 at 06:45:25PM +0200, Marek Polacek wrote:
> --- gcc/builtins.c.mp22013-09-17 16:13:26.623161281 +0200
> +++ gcc/builtins.c2013-09-17 18:42:11.338273135 +0200
> @@ -10313,7 +10313,10 @@ fold_builtin_0 (location_t loc, tree fnd
>return fold_builtin_classify_ty
On Tue, Sep 17, 2013 at 06:51:59PM +0200, Jakub Jelinek wrote:
> On Tue, Sep 17, 2013 at 06:45:25PM +0200, Marek Polacek wrote:
> > --- gcc/builtins.c.mp2 2013-09-17 16:13:26.623161281 +0200
> > +++ gcc/builtins.c 2013-09-17 18:42:11.338273135 +0200
> > @@ -10313,7 +10313,10 @@ fold_builtin_0
On Tue, Sep 17, 2013 at 1:20 AM, Richard Biener
wrote:
> On Mon, Sep 16, 2013 at 10:24 PM, Xinliang David Li
> wrote:
>> On Mon, Sep 16, 2013 at 3:13 AM, Richard Biener
>> wrote:
>>> On Fri, Sep 13, 2013 at 5:16 PM, Xinliang David Li
>>> wrote:
On Fri, Sep 13, 2013 at 1:30 AM, Richard Bi
On Sep 16, 2013, at 8:41 PM, DJ Delorie wrote:
> m32c's PSImode is 24-bits, why does it have "32" in the macro?
>
> /* 24-bit pointers, in 32-bit units */
> -PARTIAL_INT_MODE (SI);
> +PARTIAL_INT_MODE_NAME (SI, 32, PSI);
Sorry, fingers copied the wrong number. Thanks for the catch.
Index: gcc/
On Tue, Sep 17, 2013 at 8:45 AM, Jakub Jelinek wrote:
> On Tue, Sep 17, 2013 at 08:37:57AM -0700, Xinliang David Li wrote:
>> >> char a[1];
>> >>
>> >> void foo(int n)
>> >> {
>> >> int* b = (int*)(a+n);
>> >> int i = 0;
>> >> for (; i < 1000; ++i)
>> >> b[i] = 1;
>> >> }
>> >>
>> >>
On Tue, Sep 17, 2013 at 06:26:52PM +0200, Marek Polacek wrote:
> Well I wonder too ;) I thought it can't be NULL, and tried this
>
> struct C {
> C() { __builtin_unreachable (); }
> };
I was more wondering about stuff like:
int a = (__builtin_unreachable (), 1);
or similar.
Jakub
On Tue, Sep 17, 2013 at 08:37:57AM -0700, Xinliang David Li wrote:
> >> char a[1];
> >>
> >> void foo(int n)
> >> {
> >> int* b = (int*)(a+n);
> >> int i = 0;
> >> for (; i < 1000; ++i)
> >> b[i] = 1;
> >> }
> >>
> >> int main(int argn, char** argv)
> >> {
> >> foo(argn);
> >> }
> >
On Tue, Sep 17, 2013 at 06:34:38PM +0200, Jakub Jelinek wrote:
> On Tue, Sep 17, 2013 at 06:26:52PM +0200, Marek Polacek wrote:
> > Well I wonder too ;) I thought it can't be NULL, and tried this
> >
> > struct C {
> > C() { __builtin_unreachable (); }
> > };
>
> I was more wondering about stu
Mike Stump writes:
> +/* Partial integer modes are specified by relation to a full integer
> + mode. */
> +#define PARTIAL_INT_MODE(M,PREC) \
> + make_partial_integer_mode (#M, "P" #PREC #M, PREC, __FILE__, __LINE__)
> +/* Partial integer modes are specified
The new test gcc.dg/tree-ssa/slsr-39.c fails in 64 bit mode (see
http://gcc.gnu.org/ml/gcc-regression/2013-09/msg00455.html ).
Looking for MEM in the dump returns
_12 = MEM[(int[50] *)_17];
MEM[(int[50] *)_20] = _13;
TIA
Dominique
On 09/17/2013 12:39 PM, Trevor Saunders wrote:
I'd like to go ahead and get your patch installed -- do you have a
GCC copyright assignment on file with the FSF? Your change is large
enough to require one.
Its my understanding that Mozilla has one covering work done by
employees which would inc
Committed to the head.
Is this too radical to also go on the 4.8 branch?
We would need to discuss it on the RTEMS side but it
only impacts us if the multilib is there for sparc-elf
on 4.8.
Thanks Sebastian.
On 8/30/2013 6:58 AM, Daniel Hellstrom wrote:
> Hello Sebastian,
>
> That seems like a g
Hello,
after vectors, lists. I didn't touch the throw we were discussing earlier
today for now. There will be an inconsistency with debug list iterators
because they use a general wrapper:
- I would need François to tell if that wrapper is ever used with
iterators that can throw,
- the same wr
Richi,
This patch canonizes the bits above the precision for wide ints with
types or modes that are not a perfect multiple of HOST_BITS_PER_WIDE_INT.
I expect that most of the changes in rtl.h will go away. in
particular, when we decide that we can depend on richard's patch to
clean up rtl
Committed.
2013-09-17 Nick Clifton
* config/rl78/rl78.c (need_to_save): Change return type to bool.
For interrupt functions: save all call clobbered registers if the
interrupt handler is not a leaf function.
(rl78_expand_prologue): Always recompute the frame in
On Tue, 2013-09-10 at 15:34 +0200, Jan Hubicka wrote:
Thanks for reviewing this, and sorry for the late response (I lost most
of last week to illness). Some questions inline below...
> > This patch is the handwritten part of the conversion of these types
> > to C++; it requires the followup patc
GCC typically avoids using virtual registers $r24 through $r31, as
this register bank (bank 3) is reserved for hand-written assembly
interrupt handlers. If unneeded for that, this new option lets gcc
use those registers also. Committed.
* config/rl78/constraints.md (Wcv): Allow up to $r
This patch to libgo fixes a bug when calling a method when the
reflect.Value object holds a pointer to the actual value. The code was
calling iword which tests v.kind, but for a method value that is always
Func. This fixes the code to implement iword directly using
v.typ.Kind(). Bootstrapped and
This is a repost with fixes to avoid the phase-ordering problem exposed
by 58387 and 58340. I've included the testcase for 58387.
--
I recently noticed that we were failing to propagate edge equivalences
into PHI arguments in non-dominated successors.
The case loos like this:
;; basic
I merged the GCC 4.8 brach to the gccgo branch.
Ian
Hi,
this ICE is caused by error_mark_node as TREE_TYPE of a TYPE_DECL, which
leads to a crash at the beginning of the TYPE_DECL case of tsubst_decl.
I tried various approaches - for example turning all error_operand_p (t)
== true arguments passes to tsubst into error_mark_nodes also works -
Track both parts of far addresses so they don't get optimized away.
Committed.
* config/rl78/constraints.md: For each W* constraint, rename to C*
and create a W* constraint that checks for an optional ES: prefix
pattern also.
* config/rl78/rl78.md (UNS_ES_ADDR): Ne
This patch makes AutoFDO honor system paths stored in the profile.
Bootstrapped and passed regression tests.
OK for google-4_8 branch?
Thanks,
Dehao
Index: gcc/auto-profile.c
===
--- gcc/auto-profile.c (revision 202672)
+++ gcc/aut
ok.
David
On Tue, Sep 17, 2013 at 4:53 PM, Dehao Chen wrote:
> This patch makes AutoFDO honor system paths stored in the profile.
>
> Bootstrapped and passed regression tests.
>
> OK for google-4_8 branch?
>
> Thanks,
> Dehao
>
> Index: gcc/auto-profile.c
> ==
While doing some work on power8, I wanted to make sure that for existing
systems, I was generating the same code. So I built some code and ran it
through various -mcpu= options. When I built a powerpc-linuxpaired
compiler, the compiler has trouble with a simple loop that should be
vectorized.
On Sep 17, 2013, at 10:24 AM, Mike Stump wrote:
> On Sep 16, 2013, at 8:41 PM, DJ Delorie wrote:
>> m32c's PSImode is 24-bits, why does it have "32" in the macro?
>>
>> /* 24-bit pointers, in 32-bit units */
>> -PARTIAL_INT_MODE (SI);
>> +PARTIAL_INT_MODE_NAME (SI, 32, PSI);
>
> Sorry, fingers
On Tue, Sep 17, 2013 at 12:28 PM, wrote:
> If you don't mind explaining, can you tell me why you didn't apply the
> change to the 4.7 branch?
I'm not maintaining Go on the 4.7 branch. I don't object to somebody
else doing it, I'm just not doing it myself. My time is limited and I
have to draw
A few new patterns. Committed.
2013-09-17 Nick Clifton
* config/rl78/rl78-real.md (bf): New pattern.
(bt): New pattern.
* config/rl78/rl78.c (rl78_print_operand_1): Handle %B.
(rl78_print_operand): Do not put a # before a %B.
* config/rl78/rl78.opt: T
> -Original Message-
> From: Dominique Dhumieres [mailto:domi...@lps.ens.fr]
> Sent: Wednesday, September 18, 2013 1:47 AM
> To: gcc-patches@gcc.gnu.org
> Cc: hjl.to...@gmail.com; Bin Cheng
> Subject: Re: [PATCH GCC]Catch more MEM_REFs sharing common
> addressing part in gimple strength r
Thanks Richard for the review.
On 16/09/13 23:43, Richard Biener wrote:
On Mon, 16 Sep 2013, Kugan wrote:
Hi,
Updated the patch to the latest changes in trunk that splits tree.h. I also
noticed an error in printing double_int and fixed it.
Is this OK?
print_gimple_stmt (dump_file,
Hi Janus,
(Side remark: That's Janus' email which didn't make it past GCC's mail
server.)
Janus Weil wrote:
Fortran 2008 permits assignment to polymorphic variables with some
constraints. The patch, which was sitting in my tree, adds diagnostic to
reject invalid use. For valid code, it runs i
77 matches
Mail list logo
