The following fixes an oversight when computing alignment in the
vectorizer.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2016-11-07 Richard Biener
PR tree-optimization/78189
* tree-vect-data-refs.c (vect_compute_data_ref_alignment): Fix
Hi,
this patch set performs libsanitizer merge from upstream.
Patch 1 is the library merge itself.
Patch 2 is the reapplied change for SPARC by David S. Miller.
Patch 3 changes heuristic for extracting last PC from stack frame for
ARM in fast unwind routine. More details can be found here
(h
This is just reapplied patch for SPARC by David S. Miller.
From 0ff8d1c408b076970c323361922c35033aaae245 Mon Sep 17 00:00:00 2001
From: Maxim Ostapenko
Date: Tue, 25 Oct 2016 20:00:43 +0300
Subject: [PATCH 2/7] libsanitizer/
PR sanitizer/63958
Reapply:
2014-10-14 David S. Miller
* saniti
This patch adjusts the fix for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61771 to extract the last
PC from the stack frame if no valid FP is available for ARM.
From 6dc6e4f761080cf19a161fb0e27c1fd584688f40 Mon Sep 17 00:00:00 2001
From: Maxim Ostapenko
Date: Tue, 25 Oct 2016 20:27:37 +0300
Su
This is rewritten Jakub's fix for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888. Upstream now
supports new approach for ODR violation detection: compiler emits new
__odr_asan_XXX symbol for each instrumented global that indicates
whether this global was already registered and the library c
This patch just combines minimal necessary changes to support new
libasan ABI. This patch doesn't try to implement odr indicators at
compiler part, it simply pass a zero stub to runtime. The actual
implementation of odr indicators goes in patch 7.
From 33f6f98faa86c61b9895db0d71e0e88a9ae4fa59 Mo
This patch just adds several tests backported from upstream.
From b4677ed64e7aee1af7772750e0b18ed8271f4757 Mon Sep 17 00:00:00 2001
From: Maxim Ostapenko
Date: Tue, 1 Nov 2016 16:52:13 +0300
Subject: [PATCH 6/7] Backport several testcases for ASan from upstream.
gcc/
* asan.h (asan_intercepted
This patch tries to implement odr indicators functionality at compiler
side.
We emit new __odr_asan_XXX symbol for each instrumented global that
indicates whether this global was already registered and the library
checks this indicator symbol at runtime. For some globals (e.g. static
or hidden)
On Mon, Nov 07, 2016 at 11:22:28AM +0300, Maxim Ostapenko wrote:
> this patch set performs libsanitizer merge from upstream.
>
> Patch 1 is the library merge itself.
>
> Patch 2 is the reapplied change for SPARC by David S. Miller.
>
> Patch 3 changes heuristic for extracting last PC from stack
On Mon, Nov 07, 2016 at 11:31:18AM +0300, Maxim Ostapenko wrote:
> --- a/gcc/asan.c
> +++ b/gcc/asan.c
> @@ -1329,6 +1329,16 @@ asan_needs_local_alias (tree decl)
>return DECL_WEAK (decl) || !targetm.binds_local_p (decl);
> }
>
> +/* Return true if DECL, a global var, is an artificial ODR in
On 11/05/2016 09:38 AM, Jan Hubicka wrote:
> Looks OK if it passes.
>
> Honza
Thanks, fixed on trunk as r241894.
Martin
Hi Christophe,
On Fri, Nov 04, 2016 at 02:31:28PM +0100, Christophe Lyon wrote:
> Since this commit I have noticed execution failures on "old" arm targets:
>
> gcc.dg/torture/pr48124-4.c -O1 execution test
> gcc.dg/torture/pr48124-4.c -O2 execution test
> gcc.dg/torture/pr48124-4.c
On 07/11/16 11:39, Jakub Jelinek wrote:
On Mon, Nov 07, 2016 at 11:22:28AM +0300, Maxim Ostapenko wrote:
this patch set performs libsanitizer merge from upstream.
Patch 1 is the library merge itself.
Patch 2 is the reapplied change for SPARC by David S. Miller.
Patch 3 changes heuristic for
On Mon, Nov 07, 2016 at 12:14:39PM +0300, Maxim Ostapenko wrote:
> libubsan is definitely compatible.
Nice.
> For libtsan we have several changes:
>
> 1) Several interceptors (34 of them) were added and __interceptor_lstat{64}
> were removed.
That is bad, I think we need to readd those and perh
On Mon, Nov 7, 2016 at 9:20 AM, Jakub Jelinek wrote:
> On Mon, Nov 07, 2016 at 12:14:39PM +0300, Maxim Ostapenko wrote:
>> libubsan is definitely compatible.
>
> Nice.
>
>> For libtsan we have several changes:
>>
>> 1) Several interceptors (34 of them) were added and __interceptor_lstat{64}
>> wer
We transform floating point divide instructions to a faster series of
simple instructions, "swdiv". Currently we do not do that until the
first splitter pass, which is much too late for most optimisations
that can happen on those new instructions, e.g. the constant loads
are not CSEd inside an unr
On 07/11/16 12:28, Yuri Gribov wrote:
On Mon, Nov 7, 2016 at 9:20 AM, Jakub Jelinek wrote:
On Mon, Nov 07, 2016 at 12:14:39PM +0300, Maxim Ostapenko wrote:
libubsan is definitely compatible.
Nice.
For libtsan we have several changes:
1) Several interceptors (34 of them) were added and __in
> Now you don't need to have a special pool to call create_TOC_reference, you
> can call it for regular TOC references as well, as done a few lines above:
>
> /* If this is a SYMBOL_REF that refers to a constant pool entry,
>and we have put it in the TOC, we just need to make a TOC-r
On Sat, Nov 5, 2016 at 3:25 AM, Jeff Law wrote:
> On 11/04/2016 06:26 PM, Martin Sebor wrote:
>>
>> While experimenting with -Walloca and cross-referencing the manual
>> I noticed a few minor nits that I thought could stand to corrected
>> and/or clarified. Attached is a patch.
>>
>> In the updat
On Fri, Nov 4, 2016 at 9:07 PM, Marc Glisse wrote:
> Hello,
>
> since we were discussing this recently...
>
> The condition is copied from the existing 0 % X case, visible in the context
> of the diff.
>
> As far as I understand, the main case where we do not want to optimize is
> during constexpr
On Sat, Nov 5, 2016 at 3:30 AM, Jeff Law wrote:
> On 11/04/2016 02:07 PM, Marc Glisse wrote:
>>
>> Hello,
>>
>> since we were discussing this recently...
>>
>> The condition is copied from the existing 0 % X case, visible in the
>> context of the diff.
>>
>> As far as I understand, the main case w
On 07/11/16 12:20, Jakub Jelinek wrote:
On Mon, Nov 07, 2016 at 12:14:39PM +0300, Maxim Ostapenko wrote:
libubsan is definitely compatible.
Nice.
For libtsan we have several changes:
1) Several interceptors (34 of them) were added and __interceptor_lstat{64}
were removed.
That is bad, I thi
Hello.
After discussion with Jakub, I'm resending new version of the patch, where I
changed following:
1) gimplify_ctxp->live_switch_vars is used to track variables introduced in
switch_expr. Every time
a case_label_expr is seen, these are unpoisoned. It's quite conservative,
however it cove
On Mon, Nov 07, 2016 at 11:22:28AM +0300, Maxim Ostapenko wrote:
> Hi,
>
> this patch set performs libsanitizer merge from upstream.
>
> Patch 1 is the library merge itself.
>
> Patch 2 is the reapplied change for SPARC by David S. Miller.
>
> Patch 3 changes heuristic for extracting last PC fr
Third version of the patch.
Martin
>From e790d926afd3d2d6ad41d14d1e91698bf651b41a Mon Sep 17 00:00:00 2001
From: marxin
Date: Mon, 19 Sep 2016 17:39:29 +0200
Subject: [PATCH 2/2] Introduce tests for -fsanitize-address-use-after-scope
gcc/testsuite/ChangeLog:
2016-09-26 Martin Liska
* c-c++
On Sun, Nov 6, 2016 at 2:36 PM, Jack Howarth wrote:
> The use of an Apple sandbox with denied file access permissions into
> /usr/local
> exposed that cc1 fails on errors of...
>
> cc1: error: /usr/local/include: Operation not permitted
>
> The commonly suggested solution of using --with-local-pr
On Mon, Nov 07, 2016 at 11:03:11AM +0100, Martin Liška wrote:
> Hello.
>
> After discussion with Jakub, I'm resending new version of the patch, where I
> changed following:
> 1) gimplify_ctxp->live_switch_vars is used to track variables introduced in
> switch_expr. Every time
>a case_label_e
On Fri, 4 Nov 2016, Prathamesh Kulkarni wrote:
> On 4 November 2016 at 13:41, Richard Biener wrote:
> > On Thu, 3 Nov 2016, Marc Glisse wrote:
> >
> >> On Thu, 3 Nov 2016, Richard Biener wrote:
> >>
> >> > > > > The transform would also work for vectors (element_precision for
> >> > > > > the tes
On Mon, Nov 07, 2016 at 11:04:23AM +0100, Martin Liška wrote:
> Third version of the patch.
>
> Martin
> >From e790d926afd3d2d6ad41d14d1e91698bf651b41a Mon Sep 17 00:00:00 2001
> From: marxin
> Date: Mon, 19 Sep 2016 17:39:29 +0200
> Subject: [PATCH 2/2] Introduce tests for -fsanitize-address-us
On Fri, 4 Nov 2016, Jakub Jelinek wrote:
> Hi!
>
> Just 2 nits:
>
> On Fri, Oct 28, 2016 at 01:46:57PM +0200, Richard Biener wrote:
> > +/* Return a pointer to the Nth token in PARERs tokens_buf. */
>
> PARSERs ?
Fixed.
> > @@ -454,7 +423,7 @@ c_lex_one_token (c_parser *parser, c_token *toke
On Mon, 7 Nov 2016, Richard Biener wrote:
> On Fri, 4 Nov 2016, Jakub Jelinek wrote:
>
> > Hi!
> >
> > Just 2 nits:
> >
> > On Fri, Oct 28, 2016 at 01:46:57PM +0200, Richard Biener wrote:
> > > +/* Return a pointer to the Nth token in PARERs tokens_buf. */
> >
> > PARSERs ?
>
> Fixed.
>
> >
On 07/11/16 13:04, Jakub Jelinek wrote:
On Mon, Nov 07, 2016 at 11:22:28AM +0300, Maxim Ostapenko wrote:
Hi,
this patch set performs libsanitizer merge from upstream.
Patch 1 is the library merge itself.
Patch 2 is the reapplied change for SPARC by David S. Miller.
Patch 3 changes heuristic
It's a missed optimization of a redundant zero-extension on the SPARC, which
originally comes from PR rtl-optimization/58295 for ARM. The extension is
eliminated on the ARM because the load is explicitly zero-extended in RTL;
on the SPARC the load is implicitly zero-extended by means of LOAD_EXT
On Fri, Oct 21, 2016 at 05:31:14PM +0100, James Greenhalgh wrote:
> On Wed, Oct 12, 2016 at 04:56:52PM +0100, James Greenhalgh wrote:
> > On Wed, Sep 28, 2016 at 05:17:14PM +0100, James Greenhalgh wrote:
> > > On Wed, Sep 21, 2016 at 10:42:03AM +0100, James Greenhalgh wrote:
> > > > On Tue, Sep 13,
Ping.
Thanks,
Kyrill
On 31/10/16 12:10, Kyrill Tkachov wrote:
Ping.
Thanks,
Kyrill
On 24/10/16 14:12, Kyrill Tkachov wrote:
On 24/10/16 12:29, Kyrill Tkachov wrote:
Ping.
https://gcc.gnu.org/ml/gcc-patches/2016-10/msg01321.html
I just noticed my original ChangeLog entry was truncated.
I
On Mon, Oct 17, 2016 at 05:15:21PM +0100, Kyrill Tkachov wrote:
> Hi all,
>
> For the attached testcase the code ends up trying to extract bits outside the
> range of the normal register widths. The aarch64 patterns for ubfz and tbnz
> end up accepting such operands and emitting invalid assembly
>
The following fixes phiopt to not introduce undefined behavior
in its abs replacement code in case we negate only positive values
in the original code.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2016-11-07 Richard Biener
PR tree-optimization/7822
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2016-11-07 Richard Biener
PR tree-optimization/78218
* gimple-ssa-store-merging.c
(pass_store_merging::terminate_all_aliasing_chains):
Drop unused argument, fix alias check to also consider
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk
and branch.
Richard.
2016-11-07 Richard Biener
PR target/78229
* config/i386/i386.c (ix86_gimple_fold_builtin): Do not adjust
EH info.
* g++.dg/pr78229.C: New testcase.
Index: gcc/config/i
On Mon, Nov 07, 2016 at 01:17:25PM +0100, Richard Biener wrote:
>
> The following fixes phiopt to not introduce undefined behavior
> in its abs replacement code in case we negate only positive values
> in the original code.
>
> Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk
On Mon, 7 Nov 2016, Jakub Jelinek wrote:
> On Mon, Nov 07, 2016 at 01:17:25PM +0100, Richard Biener wrote:
> >
> > The following fixes phiopt to not introduce undefined behavior
> > in its abs replacement code in case we negate only positive values
> > in the original code.
> >
> > Bootstrapped
The following moves a overly conservative check that we do not access
excess elements when vectorizing a BB to a place where we can do
a better job with respect to the elements we actually use.
This means that for the included testcase we are not confused
by the read from c[4] but just do not vec
This patch adds a new command line option -mabsdata which can be ised to set
attribute absdata for all data in static storage so it can be accessed by LDS
and STS instructions.
This is only useful for some reduced Tiny devices like ATtiny40.
For other reduced Tiny where all of SRAM fits LDS /
On 7 November 2016 at 10:14, Segher Boessenkool
wrote:
> Hi Christophe,
>
> On Fri, Nov 04, 2016 at 02:31:28PM +0100, Christophe Lyon wrote:
>> Since this commit I have noticed execution failures on "old" arm targets:
>>
>> gcc.dg/torture/pr48124-4.c -O1 execution test
>> gcc.dg/torture/pr4
Hi!
The following testcases ICE at -O0, because ix86_expand_sse_cmp avoid using
the passed in dest only if optimize or if there is some value overlap, but
we actually need to do that also if we have a maskcmp where we want to use
a different mode than dest has.
Bootstrapped/regtested on x86_64-lin
On 07.11.2016 13:54, Georg-Johann Lay wrote:
This patch adds a new command line option -mabsdata which can be ised to set
attribute absdata for all data in static storage so it can be accessed by LDS
and STS instructions.
This is only useful for some reduced Tiny devices like ATtiny40.
For othe
Hi!
Richard noticed 3 misplaced tests - C++ tests don't belong into
gcc.target/ which tests just C.
I've bootstrapped/regtested this on x86_64-linux and i686-linux and
committed to trunk as obvious.
2016-11-07 Jakub Jelinek
PR middle-end/71529
* gcc.target/i386/pr71529.C: Mov
On Mon, Nov 07, 2016 at 02:00:46PM +0100, Christophe Lyon wrote:
> > Confirmed. What a nasty, nasty bug, and it has been here for decades
> > it seems. Could you please open a PR?
> >
> Sure, I've created PR78232 for this.
Thanks! I have a patch btw, it's regstrapping. Not sure it is fully
cor
On Fri, Nov 04, 2016 at 08:07:37PM +0100, Richard Biener wrote:
> >If/once this is in, I'm planning to test/submit a patch adding
> > /* If one decl is known to be a function or label in a function and
> > the other is some kind of data, they can't overlap. */
> > if ((TREE_CODE (exprx) == F
On Mon, Nov 7, 2016 at 4:32 AM, Segher Boessenkool
wrote:
> We transform floating point divide instructions to a faster series of
> simple instructions, "swdiv". Currently we do not do that until the
> first splitter pass, which is much too late for most optimisations
> that can happen on those n
On Mon, 7 Nov 2016, Jakub Jelinek wrote:
> On Fri, Nov 04, 2016 at 08:07:37PM +0100, Richard Biener wrote:
> > >If/once this is in, I'm planning to test/submit a patch adding
> > > /* If one decl is known to be a function or label in a function and
> > > the other is some kind of data, they c
This patch contains an implementation of search_line_fast for the CPP
lexer. It's based in part on the AArch32 (ARM) code but incorporates
new instructions available in AArch64 (reduction add operations) plus
some tricks for reducing the realignment overheads. We assume a page
size of 4k, but tha
On 03/11/16 21:54 +0100, François Dumont wrote:
Hi
I might not be the right one to propose this patch as I am not
sure that I fully understand gnu-versioned-namespace.ver organization.
But with it following test failures when using versioned namespace
vanish:
FAIL: 20_util/allocator/over
Hi all,
This patch (1 of 3) adds the following NEON intrinsics
to the Aarch64 back-end of GCC:
* vsli_n_p64
* vsliq_n_p64
* vld1_p64
* vld1q_p64
* vld1_dup_p64
* vld1q_dup_p64
* vst1_p64
* vst1q_p64
* vld2_p64
* vld3_p64
* vld4_p64
* vld2q_p64
* vld3q_p64
* vld4q_p64
* vld2_dup_p64
* vld3_d
Hi all,
This patch (3 of 3) adds updates tests for the NEON intrinsics
added by the previous patches:
Ran regression tests on aarch64-none-linux-gnu
and on arm-none-linux-gnueabihf.
Ok for trunk?
Thanks,
Tamar
gcc/testsuite/
2016-11-04 Tamar Christina
* gcc.target/aarch64/advsimd-
Hi all,
This patch (2 of 3) adds the following NEON intrinsics to
the ARM back-end of GCC:
* vget_lane_p64
Added new tests for these and ran regression tests on aarch64-none-linux-gnu
and on arm-none-linux-gnueabihf.
Ok for trunk?
Thanks,
Tamar
gcc/
2016-11-04 Tamar Christina
* co
The following fixes an ICE with call cdce where it fails to handle
PHIs in the fallthru destination of a call with EH. My simple fix is
to simply split the fallthru edge if the dest may contain PHI nodes.
This may also remove the need to free dominance info (hope there's
a testcase for that -- I
What about ARM maintainers?
Best regards,
Thomas
On 04/11/16 22:16, Gerald Pfeifer wrote:
On Fri, 4 Nov 2016, Thomas Preudhomme wrote:
This patch document the newly added support in GCC 7 for Cortex-M23 and
Cortex-M33 processors [1][2].
:
Is this ok for ?
Surely so for me.
Gerald
On 07/11/16 14:00, Thomas Preudhomme wrote:
What about ARM maintainers?
Fine with me too.
Thanks,
Kyrill
Best regards,
Thomas
On 04/11/16 22:16, Gerald Pfeifer wrote:
On Fri, 4 Nov 2016, Thomas Preudhomme wrote:
This patch document the newly added support in GCC 7 for Cortex-M23 and
Cor
Hi all,
This fixes (PR78142) by turning off scheduling for the test.
r241590 is causing more registers to be used and so
the SP registered happens to be picked and used.
This test I believe was checking explicitly that the
SP is not used if not needed.
Ran regression tests on aarch64-none-linu
Ping.
From: gcc-patches-ow...@gcc.gnu.org on behalf
of Tamar Christina
Sent: Tuesday, November 1, 2016 3:46:07 PM
To: GCC Patches; r...@cebitec.uni-bielefeld.de; mikest...@comcast.net
Cc: nd
Subject: [PATCH][GCC/TESTSUITE] Make test for traditional-cpp d
On 10/14/2016 08:28 PM, Bernd Schmidt wrote:
On 10/12/2016 09:27 PM, Denys Vlasenko wrote:
Yes, something like "if max_skip >= func_size, temporarily lower
max_skip to func_size-1" (because otherwise we can create padding
bigger-or-equal to the entire function in size, which is stupid
- it's bet
Hi all,
The changes in r240999 re-arranged includes and
left out signal.h for Windows x86 builds.
This breaks the build and prevents GCC builds from
completing with messages such as:
adaint.c:3317:19: error: 'SIGINT' undeclared (first use in this function); did
you mean 'SAIT'?
else if (si
On Sun, Nov 6, 2016 at 5:32 AM, Aaron Sawdey
wrote:
> On Fri, 2016-11-04 at 20:43 -0600, Jeff Law wrote:
>> So what's the motivation here? When we don't have any constants
>> then
>> I'd think we'd be better off punting into the library.
>
> When none of the args to strncmp are constant, I'd be i
Currently we force peeling for gaps whenever element overrun can occur
but for aligned accesses we know that the loads won't trap and thus
we can avoid this.
Bootstrap and regtest running on x86_64-unknown-linux-gnu (I expect
some testsuite fallout here so didn't bother to invent a new testcase).
On Mon, Nov 7, 2016 at 2:02 PM, Jakub Jelinek wrote:
> Hi!
>
> The following testcases ICE at -O0, because ix86_expand_sse_cmp avoid using
> the passed in dest only if optimize or if there is some value overlap, but
> we actually need to do that also if we have a maskcmp where we want to use
> a d
On Fri, Oct 28, 2016 at 09:12:29AM -0600, Jeff Law wrote:
>
> The PPC port is stumbling over the new integer in boolean context
warnings.
>
> In particular this code from rs6000_option_override_internal is
> problematical:
>
> HOST_WIDE_INT flags = ((TARGET_DEFAULT) ? TARGET_DEFAULT
On 11/07/2016 03:02 AM, Richard Biener wrote:
On Sat, Nov 5, 2016 at 3:30 AM, Jeff Law wrote:
On 11/04/2016 02:07 PM, Marc Glisse wrote:
Hello,
since we were discussing this recently...
The condition is copied from the existing 0 % X case, visible in the
context of the diff.
As far as I un
On Fri, 2016-11-04 at 20:40 +0100, Bernd Schmidt wrote:
> On 11/04/2016 08:25 PM, David Malcolm wrote:
>
> > return m_compact;
>
> Ok with this one plus a comment.
>
Thanks.
Using m_compact required turning the static function into a (private)
member function. For reference, here's
Hello world,
I have committed the little patchlet below as obvious, after
regression-testing.
Regards
Thomas
2016-11-07 Thomas Koenig
PR fortran/78826
* match.c (gfc_match_select_type): Add where for expr1.
* resolve.c (resolev_select_type): Add where for e
Am 07.11.2016 um 16:25 schrieb Thomas Koenig:
PR fortran/78826
... should have been PR 78226.
Ping.
https://gcc.gnu.org/ml/gcc-patches/2016-10/msg02525.html
On Mon, Oct 31, 2016 at 08:56:10PM +0100, Dominik Vogt wrote:
> The attached patch does a little change in
> combine.c:combine_simplify_rtx() to prevent a "simplification"
> where the rtl code gets more complex in reality. The comple
On Mon, 2016-11-07 at 11:03 +0100, Martin Liška wrote:
> Hello.
>
> After discussion with Jakub, I'm resending new version of the patch,
> where I changed following:
> 1) gimplify_ctxp->live_switch_vars is used to track variables
> introduced in switch_expr. Every time
>a case_label_expr is se
On 11/04/2016 05:35 PM, Jakub Jelinek wrote:
2016-11-04 Jakub Jelinek
PR target/77834
* dse.c (dse_step5): Call scan_reads even if just
insn_info->frame_read. Improve and fix dump file messages.
Sounds reasonable, and I checked and it seems not to change code
gene
On Mon, Nov 07, 2016 at 11:07:13AM -0500, David Malcolm wrote:
> The patch (r241896) introduced an error in the build of the jit:
>
> ../../src/gcc/jit/jit-builtins.c:62:1: error: invalid conversion from
> ‘int’ to ‘gcc::jit::built_in_attribute’ [-fpermissive]
> };
> ^
>
> which seems to be due
Hi,
The (HAS_MOVN) option should cause an upgrade to MIPS IV if the target is
pre-MIPS IV. However, the upgrade condition checks for "$isa < 3", which means
that we won't upgrade if we're targeting MIPS III.
This results in failures for the movcc-{1,2,3}.c and branch-cost-2.c tests
when the targe
On 7 November 2016 at 09:58, Martin Liška wrote:
> On 11/05/2016 09:38 AM, Jan Hubicka wrote:
>> Looks OK if it passes.
>>
>> Honza
>
> Thanks, fixed on trunk as r241894.
> Martin
Thanks, this fixed the problems I reported.
Christophe
On Mon, 2016-11-07 at 15:26 +0100, Richard Biener wrote:
> Your patchset doesn't contain a testcase so I really wonder which
> case
> we know the string length but it is not constant.
>
> Yes, there's COND_EXPR handling in c_strlen but that should be mostly
> dead code -- the real code should be u
On 11/03/2016 03:00 PM, Eric Botcazou wrote:
FWIW here's a more complete version of my patch which I'm currently
testing. Let me know if you think it's at least a good enough
intermediate step to be installed.
It is, thanks.
Testing showed the same issue as Jiong found, so I've committed it w
On 07/11/16 17:04, Bernd Schmidt wrote:
On 11/03/2016 03:00 PM, Eric Botcazou wrote:
FWIW here's a more complete version of my patch which I'm currently
testing. Let me know if you think it's at least a good enough
intermediate step to be installed.
It is, thanks.
Testing showed the same i
On 7 November 2016 at 15:43, Richard Biener wrote:
> On Fri, 4 Nov 2016, Prathamesh Kulkarni wrote:
>
>> On 4 November 2016 at 13:41, Richard Biener wrote:
>> > On Thu, 3 Nov 2016, Marc Glisse wrote:
>> >
>> >> On Thu, 3 Nov 2016, Richard Biener wrote:
>> >>
>> >> > > > > The transform would also
On 7 November 2016 at 23:06, Prathamesh Kulkarni
wrote:
> On 7 November 2016 at 15:43, Richard Biener wrote:
>> On Fri, 4 Nov 2016, Prathamesh Kulkarni wrote:
>>
>>> On 4 November 2016 at 13:41, Richard Biener wrote:
>>> > On Thu, 3 Nov 2016, Marc Glisse wrote:
>>> >
>>> >> On Thu, 3 Nov 2016, R
On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote:
> This is an initial patch in a series that converts Darwin's configury to
> detect ld64 features, rather than the current process of hard-coding them on
> target system version.
So, I really do hate to ask, but does this have to be a config optio
On Nov 1, 2016, at 8:46 AM, Tamar Christina wrote:
>
> A glibc update recently broke this test by adding a CPP
> macro that uses the ## string function which traditional-cpp
> does not support.
> The change in glibc that made the test fail is from
> 6962682ffe5e5f0373047a0b894fee7a774be254.
>
>
> On 7 Nov 2016, at 09:51, Mike Stump wrote:
>
> [ possible dup ]
>
>> Begin forwarded message:
>>
>> From: Mike Stump
>> Subject: Re: [PATCH fix PR71767 2/4 : Darwin configury] Arrange for ld64 to
>> be detected as Darwin's linker
>> Date: November 7, 2016 at 9:48:53 AM PST
>> To: Iain Sand
On Nov 6, 2016, at 11:37 AM, Iain Sandoe wrote:
> OK for trunk?
> OK for open branches?
Ok.
> 2016-11-06 Iain Sandoe
>
> PR target/71767
> * config/darwin.c (imachopic_indirection_name): Make data section
> indirections
> linker-visible.
> * config/darwin.h (ASM_GENE
On Nov 6, 2016, at 11:41 AM, Iain Sandoe wrote:
> OK for trunk (after the relevant patches are applied)?
> OK for open branches (likewise)?
Ok.
> PR target/71767
>
> * g++.dg/abi/key2.C: Adjust for changed Darwin sections and
> linker-visible symbols.
> * g++.dg/torture/darwi
On Nov 6, 2016, at 11:40 AM, Iain Sandoe wrote:
>
> OK for trunk?
> OK for open branches?
Ok.
> 2016-11-06 Iain Sandoe
>
> PR target/71767
> * config/darwin-sections.def (picbase_thunk_section): New.
> * config/darwin.c (darwin_init_sections): Set up picbase thunk section.
On Nov 6, 2016, at 12:53 PM, Iain Sandoe wrote:
> OK for trunk?
> OK for open branches?
Ok.
> 2016-11-06 Iain Sandoe
> Rainer Orth
>
> target/PR67710
> * config.in: Regenerate
> * config/darwin-driver.c (darwin_driver_init): Emit a version string
> for the assemble
On Nov 6, 2016, at 12:13 PM, Iain Sandoe wrote:
>
> OK for trunk?
> OK for open branches?
For the darwin parts, Ok.
> 2016-11-06 Iain Sandoe
>
> PR target/57438
> * config/i386/i386.c (ix86_code_end): Note that we emitted code where
> the
> function might otherwise appear
On Sun, 6 Nov 2016, Iain Sandoe wrote:
> This adds an option --with-ld64[=version] that allows the configurer to
New configure options should be documented in install.texi.
--
Joseph S. Myers
jos...@codesourcery.com
Hi,
when looking at stuff to merge to trunk, I have found out that this
patch has slipped thorough the cracks. It adds the UID to names of
private symbols so that variables with the same name but different
scope, particularly OpenMP re-mapped ones, do not clash.
Committed to the hsa branch, will
Hi,
this is another simple cleanup that I forgot to commit, which just
removes a lastprivate check (which hsa now can handle) at a place
where it cannot ever be anyway.
Committed to the hsa branch, will include it in the pile of OpenMP
stuff to request to merge to trunk later this week.
Thanks,
On 11/07/2016 10:48 AM, Mike Stump wrote:
On Nov 6, 2016, at 11:39 AM, Iain Sandoe wrote:
This is an initial patch in a series that converts Darwin's configury to detect
ld64 features, rather than the current process of hard-coding them on target
system version.
So, I really do hate to ask,
Hi,
this has been in my TODO list for at least two years, probably longer,
although I do no longer remember why I added it there. The idea is to
introduce a special wrapper around operands_equal_p for TYPE_SIZE
comparisons, which would try simple pointer equality before calling more
complex opera
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):
>
>> mrsx9, tpidr_el0
>> movx10, -
>
> As Jiong alread
Iain,
It certainly looks like you dropped a file here. The proposed
ChangeLog shows...
* config.in: Likewise.
but the previously proposed hunk from...
diff --git a/gcc/config.in b/gcc/config.in
index a736de3..a7ff3ee 100644
--- a/gcc/config.in
+++ b/gcc/config.in
@@ -1934,6 +1934,18 @@
#en
Gabriel and I have been tracking down an include path issue for GCC 6
Advance Toolchain builds (ie, --with-advance-toolchain=...). The solution
that fixes the problem for us is to configure with --with-local-prefix=...
and removing the following hunk from config.gcc. Gabriel has confirmed
this fi
It occurred to me that a simple trace of template instantiations would
fit simply into the stream of function declarations that
announce_function prints when -quiet is not specified to the compiler.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit ae7b4a929fbd05de433451a1d92794d962366646
Auth
On Tue, 2016-10-25 at 14:47 +0200, Bernd Schmidt wrote:
> On 10/21/2016 10:27 PM, David Malcolm wrote:
> > Thanks. I attemped to use those fields of recog_data, but it
> > doesn't
> > seem to be exactly what's needed here.
>
> Yeah, I may have been confused. I'm not sure that just looking at
> SC
1 - 100 of 130 matches
Mail list logo
