Re: [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)

2017-09-04 Thread Kyrill Tkachov
Hi Bernd, On 29/04/17 18:52, Bernd Edlinger wrote: Ping... I attached the latest version of my patch. Thanks Bernd. On 12/18/16 14:14, Bernd Edlinger wrote: Hi, this splits the *arm_negdi2, *arm_cmpdi_insn and *arm_cmpdi_unsigned also at split1 except for TARGET_NEON and TARGET_IWMMXT.

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Kyrill Tkachov
Hi Wilco, Sorry for the delay... On 27/06/17 16:38, Wilco Dijkstra wrote: ping From: Wilco Dijkstra Sent: 17 January 2017 18:00 To: GCC Patches Cc: nd; Kyrylo Tkachov; Richard Earnshaw Subject: [PATCH][ARM] Remove Thumb-2 iordi_not patterns After Bernd's DImode patch [1] almost all DImode

Re: [PATCH][ARM] Fix ldrd offsets

2017-09-04 Thread Kyrill Tkachov
On 27/06/17 16:39, Wilco Dijkstra wrote: ping From: Wilco Dijkstra Sent: 03 November 2016 12:20 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Fix ldrd offsets Fix ldrd offsets of Thumb-2 - for TARGET_LDRD the range is +-1020, without -255..4091. This reduces the number of addressing

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Wilco Dijkstra
Kyrill Tkachov wrote: > -(define_insn_and_split "*iordi_notsesidi_di" > -  [(set (match_operand:DI 0 "s_register_operand" "=,") > -   (ior:DI (not:DI (sign_extend:DI > -    (match_operand:SI 2 "s_register_operand" "r,r"))) > -   (match_operand:DI 1

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Kyrill Tkachov
On 04/09/17 18:07, Wilco Dijkstra wrote: Kyrill Tkachov wrote: -(define_insn_and_split "*iordi_notsesidi_di" - [(set (match_operand:DI 0 "s_register_operand" "=,") - (ior:DI (not:DI (sign_extend:DI -(match_operand:SI 2 "s_register_operand" "r,r"))) -

Re: Add support to trace comparison instructions and switch statements

2017-09-04 Thread Jakub Jelinek
On Mon, Sep 04, 2017 at 09:36:40PM +0800, 吴潍浠(此彼) wrote: > gcc/ChangeLog: > > 2017-09-04 Wish Wu > > * asan.c (initialize_sanitizer_builtins): > *

Re: [RFC, vectorizer] Allow single element vector types for vector reduction operations

2017-09-04 Thread Andrew Pinski
On Mon, Sep 4, 2017 at 7:28 AM, Tamar Christina wrote: >> > vect__5.25_58 = VIEW_CONVERT_EXPR> intD.11>(vect__4.21_65); >> > vect__5.25_57 = VIEW_CONVERT_EXPR> intD.11>(vect__4.22_63); >> > vect__5.25_56 =

Re: [PATCH][ARM] Improve max_insns_skipped logic

2017-09-04 Thread Kyrill Tkachov
On 27/06/17 16:38, Wilco Dijkstra wrote: ping From: Wilco Dijkstra Sent: 10 November 2016 17:19 To: GCC Patches Cc: nd Subject: [PATCH][ARM] Improve max_insns_skipped logic Improve the logic when setting max_insns_skipped. Limit the maximum size of IT to MAX_INSN_PER_IT_BLOCK as

Re: Add support to trace comparison instructions and switch statements

2017-09-04 Thread 吴潍浠(此彼)
Sorry. I made a terrible bug in it. Attachment is my new patch. Wish Wu -- From:Wish Wu Time:2017 Sep 4 (Mon) 21:17 To:Dmitry Vyukov ; Jakub Jelinek Cc:gcc

[PATCH] Fix PR82084

2017-09-04 Thread Richard Biener
The following fixes vectorization of stores from wide-char STRING_CSTs which native_encode_string doesn't handle (will post a preliminary patch for that and trunk later). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk sofar. Richard. 2017-09-04 Richard Biener

[PATCH] Another type demotion issue with ubsan (PR sanitizer/82072)

2017-09-04 Thread Marek Polacek
Vittorio reported another issue with convert_to_integer_1: for u = -l; where u is unsigned and l is long long the function does: 911 return convert (type, 912 fold_build1 (ex_form, typex, 913convert (typex,

[PATCH, i386]: Fix PR82098, ICE in elimination_costs_in_insn

2017-09-04 Thread Uros Bizjak
2017-09-04 Uros Bizjak PR target/82098 * config/i386/i386.md (*_mask): Add TARGET_USE_BT to insn constraint. (*btr_mask): Ditto. Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}. Committed to mainline SVN. Uros. Index: config/i386/i386.md

[PATCH, i386 testsuite]: Introduce gcc.target/i386/mpx/mpx-os-support.h

2017-09-04 Thread Uros Bizjak
2017-09-04 Uros Bizjak * gcc.target/i386/mpx/mpx-os-support.h: New file. * gcc.target/i386/mpx/mpx-check.h: Include mpx-os-support.h. Tested on x86_64-linux-gnu {,-m32}. Committed to mainline SVN. Uros. Index: gcc.target/i386/mpx/mpx-check.h

Re: [PATCH]: PR target/80204 (Darwin macosx-version-min problem)

2017-09-04 Thread Simon Wright
(Apologies if this is a duplicate: Mac Mail vs the list's requirement for plain text) On 4 Sep 2017, at 07:09, Jeff Law wrote: > > On 09/01/2017 04:05 PM, Simon Wright wrote: >> In gcc/config/darwin-driver.c, darwin_find_version_from_kernel() assumes >> that the minor version

Re: [PING][PATCH 2/3] retire mem_signal_fence pattern

2017-09-04 Thread Uros Bizjak
> On 09/01/2017 12:26 AM, Alexander Monakov wrote: >> On Thu, 31 Aug 2017, Jeff Law wrote: >>> This is OK. >>> >>> What's the point of the delete_insns_since calls in patch #3? >> >> Deleting the first barrier when maybe_expand_insn failed. >> Other functions in the file use a similar approach. >

Re: [PATCH]: PR target/80204 (Darwin macosx-version-min problem)

2017-09-04 Thread Jakub Jelinek
On Mon, Sep 04, 2017 at 07:56:58PM +0100, Simon Wright wrote: > >>2017-09-01 Simon Wright > >> > >>PR target/80204 > >>* config/darwin-driver.c (darwin_find_version_from_kernel): eliminate > >> calculation of the > >> minor version, always output as 0. > >>

Re: [C++] Fix PR bootstrap/81926

2017-09-04 Thread Richard Biener
On Sun, Sep 3, 2017 at 4:15 PM, Eric Botcazou wrote: >> But not types, less using langhooks. > > #0 cp_get_debug_type (type=0x707702a0) > at /home/eric/svn/gcc-7.2.0/gcc/cp/cp-objcp-common.c:142 > #1 0x009f2c86 in gen_type_die_with_usage ( >

Re: [PATCH] [PR79542][Ada] Fix ICE in dwarf2out.c with nested func. inlining

2017-09-04 Thread Pierre-Marie de Rodat
On 08/18/2017 12:10 PM, Richard Biener wrote: ok, not doing this at all doesn't work, doing it only in the above case neither. Bah. Can anyone explain to me why we do the set_decl_origin_self dance? Ok, so I need the following incremental patch to fix the fallout. This allows Ada LTO

Re: [PATCH v2] Python testcases to check DWARF output

2017-09-04 Thread Pierre-Marie de Rodat
Hello, I would like to ping for the patch I submitted at . Thank you in advance! -- Pierre-Marie de Rodat

Re: [PATCH 0/2] add unique_ptr class

2017-09-04 Thread Richard Biener
On Fri, Aug 11, 2017 at 10:43 PM, Jonathan Wakely wrote: > On 05/08/17 20:05 +0100, Pedro Alves wrote: >> >> On 08/04/2017 07:52 PM, Jonathan Wakely wrote: >>> >>> On 31/07/17 19:46 -0400, tbsaunde+...@tbsaunde.org wrote: I've been saying I'd do this for a long time,

[5/9] Add mode_for_int_vector helper functions

2017-09-04 Thread Richard Sandiford
There are at least a few places that want to create an integer vector with a specified element size and element count, or to create the integer equivalent of an existing mode. This patch adds helpers for doing that. The require ()s are all used in functions that go on to emit instructions that

Re: [RFC, vectorizer] Allow single element vector types for vector reduction operations

2017-09-04 Thread Richard Biener
On Mon, 4 Sep 2017, Tamar Christina wrote: > Indeed it does, but even worse if you look at the gimple, > you see lots of > > vect__5.25_58 = VIEW_CONVERT_EXPR intD.11>(vect__4.21_65); > vect__5.25_57 = VIEW_CONVERT_EXPR

Re: [PATCH 0/2] add unique_ptr class

2017-09-04 Thread Pedro Alves
On 09/04/2017 11:31 AM, Richard Biener wrote: > On Fri, Aug 11, 2017 at 10:43 PM, Jonathan Wakely wrote: >> On 05/08/17 20:05 +0100, Pedro Alves wrote: >>> >>> On 08/04/2017 07:52 PM, Jonathan Wakely wrote: On 31/07/17 19:46 -0400, tbsaunde+...@tbsaunde.org wrote:

[PATCH] Fix PR82060

2017-09-04 Thread Richard Biener
The following fixes PR82060 but canonicalizing mems before dispatching to the devrit machinery. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2017-09-04 Richard Biener PR tree-optimization/82060 * tree-ssa-pre.c

Re: [PATCH][GCC][Testsuite][ARM][AArch64] Enable Dot Product for generic tests for ARM and AArch64 [Patch (7/8)]

2017-09-04 Thread James Greenhalgh
On Fri, Sep 01, 2017 at 02:23:39PM +0100, Tamar Christina wrote: > Hi All, > > This patch enables tests for Dot Product vectorization > in gcc for ARM and AArch64. > > The ARMv8.2-a Dot Product instructions only support 8-bit > element vectorization. > > Dot product is available from ARMv8.2-a

[0/9] Make more use of opt_mode

2017-09-04 Thread Richard Sandiford
The 77-patch machine-mode series originally targetted only the places that needed to change for variable-sized modes, but as Richard B. said on IRC, it meant that the interfaces of mode_for_size vs. int_mode_for_size were inconsistent: the former still returns BLKmode on failure, while the latter

[libstdc++/71500] make back reference work with icase

2017-09-04 Thread Tim Shen via gcc-patches
This fixes the follow-up comments in 71500. Back-reference matching is different from other matching, as the content the back-reference refers to is at "run-time", aka during regex_match(), not regex() compilation. For compilation we do have an abstraction layer to catch all comparison

[2/9] Make more use of int_mode_for_size

2017-09-04 Thread Richard Sandiford
This patch converts more places that could use int_mode_for_size instead of mode_for_size. This is in preparation for an upcoming patch that makes mode_for_size itself return an opt_mode. require () seems like the right choice in expand_builtin_powi because we have got past the point of backing

[3/9] (decimal_)float_mode_for_size in real.h

2017-09-04 Thread Richard Sandiford
This patch makes the binary float macros in real.h use float_mode_for_size and adds a corresponding decimal_float_mode_for_size for the decimal macros. 2017-09-04 Richard Sandiford gcc/ * machmode.h (decimal_float_mode_for_size): New function. *

Re: [RFA] [PATCH][PR tree-optimization/64910] Fix reassociation of binary bitwise operations with 3 operands

2017-09-04 Thread Richard Biener
On Sun, Sep 3, 2017 at 4:44 PM, Jeff Law wrote: > On 01/13/2016 05:30 AM, Richard Biener wrote: >> On Wed, Jan 13, 2016 at 7:39 AM, Jeff Law wrote: >>> On 01/12/2016 08:11 AM, Richard Biener wrote: On Tue, Jan 12, 2016 at 6:10 AM, Jeff Law

Re: [PATCH][GCC][AArch64] Dot Product SIMD patterns [Patch (5/8)]

2017-09-04 Thread James Greenhalgh
On Fri, Sep 01, 2017 at 02:22:17PM +0100, Tamar Christina wrote: > Hi All, > > This patch adds the instructions for Dot Product to AArch64 along > with the intrinsics and vectorizer pattern. > > Armv8.2-a dot product supports 8-bit element values both > signed and unsigned. > > Dot product is

[1/9] Make more use of int_mode_for_mode

2017-09-04 Thread Richard Sandiford
This patch converts more places that could use int_mode_for_mode instead of mode_for_size. This is in preparation for an upcoming patch that makes mode_for_size itself return an opt_mode. The reason for using required () in exp2_immediate_p is that we go on to do: trunc_int_for_mode (...,

[8/9] Make mode_for_size_tree return an opt_mode

2017-09-04 Thread Richard Sandiford
...for consistency with mode_for_size 2017-09-04 Richard Sandiford gcc/ * stor-layout.h (mode_for_size_tree): Return an opt_mode. * stor-layout.c (mode_for_size_tree): Likewise. (mode_for_array): Update accordingly. (layout_decl):

[9/9] Make bitsize_mode_for_mode return an opt_mode

2017-09-04 Thread Richard Sandiford
2017-09-04 Richard Sandiford gcc/ * machmode.h (bitwise_mode_for_mode): Return opt_mode. * stor-layout.c (bitwise_mode_for_mode): Likewise. (bitwise_type_for_mode): Update accordingly. Index: gcc/machmode.h

Re: [PATCH][GCC][AArch64] Dot Product commandline options [Patch (4/8)]

2017-09-04 Thread James Greenhalgh
On Fri, Sep 01, 2017 at 02:20:59PM +0100, Tamar Christina wrote: > Hi All, > > This patch adds support for the +dotprod extension to AArch64. > Dot Product requires Adv.SIMD to work and so enables this option > by default when enabled. > > It is available from ARMv8.2-a and onwards and is

Re: [PATCH][GCC][AArch64] Dot Product NEON intrinsics [Patch (6/8)]

2017-09-04 Thread James Greenhalgh
On Fri, Sep 01, 2017 at 02:22:55PM +0100, Tamar Christina wrote: > Hi All, > > This patch adds the Adv.SIMD intrinsics for Dot product. > > Dot product is available from ARMv8.2-a and onwards. > > Regtested and bootstrapped on aarch64-none-elf and no issues. > > Ok for trunk? OK. Thanks,

[7/9] Make targetm.get_mask_mode return an opt_mode

2017-09-04 Thread Richard Sandiford
...for consistency with mode_for_vector. 2017-09-04 Richard Sandiford gcc/ * target.def (get_mask_mode): Change return type to opt_mode. Expand commentary. * doc/tm.texi: Regenerate. * targhooks.h (default_get_mask_mode): Return an

[6/9] Make mode_for_vector return an opt_mode

2017-09-04 Thread Richard Sandiford
...following on from the mode_for_size change. The patch also removes machmode.h versions of the stor-layout.c comments, since the comments in the .c file are more complete. 2017-09-04 Richard Sandiford gcc/ * machmode.h (mode_for_vector): Return an

Re: [PATCH] [PR79542][Ada] Fix ICE in dwarf2out.c with nested func. inlining

2017-09-04 Thread Richard Biener
On Mon, Sep 4, 2017 at 11:20 AM, Pierre-Marie de Rodat wrote: > On 08/18/2017 12:10 PM, Richard Biener wrote: >>> >>> ok, not doing this at all doesn't work, doing it only in the above case >>> neither. >>> >>> Bah. >>> >>> Can anyone explain to me why we do the

[4/9] Make mode_for_size return an opt_mode

2017-09-04 Thread Richard Sandiford
...to make it consistent with int_mode_for_size etc. require () seems like the right choice in replace_reg_with_saved_mem because we use the chosen mode for saving and restoring registers, which cannot be done in BLKmode. Similarly require () seems like the right choice in calls related to

RE: [PATCH][GCC][Testsuite][ARM][AArch64] Enable Dot Product for generic tests for ARM and AArch64 [Patch (7/8)]

2017-09-04 Thread Tamar Christina
> I'm surprised that this worked! > > It looks like you unconditionally add the -march=armv8.2-a+dotprod options, > which should cause you to generate instructions which will not execute on > targets which don't support this instruction. As far as I can see, this is an > execute test, so that

Re: [PATCH] Expand switch statements with a single (or none) non-default case.

2017-09-04 Thread Richard Biener
On Fri, Sep 1, 2017 at 3:01 PM, Martin Liška wrote: > On 09/01/2017 12:57 PM, Richard Biener wrote: >> On Fri, Sep 1, 2017 at 12:44 PM, Martin Liška wrote: >>> On 09/01/2017 10:26 AM, Richard Biener wrote: On Fri, Sep 1, 2017 at 10:07 AM, Martin Liška

Re: [PATCH] Forbid section anchors for ASan build (PR sanitizer/81697)

2017-09-04 Thread Maxim Ostapenko
+ Jakub On 09/08/17 19:21, Jeff Law wrote: On 08/08/2017 12:46 AM, Vyacheslav Barinov wrote: Hello, Andrew Pinski writes: On Mon, Aug 7, 2017 at 11:15 PM, Slava Barinov wrote: gcc/ * varasm.c (use_object_blocks_p): Forbid section

Re: Add a partial_integral_type_p helper function

2017-09-04 Thread Jeff Law
On 08/18/2017 02:13 AM, Richard Sandiford wrote: > This patch adds a partial_integral_type_p function, to go along > with the full_integral_type_p added by the previous patch. > > Of the changes that didn't previously have an INTEGRAL_TYPE_P check: > > - the convert_to_integer_1 hunks are

Re: [C++] Fix PR bootstrap/81926

2017-09-04 Thread Eric Botcazou
> A solution would be to put them into a global GCed pointer-map or vector, > freeing that at free-lang-data time. Here's a version that implements a bona-fide type->offset_type map. PR bootstrap/81926 * cp-objcp-common.c (struct offset_type_hasher): New class.

Re: Add a partial_integral_type_p helper function

2017-09-04 Thread Jeff Law
On 08/25/2017 10:48 AM, Martin Sebor wrote: > On 08/18/2017 02:13 AM, Richard Sandiford wrote: >> This patch adds a partial_integral_type_p function, to go along >> with the full_integral_type_p added by the previous patch. >> >> Of the changes that didn't previously have an INTEGRAL_TYPE_P check:

[committed] Fix pr70043.f90 testcase (PR testsuite/82093)

2017-09-04 Thread Jakub Jelinek
Hi! This testcase contains an out-of-bound access, which isn't necessary for the thing the testcase tests. I've verified that both the original and patched testcase hangs r233928 and succeeds with r233934 and committed to trunk as obvious. 2017-09-04 Jakub Jelinek

Re: [RFC, vectorizer] Allow single element vector types for vector reduction operations

2017-09-04 Thread Tamar Christina
Indeed it does, but even worse if you look at the gimple, you see lots of vect__5.25_58 = VIEW_CONVERT_EXPR(vect__4.21_65); vect__5.25_57 = VIEW_CONVERT_EXPR(vect__4.22_63); vect__5.25_56 = VIEW_CONVERT_EXPR

[PATCH] Add a -Wcast-align=strict warning

2017-09-04 Thread Bernd Edlinger
Hi, as you know we have a -Wcast-align warning which works only for STRICT_ALIGNMENT targets. But occasionally it would be nice to be able to switch this warning on even for other targets. Therefore I would like to add a strict version of this option which can be invoked with

Re: [PATCH] Disable type demotion for sanitizer (PR sanitizer/82072)

2017-09-04 Thread Jeff Law
On 09/01/2017 11:47 AM, Marek Polacek wrote: > Here, do_narrow and convert_to_integer_1 is demoting signed types to unsigned, > e.g. for > i = i - lmin > where i is int and lmin is long int, so what we should produce is > i = (int) ((long int) i - lmin) > but instead it produces > i = (int)

Re: [PATCH]: PR target/80204 (Darwin macosx-version-min problem)

2017-09-04 Thread Jeff Law
On 09/01/2017 04:05 PM, Simon Wright wrote: > In gcc/config/darwin-driver.c, darwin_find_version_from_kernel() assumes > that the minor version in the Darwin kernel version (16.7.0, => minor > version 7) is equal to the bugfix component of the macOS version, so that > the compiler receives

Re: [00/77] Add wrapper classes for machine_modes

2017-09-04 Thread Gerald Pfeifer
On Sun, 3 Sep 2017, Richard Sandiford wrote: > This sounds like the same as PR82045. Could you try the patch I posted > here: https://gcc.gnu.org/ml/gcc-patches/2017-09/msg00062.html ? Yep, that resolves it; tested per what you committed last night. Thank you! Gerald

RE: [PATCH][GCC][ARM][AArch64] Testsuite framework changes and execution tests [Patch (8/8)]

2017-09-04 Thread Tamar Christina
Hi Christophe, > > > > gcc/testsuite > > 2017-09-01 Tamar Christina > > > > * lib/target-supports.exp > > (check_effective_target_arm_v8_2a_dotprod_neon_ok_nocache): > New. > > (check_effective_target_arm_v8_2a_dotprod_neon_ok): New. > >

Re: Add support to trace comparison instructions and switch statements

2017-09-04 Thread 吴潍浠(此彼)
Hi I updated the patch and put it in attachment. gcc/ChangeLog: 2017-09-04 Wish Wu * asan.c (initialize_sanitizer_builtins): * builtin-types.def

Re: [PATCH][GCC][ARM] Dot Product commandline options [Patch (1/8)]

2017-09-04 Thread Richard Earnshaw (lists)
On 01/09/17 14:19, Tamar Christina wrote: > Hi All, > > This patch adds support for the +dotprod extension to ARM. > Dot Product requires Adv.SIMD to work and so enables this option > by default when enabled. > > It is available from ARMv8.2-a and onwards and is enabled by > default on

RE: [RFC, vectorizer] Allow single element vector types for vector reduction operations

2017-09-04 Thread Tamar Christina
> > vect__5.25_58 = VIEW_CONVERT_EXPR intD.11>(vect__4.21_65); > > vect__5.25_57 = VIEW_CONVERT_EXPR intD.11>(vect__4.22_63); > > vect__5.25_56 = VIEW_CONVERT_EXPR intD.11>(vect__4.23_61); > > vect__5.25_55 =

Re: [PATCH][GCC][ARM][AArch64] Testsuite framework changes and execution tests [Patch (8/8)]

2017-09-04 Thread Christophe Lyon
Hi Tamar, On 1 September 2017 at 15:24, Tamar Christina wrote: > Hi All, > > This patch enables the execution runs for Dot product and also > adds the feature tests. > > The ARMv8.2-a Dot Product instructions only support 8-bit > element vectorization. > > Dot product is

[ARC] Fix stack unwinding for ARC

2017-09-04 Thread Cupertino Miranda
Hi everyone, Here is patch to stack unwinding for ARC. - Fix to unwinding. Now is is possible to unwind from syscall wrappers, signals and functions with dynamic stack allocation. - Patch also fixes millicode. Although millicode save and restore functions would change blink, the calls to those

Re: [PATCH][aarch64] Enable ifunc resolver attribute by default

2017-09-04 Thread Szabolcs Nagy
On 12/06/17 16:02, Steve Ellcey wrote: > I recently noticed that the GCC 'resolver' attribute used for ifunc's is not > on by default for aarch64 even though all the infrastructure to support it is > in place. I made memcpy an ifunc on aarch64 in glibc and am looking at > possibly using it for

Re: [PATCH GCC]A simple implementation of loop interchange

2017-09-04 Thread Bin.Cheng
On Mon, Sep 4, 2017 at 2:54 PM, Richard Biener wrote: > On Wed, Aug 30, 2017 at 6:32 PM, Bin.Cheng wrote: >> On Wed, Aug 30, 2017 at 3:19 PM, Richard Biener >> wrote: >>> On Wed, Aug 30, 2017 at 3:18 PM, Bin Cheng

Re: [PATCH GCC]A simple implementation of loop interchange

2017-09-04 Thread Richard Biener
On Wed, Aug 30, 2017 at 6:32 PM, Bin.Cheng wrote: > On Wed, Aug 30, 2017 at 3:19 PM, Richard Biener > wrote: >> On Wed, Aug 30, 2017 at 3:18 PM, Bin Cheng wrote: >>> Hi, >>> This patch implements a simple loop interchange

Re: [PATCH, ARM] correctly encode the CC reg data flow

2017-09-04 Thread Kyrill Tkachov
Hi Bernd, On 18/01/17 15:36, Bernd Edlinger wrote: On 01/13/17 19:28, Bernd Edlinger wrote: On 01/13/17 17:10, Bernd Edlinger wrote: On 01/13/17 14:50, Richard Earnshaw (lists) wrote: On 18/12/16 12:58, Bernd Edlinger wrote: Hi, this is related to PR77308, the follow-up patch will depend

[PATCH] Handle wide-chars in native_encode_string

2017-09-04 Thread Richard Biener
The following patch cures the missed optimization in PR82084, vectorizing a wide-char initializer wchar_t strs[4][2]= { L"A", L"B", L"C" , L"D"}; with AVX to MEM[(wchar_t[2] *)] = { 65, 66, 67, 68 }; it's not entirely clear to me whether the individual STRING_CSTs we build for the

Re: [PING**2] [PATCH, ARM] Further improve stack usage on sha512 (PR 77308)

2017-09-04 Thread Kyrill Tkachov
On 29/04/17 18:45, Bernd Edlinger wrote: Ping... I attached a rebased version since there was a merge conflict in the xordi3 pattern, otherwise the patch is still identical. It splits adddi3, subdi3, anddi3, iordi3, xordi3 and one_cmpldi2 early when the target has no neon or iwmmxt. Thanks

Re: [PATCH] Handle wide-chars in native_encode_string

2017-09-04 Thread Joseph Myers
On Mon, 4 Sep 2017, Richard Biener wrote: > always have a consistend "character" size and how the individual > "characters" are encoded. The patch assumes that the array element > type of the STRING_CST can be used to get access to individual > characters by means of the element type size and

Re: [RFC, vectorizer] Allow single element vector types for vector reduction operations

2017-09-04 Thread Christophe Lyon
On 30 August 2017 at 17:11, Jon Beniston wrote: > Hi Richard, > >> I think the issue is that with TARGET_VECTOR_MODE_SUPPORTED_P false >> for V1SI you'll get a SImode vector type and the >> >> if (VECTOR_BOOLEAN_TYPE_P (type_in) >> || VECTOR_MODE_P (TYPE_MODE (type_in)))

Re: [PATCH] PR libstdc++/79162 ambiguity in string assignment due to string_view overload (LWG 2946)

2017-09-04 Thread Jonathan Wakely
On 30/07/17 15:01 +0200, Daniel Krügler wrote: 2017-07-28 22:40 GMT+02:00 Daniel Krügler : 2017-07-28 22:29 GMT+02:00 Daniel Krügler : 2017-07-28 22:25 GMT+02:00 Tim Song : On Fri, Jul 28, 2017 at 4:10 PM, Daniel

Re: [RFC PATCH, i386]: Convert TLS location to DEFAULT_TLS_SEG_REG address space

2017-09-04 Thread Uros Bizjak
On Thu, Aug 31, 2017 at 9:32 PM, Uros Bizjak wrote: > Hello! > > Using following testcase: > > --cut here-- > __thread int a; > > int foo (void) > { > return a; > } > --cut here-- > > Attached patch converts TLS location in the form of: > > (mem/c:SI (plus:DI

Re: [PATCH]: PR target/80204 (Darwin macosx-version-min problem)

2017-09-04 Thread Simon Wright
On 1 Sep 2017, at 23:05, Simon Wright wrote: > > 2017-09-01 Simon Wright > > PR target/80204 > * config/darwin-driver.c (darwin_find_version_from_kernel): eliminate > calculation of the > minor version, always output as 0.

Re: std::vector default default and move constructors

2017-09-04 Thread François Dumont
Hi Gentle reminder. Thanks On 21/08/2017 21:15, François Dumont wrote: Following feedback on std::list patch this one had the same problem of unused code being deleted. So here is a new version. Ok to commit ? François On 28/07/2017 18:45, François Dumont wrote: Hi There was a

Re: [PATCH]: PR target/80204 (Darwin macosx-version-min problem)

2017-09-04 Thread Jakub Jelinek
On Mon, Sep 04, 2017 at 08:47:07PM +0100, Simon Wright wrote: > On 1 Sep 2017, at 23:05, Simon Wright wrote: > > > > 2017-09-01 Simon Wright > > > > PR target/80204 > > * config/darwin-driver.c (darwin_find_version_from_kernel): eliminate >

Re: [PATCH, ARM] correctly encode the CC reg data flow

2017-09-04 Thread Bernd Edlinger
Hi Kyrill, Thanks for your review! On 09/04/17 15:55, Kyrill Tkachov wrote: > Hi Bernd, > > On 18/01/17 15:36, Bernd Edlinger wrote: >> On 01/13/17 19:28, Bernd Edlinger wrote: >>> On 01/13/17 17:10, Bernd Edlinger wrote: On 01/13/17 14:50, Richard Earnshaw (lists) wrote: > On

Re: [PATCH][ARM] Remove Thumb-2 iordi_not patterns

2017-09-04 Thread Wilco Dijkstra
Kyrill Tkachov wrote: > > After Bernd's change almost all DI mode instructions are split before > > register > > allocation. So instructions using DI mode no longer exist and thus these > > extend variants can never be matched and are thus redundant. > > Bernd's patch splits them when we don't