From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* config/v850/v850.c (expand_prologue): Adjust.
(expand_epilogue): Likewise.
* expr.c (init_expr_target): Likewise.
* genrecog.c (print_subroutine): Always make the argument type
rtx_insn
From: Trevor Saunders
Hi,
Basically $subject which gets rid of a few more casts over all.
I ment to get this out a little while back, but life got busy, and I didn't
read the status announcement properly, so virtually working from hawaii for
now. patches individually built and regtested on x86_
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* cse.c (count_reg_usage): Adjust.
* rtl.h: Adjust prototypes.
* rtlanal.c (find_reg_equal_equiv_note): Change argument type to
rtx_insn *.
---
gcc/cse.c | 63 +++-
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* config/aarch64/aarch64.c (aarch64_emit_unlikely_jump): split
up variables to make some rtx_insn *.
* config/alpha/alpha.c (emit_unlikely_jump): Likewise.
* config/arc/arc.c: Likewise.
*
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* config/arm/arm.c (legitimize_pic_address): Change to use
rtx_insn * as the type of variables.
(arm_pic_static_addr): Likewise.
(arm_emit_movpair): Likewise.
* config/c6x/c6x.c (reorg_spl
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* rtl.h: Adjust prototype.
* rtlanal.c (dead_or_set_p): Change argument type to rtx_insn *.
(dead_or_set_regno_p): Likewise.
---
gcc/rtl.h | 4 ++--
gcc/rtlanal.c | 4 ++--
2 files changed, 4 inserti
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* cfgrtl.c (delete_insn): Change argument type to rtx_insn *.
(fixup_reorder_chain): Adjust.
* cfgrtl.h: Adjust prototype.
---
gcc/cfgrtl.c | 5 ++---
gcc/cfgrtl.h | 2 +-
2 files changed, 3 insertions(+
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* rtl.h: Adjust prototype.
* rtlanal.c (replace_label_in_insn): Change argument type to
rtx_insn *.
---
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
dif
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* config/arm/arm-protos.h: Adjust prototype.
* config/arm/arm.c (use_return_insn): Change argument type to
rtx_insn *.
* rtl.h (is_a_helper ::test): New specialization.
* rtlanal.c (reg_se
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* function.c (contains): Change argument type to rtx_insn *.
(prologue_contains): Likewise.
(epilogue_contains): Likewise.
(prologue_epilogue_contains): Likewise.
* function.h: Adjust prot
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* optabs.c (emit_libcall_block): Change argument type to
rtx_insn *.
* optabs.h: Adjust prototype.
---
gcc/optabs.c | 5 ++---
gcc/optabs.h | 2 +-
2 files changed, 3 insertions(+), 4 deletions(-)
diff
From: Trevor Saunders
gcc/ChangeLog:
2016-11-14 Trevor Saunders
* rtl.h: Adjust prototype.
* rtlanal.c (add_int_reg_note): Change argument type to rtx_insn *.
---
gcc/rtl.h | 2 +-
gcc/rtlanal.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/r
Hi,
this is the second attempt to support ASan odr indicators in GCC. I've
fixed issues with several flags (e.g.TREE_ADDRESSABLE) and introduced
new "asan odr indicator" attribute to distinguish indicators from other
symbols.
Looks better now?
Tested and ASan bootstrapped on x86_64-unknown-l
Hi Steve,
> The attach patch allows a procedure with a class result to
> be an actual argument to subprogram where the dummy argument
> expected to be a class. OK to commit?
that patch actually does not look quite right to me. Does it survive a regtest?
I think one should rather check why the c
In various situations the cplus_demangle () function could read past the
end of input causing crashes. Add checks in various places to not advance
the demangle string location and fail early when end of string is reached.
Add various examples of input strings to the testsuite that would crash
test-
On Sat, Nov 12, 2016 at 8:36 AM, Evgeny Kudryashov wrote:
> On 2016-11-10 13:30, Bin.Cheng wrote:
>>
>> Hi,
>> I see the cost problem with your test now. When computing an address
>> type iv_use with a candidate, the computation consists of two parts,
>> for computation can be represented by addr
I was looking at the possibility of dropping threading from VRP1/VRP2 or
DOM1/DOM2 in favor of the backwards threader -- the obvious idea being
to recover some compile-time for gcc-7.
Of the old-style threader passes (VRP1, VRP2, DOM1, DOM2), VRP2 is by
far the least useful. But I can't se
On Fri, Nov 11, 2016 at 09:42:32PM -0700, Sandra Loosemore wrote:
> On 11/11/2016 08:37 AM, James Greenhalgh wrote:
>
> >diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
> >index f133b3a..75ff8ec 100644
> >--- a/gcc/doc/invoke.texi
> >+++ b/gcc/doc/invoke.texi
> >@@ -378,7 +378,8 @@ Objectiv
Hi!
Working virtually out of Samoa.
The following patch is an attempt to handle -fsanitize=undefined
for vectors. We already diagnose out of bounds accesses for vector
subscripts, this patch adds expansion for vector UBSAN_CHECK_* and generates
those in ubsan. Haven't finished up the many vect
On Fri, Nov 11, 2016 at 7:55 PM, Jim Wilson wrote:
> On Thu, Nov 10, 2016 at 2:53 AM, Richard Biener
> wrote:
>> The biggest "lack" of loop distribution is the ability to undo CSE so for
>
> I hadn't noticed this problem yet. I will have to take a look.
>
>> Then of course the cost model is pure
On Sat, Nov 12, 2016 at 9:31 AM, Segher Boessenkool
wrote:
> So far all target implementations of the separate shrink-wrapping hooks
> use the DF LIVE info to figure out around which basic blocks the non-
> volatile registers need to be saved. This is done by looking at the
> IN+GEN+KILL sets of
On Sun, Nov 13, 2016 at 11:31 PM, Eric Botcazou wrote:
> It's the note issued by the -Wlto-type-mismatch warning:
>
> q.ads:7:13: warning: type of 'q__proc' does not match original declaration [-
> Wlto-type-mismatch]
>procedure Proc (A : Arr);
> ^
> q.adb:7:3: note: 'q__proc' was
Eric Botcazou writes:
> Similarly to x86, PowerPC and SPARC, this enables the use of custom run-
> time descriptors in Ada, thus eliminating the need for trampolines and
> executable stack in presence of pointers to nested functions.
>
> Unfortunately I don't have access to MIPS hardware any more
backprop changes values of SSA names - while it properly handles
debug info it fails to reset range/align info.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2016-11-14 Richard Biener
PR tree-optimization/78312
* gimple-ssa-backprop.c (backprop::prep
* Claudiu Zissulescu [2016-11-10 12:02:34
+0100]:
> Hi,
>
> Please find the revised patch which includes the refurbishing of
> mmpy-option option, and a new comment on DEFAULT_arc_fpu_build
> define. As for the last suggestion, my proposal is to have a latter
> patch on the topic of .cpu, synce
Maciej Rozycki writes:
> This small patch series addresses an issue uncovered by a recent
> binutils master branch update, scheduled for the upcoming 2.28 release,
> where we fail to annotate stray code labels -- generally produced for
> code marked as unreachable -- that have no instruction foll
> HEAD binutils supports '.cpu NPS400' and HEAD GCC also correctly emits
> '.cpu NPS400' (when -mcpu=nps400 is passed). After your change we no
> longer correctly emit '.cpu NPS400'.
>
Sorry for this miss-understanding. Updating the patch to emit .cpu NSPS400.
Thanks,
Claudiu
Maciej Rozycki writes:
> gcc/testsuite/
> * gcc.target/mips/mips.exp (mips_option_tests): Add
> `-mmicromips' array element.
OK, thanks.
Matthew
Maciej Rozycki writes:
> gcc/testsuite/
> * gcc.target/mips/mips.exp (mips_option_tests): Add
> `-mcode-readable=yes' array element.
> ---
> OK to apply?
OK, thanks.
Matthew
Mark,
Thank you for your answer!
On 11/10/2016 01:38 PM, Mark Wielaard wrote:
IMHO having an explicit DW_AT_type pointing at the base type with size
and encoding for the DW_TAG_enumerator_type is better for consumers than
having to try and interpret the DW_FORM used to encode the values.
I’m
Hello.
Following patch adds TAB settings to contrib/vimrc file.
Hope it looks reasonable?
Thanks,
Martin
>From 84eb32c4e84b87d690033f7505b8570427ab8468 Mon Sep 17 00:00:00 2001
From: marxin
Date: Sun, 13 Nov 2016 12:05:48 +0100
Subject: [PATCH] vimrc: fix TAB settings
---
contrib/vimrc | 3 +++
2016-11-14 9:56 GMT+01:00 Janus Weil :
> Hi Steve,
>
>> The attach patch allows a procedure with a class result to
>> be an actual argument to subprogram where the dummy argument
>> expected to be a class. OK to commit?
>
> that patch actually does not look quite right to me. Does it survive a
>
Maciej Rozycki writes:
> This however requires the correct annotation of branch targets as code,
> because the ISA mode is not relevant for data symbols and is therefore
> not recorded for them.
I wonder if it would have been possible to add the ISA mode to data
symbols and hide it in readelf/ob
> After looking into this a little bit more, I found that the culprit
> seems to be 'resolve_procedure_interface', which does not properly
> copy the 'class_ok' attribute. I propose the attached patch to fix
> this (regtesting right now) ...
The regtest finished successfully. Is that patch ok for
From: tbsaunde
Last patch I'm squeezing in for stage 1. Jeff approved this back in september,
but I never committed it for some reason. So I updated it to trunk,
rebootstrapped and regtested and committed it.
Thanks!
Trev
gcc/ChangeLog:
2016-08-20 Trevor Saunders
* acinclude.m4
On Mon, 14 Nov 2016, Matthew Fortune wrote:
> At some point in the future we may therefore have to consider backports
> of this work to pre-existing GCC releases or a configure time check in
> binutils to relax the new checks if GCC does not have the .insn fix. I
> don't know how reliable the latt
On Mon, 2016-11-14 at 12:08 +0100, Pierre-Marie de Rodat wrote:
> Thank you for your answer!
>
> On 11/10/2016 01:38 PM, Mark Wielaard wrote:
> > IMHO having an explicit DW_AT_type pointing at the base type with size
> > and encoding for the DW_TAG_enumerator_type is better for consumers than
> >
PING^1
On 10/31/2016 01:13 PM, Martin Liška wrote:
> On 10/31/2016 11:07 AM, Rainer Orth wrote:
>> Hi Martin,
>>
>>> Using priority {cd}tors on a target that does not support that can cause
>>> failures (see the PR).
>>> Apart from that, I decided to use priority 100 for both gcov_init and
>>> gco
On Fri, 11 Nov 2016, Yuri Rumyantsev wrote:
> Richard,
>
> I prepare updated 3 patch with passing additional argument to
> vect_analyze_loop as you proposed (untested).
>
> You wrote:
> tw, I wonder if you can produce a single patch containing just
> epilogue vectorization, that is combine patch
On Fri, 11 Nov 2016, Yuri Rumyantsev wrote:
> Richard,
>
> Here is fixed version of updated patch 3.
>
> Any comments will be appreciated.
Looks good apart from
+ if (epilogue)
+{
+ epilogue->force_vectorize = loop->force_vectorize;
+ epilogue->safelen = loop->safelen;
+ ep
ping
From: Wilco Dijkstra
Sent: 02 November 2016 16:49
To: Ramana Radhakrishnan; GCC Patches
Cc: nd
Subject: Re: [PATCH][AArch64 - v3] Simplify eh_return implementation
ping
From: Wilco Dijkstra
Sent: 02 September 2016 12:31
To: Ramana Radhakrishnan; GCC Patches
Cc: nd
Subject: Re: [
ping
From: Wilco Dijkstra
Sent: 25 October 2016 18:08
To: GCC Patches
Cc: nd
Subject: [PATCH][AArch64] Improve SHA1 scheduling
SHA1H instructions may be scheduled after a SHA1C instruction
that uses the same input register. However SHA1C updates its input,
so if SHA1H is scheduled after it
ping
From: Wilco Dijkstra
Sent: 12 September 2016 15:50
To: Richard Earnshaw; GCC Patches
Cc: nd
Subject: Re: [PATCH v2][AArch64] Fix symbol offset limit
Wilco wrote:
> The original example is from GCC itself, the fixed_regs array is small but
> due to
> optimization we can end up
ping
From: Wilco Dijkstra
Sent: 31 October 2016 18:29
To: GCC Patches
Cc: nd
Subject: [RFC][PATCH][AArch64] Cleanup frame pointer usage
This patch cleans up all code related to the frame pointer. On AArch64 we
emit a frame chain even in cases where the frame pointer is not required.
So mak
On 11/11/2016 06:10 PM, Aldy Hernandez wrote:
The problem in this PR is that -fself-test is being run on a non empty
source file. This causes init_emit() to run, which sets:
REG_POINTER (virtual_incoming_args_rtx) = 1;
Setting REG_POINTER on the virtual incoming args, causes /f to be
print
-- Forwarded message --
From:
Date: Mon, Nov 14, 2016 at 8:15 AM
Subject: failure notice
To: howarth.at@gmail.com
Hi. This is the qmail-send program at sourceware.org.
I'm afraid I wasn't able to deliver your message to the following addresses.
This is a permanent error; I've
On Mon, Nov 14, 2016 at 02:18:02PM +0100, Bernd Schmidt wrote:
> On 11/11/2016 06:10 PM, Aldy Hernandez wrote:
> >The problem in this PR is that -fself-test is being run on a non empty
> >source file. This causes init_emit() to run, which sets:
> >
> >REG_POINTER (virtual_incoming_args_rtx) =
On 11/14/2016 02:20 PM, Jakub Jelinek wrote:
On Mon, Nov 14, 2016 at 02:18:02PM +0100, Bernd Schmidt wrote:
On 11/11/2016 06:10 PM, Aldy Hernandez wrote:
The problem in this PR is that -fself-test is being run on a non empty
source file. This causes init_emit() to run, which sets:
REG_POIN
Ping.
On 07/11/2016 16:59, Adhemerval Zanella wrote:
>
>
> On 14/10/2016 15:59, Wilco Dijkstra wrote:
>> Hi,
>>
>
> Thanks for the thoughtful review and sorry for late response.
>
>>> Split-stack prologue on function entry is as follow (this goes before the
>>> usual function prologue):
>>
>>
Richard,
In my previous patch I forgot to remove couple lines related to aux field.
Here is the correct updated patch.
Thanks.
Yuri.
2016-11-14 15:51 GMT+03:00 Richard Biener :
> On Fri, 11 Nov 2016, Yuri Rumyantsev wrote:
>
>> Richard,
>>
>> I prepare updated 3 patch with passing additional arg
On 20 October 2016 at 19:40, Jonathan Wakely wrote:
> On 20/10/16 10:33 -0700, Mike Stump wrote:
>>
>> On Oct 20, 2016, at 9:34 AM, Jonathan Wakely wrote:
>>>
>>>
>>> On 20/10/16 09:26 -0700, Mike Stump wrote:
On Oct 20, 2016, at 5:20 AM, Jonathan Wakely wrote:
>
>
> I am c
Ping!
On Thursday 10 November 2016 01:53 PM, Pitchumani Sivanupandi wrote:
On Wednesday 09 November 2016 08:05 PM, Georg-Johann Lay wrote:
On 09.11.2016 10:14, Pitchumani Sivanupandi wrote:
On Tuesday 08 November 2016 02:57 PM, Georg-Johann Lay wrote:
On 08.11.2016 08:08, Pitchumani Sivanupan
Hello world,
the attached patch runs through gfortran's AST to check for missing
location information.
If something is found, a warning is issued. I chose not to issue an
error because missing location information does not make a program
invalid. The patch would cause testsuite regressions for
On Mon, 14 Nov 2016, Yuri Rumyantsev wrote:
> Richard,
>
> In my previous patch I forgot to remove couple lines related to aux field.
> Here is the correct updated patch.
Yeah, I noticed. This patch would be ok for trunk (together with
necessary parts from 1 and 2) if all not required parts are
On 11/14/2016 09:09 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders
Hi,
Basically $subject which gets rid of a few more casts over all.
I ment to get this out a little while back, but life got busy, and I didn't
read the status announcement properly, so virtually working from hawai
the attached patch runs through gfortran's AST to check for missing
location information.
... this time with attachment.
2016-11-14 Thomas Koenig
PR fortran/78226
* error.c (gfc_warning_internal): New function.
* frontend-passes.c (CHECK_LOCUS): New macro.
On Fri, Nov 11, 2016 at 4:07 PM, Martin Liška wrote:
> On 11/11/2016 01:10 PM, Richard Biener wrote:
>> On Thu, Nov 10, 2016 at 4:36 PM, Martin Liška wrote:
>>> I've just noticed that tree-ssa-dse wrongly prints a new line to dump file.
>>> For the next stage1, I'll go through usages of print_gim
On 11/11/16 22:12, Joseph Myers wrote:
On Fri, 11 Nov 2016, Jiong Wang wrote:
There are two ways of introducing these AArch64 support:
* Introducing a few target macros so we can customize functions like
uw_init_context, uw_install_context etc.
* Use target private unwind-dw2 impleme
Hi,
Currently arm_feature_set is defined in gcc/config/arm/arm-flags as an array of
2 unsigned long. However, the flags stored in these two entries are (signed)
int, being combinations of bits set via expression of the form 1 << bitno. This
creates 3 issues:
1) undefined behavior when settin
I forgot to mention that this patch is needed for the optional -mthumb patch [1]
to bootstrap.
[1] https://gcc.gnu.org/ml/gcc-patches/2016-11/msg00735.html
Best regards,
Thomas
On 14/11/16 14:07, Thomas Preudhomme wrote:
Hi,
Currently arm_feature_set is defined in gcc/config/arm/arm-flags a
Hi Jack,
> > On darwin15, the proposed patch is insufficient to restore the bootstrap
> > (after running genfixes in the fixincludes directory) unless I also apply
> > the previously proposed change...
>
> no wonder: it's only been tested on darwin16. Care to explain what
> error you're seein
Wilco Dijkstra wrote:
> Richard Earnshaw wrote:
> On 10/11/16 17:19, Wilco Dijkstra wrote:
> Long conditional sequences are slow on modern cores - the value 6 for
> max_insns_skipped is a few decades out of date as it was meant for ARM2!
> Even with -Os the performance loss for larger values is no
On 11/11/2016 10:15 PM, David Malcolm wrote:
Link to earlier version of the patch:
https://gcc.gnu.org/ml/gcc-patches/2016-10/msg00278.html
Isn't this the same one?
Bernd
Hi Jack,
> In case it helps, attached is a diff of the os/trace.h header from OS X 11.11
> vs that
> from MacOS X 11.12. It looks like the context of the changes that you are
> trying to
> apply into are different between the two OS releases and are the cause of the
> build
> failures on darwin
On 11/11/16 15:31, Kyrill Tkachov wrote:
On 11/11/16 10:17, Kyrill Tkachov wrote:
On 10/11/16 23:39, Segher Boessenkool wrote:
On Thu, Nov 10, 2016 at 02:42:24PM -0800, Andrew Pinski wrote:
On Thu, Nov 10, 2016 at 6:25 AM, Kyrill Tkachov
I ran SPEC2006 on a Cortex-A72. Overall scores were
On Mon, Nov 14, 2016 at 6:04 AM, Jason Merrill wrote:
> On Wed, Nov 9, 2016 at 7:24 AM, Jakub Jelinek wrote:
>> The match.pd hunk is needed, otherwise the generic folding happily folds
>> int arr[2];
>> ...
>> auto [ x, y ] = arr;
>> &x == &arr[0]
>> into 0, because it thinks x and arr are distin
On Mon, 2016-11-14 at 15:17 +0100, Bernd Schmidt wrote:
> On 11/11/2016 10:15 PM, David Malcolm wrote:
> > Link to earlier version of the patch:
> > https://gcc.gnu.org/ml/gcc-patches/2016-10/msg00278.html
>
> Isn't this the same one?
>
It is; sorry. The rest of the patch kit has changed great
On Mon, Nov 14, 2016 at 12:04:24AM -0500, Jason Merrill wrote:
> On Wed, Nov 9, 2016 at 7:24 AM, Jakub Jelinek wrote:
> > The match.pd hunk is needed, otherwise the generic folding happily folds
> > int arr[2];
> > ...
> > auto [ x, y ] = arr;
> > &x == &arr[0]
> > into 0, because it thinks x and
On 14/11/16 14:07, Thomas Preudhomme wrote:
Hi,
Currently arm_feature_set is defined in gcc/config/arm/arm-flags as an array of 2 unsigned long. However, the flags stored in these two entries are (signed) int, being combinations of bits set via expression of the form 1 << bitno. This
creates 3
On Fri, 11 Nov 2016, Thomas Preudhomme wrote:
> Hi,
>
> To fix PR69714, code was added to disable bswap when the resulting symbolic
> expression (a load or load + byte swap) is smaller than the source expression
> (eg. some of the bytes accessed in the source code gets bitwise ANDed with 0).
> As
On Sun, Nov 13, 2016 at 11:53:10PM -0500, Jason Merrill wrote:
> On Wed, Nov 9, 2016 at 8:05 AM, Jakub Jelinek wrote:
> > On Wed, Nov 09, 2016 at 01:24:22PM +0100, Jakub Jelinek wrote:
> >> The following patch is a WIP on P0217R3 - decomposition declarations.
> >> It contains various FIXMEs, Jason
On Mon, Nov 14, 2016 at 12:29:31PM +0100, Janus Weil wrote:
> > After looking into this a little bit more, I found that the culprit
> > seems to be 'resolve_procedure_interface', which does not properly
> > copy the 'class_ok' attribute. I propose the attached patch to fix
> > this (regtesting righ
On Fri, Nov 11, 2016 at 10:15 PM, David Malcolm wrote:
> Changed in this version:
>
> * Rather than running just one pass, run *all* passes, but start at
> the given pass; support for "dg-do run" tests that execute the
> resulting code.
> * Updated test cases to new "compact" dump format; more
On Mon, Nov 14, 2016 at 9:51 AM, Jakub Jelinek wrote:
> On Sun, Nov 13, 2016 at 11:53:10PM -0500, Jason Merrill wrote:
>> On Wed, Nov 9, 2016 at 8:05 AM, Jakub Jelinek wrote:
>> > On Wed, Nov 09, 2016 at 01:24:22PM +0100, Jakub Jelinek wrote:
>> >> The following patch is a WIP on P0217R3 - decomp
On Mon, Nov 14, 2016 at 02:45:40PM +0100, Bernd Schmidt wrote:
> On 11/14/2016 09:09 AM, tbsaunde+...@tbsaunde.org wrote:
> > From: Trevor Saunders
> >
> > Hi,
> >
> > Basically $subject which gets rid of a few more casts over all.
> >
> > I ment to get this out a little while back, but life go
Richard,
I checked one of the tests designed for epilogue vectorization using
patches 1 - 3 and found out that build compiler performs vectorization
of epilogues with --param vect-epilogues-nomask=1 passed:
$ gcc -Ofast -mavx2 t1.c -S --param vect-epilogues-nomask=1 -o
t1.new-nomask.s -fdump-tree
Hi,
yesterday I forgot to send out the following patch. The test
libgomp/testsuite/libgomp.fortran/examples-4/device-3.f90 was failing
for me when I was testing the HSA branch merge but I believe the test
itself is wrong and the failure is due to us now adhering to OpenMP
4.5 default mapping of s
No functional changes.
2016-11-14 Uros Bizjak
* config/i386/i386.md (*andndi3_doubleword): Merge operand constraints.
(*ashl3_doubleword): Ditto.
Bootstrapped and regression tested on x86_64-linux-gnu, committed to
mainline SVN.
Uros.
diff --git a/gcc/config/i386/i386.md b/gcc/config
The attached patch fixes PR sanitizer/78267 by conditionalizing the
include of on the compiler defining __BLOCKS__ as a
supported extension. Passes bootstrap on x86_64-apple-darwin15. Okay
for gcc trunk?
2016-11-14 Jack Howarth
libsanitizer/
PR sanitizer/78267
* sanitizer_comm
> On 14 Nov 2016, at 16:57, Jack Howarth wrote:
>
> The attached patch fixes PR sanitizer/78267 by conditionalizing the
> include of on the compiler defining __BLOCKS__ as a
> supported extension. Passes bootstrap on x86_64-apple-darwin15. Okay
> for gcc trunk?
>
Rainer has that PR assigned a
Hi,
On 14 November 2016 at 15:07, Thomas Preudhomme
wrote:
> Hi,
>
> Currently arm_feature_set is defined in gcc/config/arm/arm-flags as an array
> of 2 unsigned long. However, the flags stored in these two entries are
> (signed) int, being combinations of bits set via expression of the form 1 <
The testcase powerpc/fusion3.c causes an ICE when compiled with
-msoft-float.
The key line in the testcase looks fairly harmless:
void fusion_float_write (float *p, float f){ p[LARGE] = f; }
The error message look like this:
.../gcc.target/powerpc/fusion3.c: In function 'fusion_float_write
On Mon, 14 Nov 2016, Martin Jambor wrote:
> Hi,
>
> yesterday I forgot to send out the following patch. The test
> libgomp/testsuite/libgomp.fortran/examples-4/device-3.f90 was failing
> for me when I was testing the HSA branch merge but I believe the test
> itself is wrong and the failure is
2016-11-14 16:10 GMT+01:00 Steve Kargl :
> On Mon, Nov 14, 2016 at 12:29:31PM +0100, Janus Weil wrote:
>> > After looking into this a little bit more, I found that the culprit
>> > seems to be 'resolve_procedure_interface', which does not properly
>> > copy the 'class_ok' attribute. I propose the a
On Mon, Nov 14, 2016 at 06:21:06PM +0100, Janus Weil wrote:
> 2016-11-14 16:10 GMT+01:00 Steve Kargl :
> > On Mon, Nov 14, 2016 at 12:29:31PM +0100, Janus Weil wrote:
> >> > After looking into this a little bit more, I found that the culprit
> >> > seems to be 'resolve_procedure_interface', which d
On Oct 21, 2016, at 1:00 AM, Christophe Lyon wrote:
>
> So if we say that the current behaviour has to keep being the default,
> so that users think about what they are really doing,
Having a toolchain not work by default to force users to think, isn't a winning
strategy.
Everything should al
On November 14, 2016 4:39:40 PM GMT+01:00, Yuri Rumyantsev
wrote:
>Richard,
>
>I checked one of the tests designed for epilogue vectorization using
>patches 1 - 3 and found out that build compiler performs vectorization
>of epilogues with --param vect-epilogues-nomask=1 passed:
>
>$ gcc -Ofast -m
On Nov 14, 2016, at 8:59 AM, Iain Sandoe wrote:
>
>> On 14 Nov 2016, at 16:57, Jack Howarth wrote:
>>
>> The attached patch fixes PR sanitizer/78267 by conditionalizing the
>> include of on the compiler defining __BLOCKS__ as a
>> supported extension. Passes bootstrap on x86_64-apple-darwin15.
Hi Mike,
> On Nov 14, 2016, at 8:59 AM, Iain Sandoe wrote:
>>
>>> On 14 Nov 2016, at 16:57, Jack Howarth wrote:
>>>
>>> The attached patch fixes PR sanitizer/78267 by conditionalizing the
>>> include of on the compiler defining __BLOCKS__ as a
>>> supported extension. Passes bootstrap on x86_
Hello world,
I have just committed the attached patch as obvious. No test case,
the function is only supposed to be called when debugging the
compiler itself.
Regards
Thomas
2016-11-14 Thomas Koenig
* dump-parse-tree.c (show_code): Add prototype.
(gfc_debug_code):
On Sun, Nov 6, 2016 at 10:03 AM, Mark Wielaard wrote:
>
> We now have -Wshadow=local. So the attached patch uses that for
> libiberty. Is the attached patch OK to commit?
This is OK.
Thanks.
Ian
2016-11-11 14:16 GMT+03:00 Uros Bizjak :
> The x86 part of the patch is OK with the above changes and additional
> target attribute test for flags2 ISA features..
Fixed according your comments, I will followup with additional tests soon.
--
WBR,
Andrew
new_avx512_instructions_14.11.patch
Descr
2016-11-11 14:29 GMT+03:00 Jakub Jelinek :
> Hi!
>
> I've noticed preexisting:
>
> On Thu, Nov 10, 2016 at 07:27:00PM +0300, Andrew Senkevich wrote:
>
>> --- a/gcc/config/i386/i386-modes.def
>> +++ b/gcc/config/i386/i386-modes.def
>> @@ -84,6 +84,7 @@ VECTOR_MODES (FLOAT, 16); /* V8HF V
Hi Christophe,
No, they were seen when bootstrapping on arm-linux-gnueabihf with the patch to
to make -mthumb optional. The patch store flags in an arm_feature_set array and
GCC -Wnarrowing complained about the difference of type.
Best regards,
Thomas
On 14/11/16 17:00, Christophe Lyon wrot
-7-20161114/libsanitizer/sanitizer_common -I.. -I
../../../../gcc-7-20161114/libsanitizer/include -isystem
../../../../gcc-7-20161114/libsanitizer/include/system -Wall -W
-Wno-unused-parameter -Wwrite-strings -pedantic -Wno-long-long -fPIC
-fno-builtin -fno-exceptions -fno-rtti -fomit-frame-pointer
My apologize, I realized when trying to apply the patch that I wrote it on top
of the optional -mthumb patch instead of the reverse. I'll rebase it to not
screw up bisect.
Best regards,
Thomas
On 14/11/16 14:47, Kyrill Tkachov wrote:
On 14/11/16 14:07, Thomas Preudhomme wrote:
Hi,
Current
On 14 November 2016 at 18:54, Mike Stump wrote:
> On Oct 21, 2016, at 1:00 AM, Christophe Lyon
> wrote:
>>
>> So if we say that the current behaviour has to keep being the default,
>> so that users think about what they are really doing,
>
> Having a toolchain not work by default to force users
Thanks for doing this.
tbsaunde+...@tbsaunde.org writes:
> diff --git a/gcc/genrecog.c b/gcc/genrecog.c
> index a8e8c22..aa7f629 100644
> --- a/gcc/genrecog.c
> +++ b/gcc/genrecog.c
> @@ -5102,8 +5102,7 @@ print_subroutine (output_state *os, state *s, int
> proc_id)
>/* For now, the top-level
On 11/13/16, Martin Sebor wrote:
> Bug 77531 requests a new warning for calls to allocation functions
> (those declared with attribute alloc_size(X, Y)) that overflow the
> computation X * Z of the size of the allocated object.
>
> Bug 78284 suggests that detecting and diagnosing other common erro
Any feedback regarding this patch ?
François
On 02/11/2016 22:37, François Dumont wrote:
Hi
Here is an updated proposal, I realized that the newly introduced
_M_move_data can be also used in the swap implementation.
Let me know if you prefer without it or not.
François
On 28/10/2
1 - 100 of 115 matches
Mail list logo
