[PATCH, rs6000] Fix PR70957 (skip vsx-elemrev-[24].c tests for a downlevel assembler)

Hi, PR70957 reports that the two subject tests fail on an older P7 machine. These tests rely on built-ins that exploit POWER9 vector support. It turns out that the failure occurs because the configured assembler is downlevel, and does not support even POWER8 instructions. This causes

[PATCH 11/16] Add hash-set-tests.c

Jeff approved an earlier version of this (as unittests/test-hash-set.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03300.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is > pre-approved. This version moves the

[PATCH 06/16] et-forest.c: add selftests

Jeff approved an earlier version of this: https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03295.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files > is pre-approved. This version has been updated to the new style.

[C++ Patch] Change a few error + error to error + inform

Hi, while working on c++/70202, I noticed a few more of those pairs for errors which we changed in the past to error + inform. Tested x86_64-linux. Thanks, Paolo. / /cp 2016-06-02 Paolo Carlini * decl.c (xref_tag_1): Change pairs

[PATCH v2] gcc/config/tilegx/tilegx.c (tilegx_function_profiler): Save r10 to stack before call mcount

From: Chen Gang r10 may also be as parameter stack pointer for the nested function, so need save it before call mcount. 2016-06-03 Chen Gang gcc/ PR target/71331 * config/tilegx/tilegx.c (tilegx_function_profiler):

[PATCH 02/16] diagnostic-show-locus.c: add selftests

gcc/ChangeLog: * diagnostic-show-locus.c: Include "selftest.h". (make_range): New function. (test_range_contains_point_for_single_point): New function. (test_range_contains_point_test_for_single_line): New function.

[PATCH 05/16] tree-cfg.c: add selftests

Jeff approved an older version of this: https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03285.html with: > Unless there's a good reason, drop the presumably redundant tests > and this is OK. Save preapprovald for these changes as the bitmap > patch. This version removes the redundant tests, moves

[PATCH 10/16] Add hash-map-tests.c

Jeff approved an earlier version of this (as unittests/test-hash-map.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03301.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is > pre-approved. This version moves the

[PATCH, RS6000] Add RS6000_BTM_MODULO to set of RS6000_BTM_COMMON flags

This patch adds the RS6000_BTM_MODULO flag to to the set of flags associated with the RS6000_BTM_COMMON variable. This patch has bootstrapped with the trunk and the gcc-6-branch on powerpc64le-unknown-linux-gnu and there were no regressions. Is it ok to merge this with the trunk? Can I merge

[PATCH 12/16] input.c: add selftests

Jeff conditionally approved an earlier version of this (as unittests/test-locations.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03307.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is > pre-approved.

[PATCH 13/16] Add rtl-tests.c

Jeff approved an earlier version of this (as unittests/test-rtl.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03302.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is >pre-approved. gcc/ChangeLog: *

[PATCH 16/16] wide-int.cc: add selftests

Jeff approved an early version of this: https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03309.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is > pre-approved. This version moves it to wide-int.cc and converts it to

[PATCH 14/16] tree.c: add selftests

Jeff approved an earlier version of this (as unittests/test-tree.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03303.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is > pre-approved. gcc/ChangeLog: *

[PATCH 07/16] fold-const.c: add selftests

Jeff approved an older version of this (as a separate unittests/test-folding.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03305.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files > is pre-approved. gcc/ChangeLog:

[PATCH] Fix cgraph edge redirection with non-POD lhs (PR middle-end/71387)

Hi! Apparently my r236430 change (trunk) and r236431 (6.x) broke the following testcase. In the later similar change to gimple-fold.c in r236506 I've added code to tweak gimple_call_fntype if we have newly one of the void something (void) __attribute__((noreturn)) functions like

Re: [PATCH] Fix 416.gamess miscompare (PR71311)

On June 2, 2016 9:05:36 PM GMT+02:00, Richard Biener wrote: >On June 2, 2016 7:47:19 PM GMT+02:00, Bernhard Reutner-Fischer > wrote: >>On June 1, 2016 9:37:26 AM GMT+02:00, Richard Biener >> wrote: >> PR tree-optimization/71311

[PATCH 01/16] Core of selftest framework (v6)

gcc/ChangeLog: * Makefile.in (OBJS): Add function-tests.o, hash-map-tests.o, hash-set-tests.o, rtl-tests.o, selftest-run-tests.o. (OBJS-libcommon): Add selftest.o. (OBJS-libcommon-target): Add selftest.o. (all.internal): Add "selftests".

[PATCH 09/16] gimple.c: add selftests

Jeff approved an earlier version of this (as unittests/test-gimple.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03304.html > OK if/when prereqs are approved. Minor twiddling if we end > up moving it elsewhere or standardizing/reducing header files > is pre-approved. This version moves the

[PATCH 03/16] spellcheck.c: convert Levenshtein test from a plugin to a selftest

This is an example of converting one of our existing plugin-based tests to run within -fself-test instead. gcc/ChangeLog: * spellcheck.c: Include "selftest.h". (levenshtein_distance_unit_test_oneway): New function, adapted from testsuite/gcc.dg/plugin/levenshtein_plugin.c.

[PATCH 04/16] bitmap.c: add selftests

Jeff pre-approved the plugin version of this (as a new file unittests/test-bitmap.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03284.html with: > OK if/when prereqs are approved. Minor twiddling if we end up moving it > elsewhere or standardizing/reducing header files is pre-approved. This

[PATCH 15/16] vec.c: add selftests

Jeff approved an earlier version of this (as unittests/test-vec.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03308.html > OK if/when prereqs are approved. Minor twiddling if we end up > moving it elsewhere or standardizing/reducing header files is > pre-approved. gcc/ChangeLog: *

tuple code simplification patch

Hi I was trying to play with tuple implementation and was annoyed by repetition of _Head type when instantiating _Head_base so I thought about this patch. How do you like it ? I still need to run tests, will do before commit, ok ? François Index: include/std/tuple

[PATCH, applied] Backport PowerPC ISA 3.0 C min/max support to GCC 6.2

I backported the changes for adding the ISA 3.0 C min/max and floating point scalar compare generating a mask (for conditional move) to the GCC 6.2 branch. The changes in this patch were checked into the trunk on May 25th (with one patch from May 18th from a patch set that has not been backported

[PATCH 00/16] v6 of -fself-test/unit-testing patch

Given that we're now using an "abort on first failure" model, I renamed all of the EXPECT_ macros to ASSERT_ (for consistency with GTest). As per Bernd's suggestions I eliminated the runner class and moved to a more function-based rather than class-based approach. At this point, the only reasons

[PATCH 08/16] Add function-tests.c

Jeff approved an earlier version of this (as unittests/test-functions.c): https://gcc.gnu.org/ml/gcc-patches/2015-10/msg03310.html with: > OK if/when prereqs are approved. Minor twiddling if we end up moving > it elsewhere or standardizing/reducing header files is pre-approved. gcc/ChangeLog:

Re: [PATCH] nvptx per-warp compiler-defined stacks (-msoft-stack)

On Wed, 25 May 2016, Nathan Sidwell wrote: > > > It seems like we should reject the combination of -msoft-stack -fopenacc? > > > > Possibly; the doc text makes it explicit that the option is exposed only for > > the purpose of testing the compiler, anyway. > > It is always best to prevent the

Re: [C++ Patch] Change a few error + error to error + inform

OK. Jason

Re: [PATCH] integer overflow checking builtins in constant expressions

+{ + /* Perform the computation in the target type and check for overflow. */ + arg0 = fold_convert (type, arg0); + arg1 = fold_convert (type, arg1); + + if (tree result = size_binop_loc (loc, opcode, arg0, arg1)) +return TREE_OVERFLOW (result) ? build_one_cst

Re: [PATCH] integer overflow checking builtins in constant expressions

+ /* For the "generic" overloads, the first two arguments can have different + types and the last argument determines the target type to use to check + for overflow. The arguments of the other overloads all have the same + type. */ + tree type = TREE_TYPE (TREE_TYPE (arg2)); +

[PATCH] c/71392 - SEGV calling integer overflow built-ins with a null pointer

In a discussion of a patch in a this area (c/68120 and c++/70507) Jakub noticed that the integer arithmetic built-ins with overflow checking that expect a pointer to an integer as the last argument silently (i.e., without a warning) accept a null pointer. As the test case in the bug referenced

Re: [PATCH 01/16] Core of selftest framework (v6)

On 06/02/2016 11:06 PM, David Malcolm wrote: gcc/ChangeLog: * Makefile.in (OBJS): Add function-tests.o, hash-map-tests.o, hash-set-tests.o, rtl-tests.o, selftest-run-tests.o. (OBJS-libcommon): Add selftest.o. (OBJS-libcommon-target): Add selftest.o.

Re: [PATCH v2] gcc/config/tilegx/tilegx.c (tilegx_function_profiler): Save r10 to stack before call mcount

On 06/02/2016 03:23 PM, cheng...@emindsoft.com.cn wrote: fprintf (file, + "\t{\n" + "\taddi\tsp, sp, -8\n" + "\tst\tsp, r10\n" + "\t}\n" "\t{\n" You need only do this if cfun->static_chain_decl is set. r~

[PR tree-optimization/71328] Fix off-by-one error in CFG/SSA updates for backward threading

This bug has actually been around since the introduction of the FSM/backwards threader a few years ago. It was just latent. Essentially we need to look at the copied path for branches back into the path -- due to an off-by-one error, we missed the case where the last block in the path

Re: [ARM] Enable __fp16 as a function parameter and return type.

On 01/06/16 15:43, Christophe Lyon wrote: On 13 May 2016 at 15:41, Ramana Radhakrishnan wrote: On Thu, Apr 28, 2016 at 10:20 AM, Matthew Wahab wrote: This patch enables data movement for HF-mode values using VFP registers, when they are

Re: C, C++: Fix PR 69733 (bad location for ignored qualifiers warning)

On 05/04/2016 05:17 PM, Bernd Schmidt wrote: How's this? Fully retested on x86_64-linux. Ping. Bernd

Re: [PATCH][ARM] PR target/70830: Avoid POP-{reglist}^ when returning from interrupt handlers

On 02/06/16 09:47, Ramana Radhakrishnan wrote: On 17/05/16 11:40, Kyrill Tkachov wrote: On 13/05/16 12:05, Kyrill Tkachov wrote: Hi Christophe, On 12/05/16 20:57, Christophe Lyon wrote: On 12 May 2016 at 11:48, Ramana Radhakrishnan wrote: On Thu, May 5, 2016 at

Re: [PATCH 2/2][GCC] Add one more pattern to RTL if-conversion

On 05/24/2016 07:21 PM, Mikhail Maltsev wrote: + target = expand_simple_binop (mode, (diff < 0) ? MINUS : PLUS, Parentheses around (diff < 0) aren't needed. Otherwise I think this is also OK. Bernd

Re: [PATCH][RFC][rtlanal] Fix rtl-optimization/71295

On 05/31/2016 04:26 PM, Kyrill Tkachov wrote: we're talking about (subreg:V2DI (reg/v:EI 111 [ b ]) 0). EImode is a 24 byte arm-specific integer mode used to represent 3 D-registers. [...] I'm not familiar with the processes in that part of the code but my suspicion is that we should be

Re: [PATCH][wwwdocs] Improve arm and aarch64-related info in readings.html

On 28/05/16 21:04, Gerald Pfeifer wrote: Hi Kyrill, On Thu, 19 May 2016, Kyrill Tkachov wrote: I noticed that we have a readings.html page that has pointers to documentation of various backends that GCC supports. The info on arm seems a bit out of date and somewhat confusing, and there is no

Re: move increase_alignment from simple to regular ipa pass

On Thu, 2 Jun 2016, Prathamesh Kulkarni wrote: > On 1 June 2016 at 18:37, Richard Biener wrote: > > On Wed, 1 Jun 2016, Prathamesh Kulkarni wrote: > > > >> Hi Richard, > >> This patch tries to move increase_alignment pass from small to regular ipa > >> pass. > >> Does the

Re: [PATCH][RTL ifcvt] PR rtl-optimization/66940: Avoid signed overflow in noce_get_alt_condition

On 02/06/16 10:08, Bernd Schmidt wrote: On 05/23/2016 01:17 PM, Kyrill Tkachov wrote: In this PR we end up hitting a signed overflow in noce_get_alt_condition when we try to increment or decrement a HOST_WIDE_INT that might be HOST_WIDE_INT_MAX or HOST_WIDE_INT_MIN. I've confirmed the

Re: move increase_alignment from simple to regular ipa pass

On 1 June 2016 at 18:37, Richard Biener wrote: > On Wed, 1 Jun 2016, Prathamesh Kulkarni wrote: > >> Hi Richard, >> This patch tries to move increase_alignment pass from small to regular ipa >> pass. >> Does the patch look correct ? >> Since we are only increasing alignment of

Re: move increase_alignment from simple to regular ipa pass

On 2 June 2016 at 13:23, Richard Biener wrote: > On Thu, 2 Jun 2016, Prathamesh Kulkarni wrote: > >> On 1 June 2016 at 18:37, Richard Biener wrote: >> > On Wed, 1 Jun 2016, Prathamesh Kulkarni wrote: >> > >> >> Hi Richard, >> >> This patch tries to move

Re: [PATCH] integer overflow checking builtins in constant expressions

On Wed, Jun 01, 2016 at 09:10:52PM -0600, Martin Sebor wrote: > @@ -7957,8 +7957,8 @@ fold_builtin_arith_overflow (location_t loc, enum > built_in_function fcode, >tree arg0, tree arg1, tree arg2) > { >enum internal_fn ifn = IFN_LAST; > - tree type = TREE_TYPE

Re: [PATCH] Improve XMM16+ handling in *vec_concatv2di

Hello Jakub, On 01 Jun 22:17, Jakub Jelinek wrote: > Hi! > > This is the last pattern I'm aware of that didn't have any v/Yv constraints > that ought to be changed (there perhaps are others which have v/Yv in some > of the alternatives, but not in all the ones that could use it). > > The

Re: move increase_alignment from simple to regular ipa pass

On 2 June 2016 at 14:44, Prathamesh Kulkarni wrote: > On 2 June 2016 at 13:23, Richard Biener wrote: >> On Thu, 2 Jun 2016, Prathamesh Kulkarni wrote: >> >>> On 1 June 2016 at 18:37, Richard Biener wrote: >>> > On Wed, 1 Jun

Re: [PATCH] integer overflow checking builtins in constant expressions

On Thu, Jun 02, 2016 at 09:23:16AM +0200, Jakub Jelinek wrote: > Also, perhaps just a documentation thing, it would be good to clarify the > NULL last argument. From the POV of generating efficient code, I think we > should say something that the last argument (for the GNU builtins) must be >

Patch ping

Hi! I'd like to ping 3 XMM16+ patches: 1) avx_vec_concat http://gcc.gnu.org/ml/gcc-patches/2016-05/msg01826.html 2) *vec_concatv4si http://gcc.gnu.org/ml/gcc-patches/2016-05/msg02115.html 3) *vec_concatv2si_sse4_1 http://gcc.gnu.org/ml/gcc-patches/2016-05/msg02129.html Jakub

Re: [PATCH][ARM] PR target/70830: Avoid POP-{reglist}^ when returning from interrupt handlers

Ping. Thanks, Kyrill On 24/05/16 13:48, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2016-05/msg01211.html Thanks, Kyrill On 17/05/16 11:40, Kyrill Tkachov wrote: On 13/05/16 12:05, Kyrill Tkachov wrote: Hi Christophe, On 12/05/16 20:57, Christophe Lyon wrote: On 12

Re: [PATCH][ARM] PR target/70830: Avoid POP-{reglist}^ when returning from interrupt handlers

On 17/05/16 11:40, Kyrill Tkachov wrote: > > On 13/05/16 12:05, Kyrill Tkachov wrote: >> Hi Christophe, >> >> On 12/05/16 20:57, Christophe Lyon wrote: >>> On 12 May 2016 at 11:48, Ramana Radhakrishnan >>> wrote: On Thu, May 5, 2016 at 12:50 PM, Kyrill Tkachov

[PATCH] rs6000: Correct the "length" attribute for trunctddd2

Found in one of the ubsan testcases: a branch went out of range because of this incorrect length. Committing to mainline. Segher 2016-06-02 Segher Boessenkool * config/rs6000/dfp.md (trunctddd2): Correct the "length" attribute. ---

Re: Fix profile updating in loop peeling

On Mon, May 30, 2016 at 02:38:40PM +0200, Jan Hubicka wrote: > Hi, > this patch fixes profile updates in loop peeling pass. First it correctly > set wont_exit which can only be set when we know the number of iterations > tested by EXIT and this number is higher than maximal number of iterations >

Re: [PATCH][RTL ifcvt] PR rtl-optimization/66940: Avoid signed overflow in noce_get_alt_condition

On 05/23/2016 01:17 PM, Kyrill Tkachov wrote: In this PR we end up hitting a signed overflow in noce_get_alt_condition when we try to increment or decrement a HOST_WIDE_INT that might be HOST_WIDE_INT_MAX or HOST_WIDE_INT_MIN. I've confirmed the overflow by adding an assert before the

Re: [PATCH 1/2][GCC] Refactor noce_try_store_flag_constants

On 05/23/2016 03:58 PM, Mikhail Maltsev wrote: This patch refactors 'noce_try_store_flag_constants' a bit to make it easier to reason about. The function contains two series of conditions, and each branch of the second series corresponds to one or two branches of the first series. The patch

[PATCH v1] Support for SPARC M7 and VIS 4.0

[Changes from version 0 of the patch: - Added a comment to config/sparc/niagara4.md documenting the discrepancy of the documented instruction latency numbers vs. the implemented ones. - Added comments to sparc_option_override documenting the changes in the cache parameters. - Use the

Re: [PATCH] Support for SPARC M7 and VIS 4.0

> I think I will change v3pipe to v3pipe_m7 in this patch, to make it more > explicit that the insn<->v3pipe association is processor-dependant. No strong opinion, IOW it's your call. > Yes, it is intended. I will add a little note. Thanks. > I will also fix all the other points you raised. >

Re: [PATCH][ARM][1/4] Replace uses of int_log2 by exact_log2

I wanted to ping this patch, but checking the gcc-patches archive I see this wasn't archived, though I have confirmed that the patch was sent out and distributed by the mail server correctly. Anyway, resending... Thanks, Kyrill On 24/05/16 14:25, Kyrill Tkachov wrote: Hi all, The int_log2

Ping Re: Add option for whether ceil etc. can raise "inexact", adjust x86 conditions

Ping. This patch is pending review (for the non-x86-specific parts). -- Joseph S. Myers jos...@codesourcery.com

Fix up dg-set-compiler-env-var

Hi! On Fri, May 13, 2016 at 07:09:30PM +0200, Dhole wrote: > --- a/gcc/testsuite/lib/gcc-dg.exp > +++ b/gcc/testsuite/lib/gcc-dg.exp > @@ -451,6 +451,38 @@ proc restore-target-env-var { } { > } > } > > +proc dg-set-compiler-env-var { args } { I've noticed last night pr61861.c FAIL in

Re: [PATCH] Fix first match heuristics

> Hi. > > Following patch fixes a typo in first match heuristics that blocks > selection of best first match heuristics. > > It's questionable whether to add a spacial test-case for that? I'm bit > concerned that it can be a bit fragile. Lets add it now to have better coverage. If it will keep

Re: Fix up dg-set-compiler-env-var

On 06/02/2016 02:19 PM, Jakub Jelinek wrote: The problem is that in cleanup-after-saved-dg-test, there are missing global directives, so when it tests/uses the set_compiler_env_var and saved_compiler_env_var vars to see if it should call restore-compiler-env-var, it uses local (non-existing)

[PATCH 2/2] Add edge predictions pruning

contrib/ChangeLog: 2016-06-01 Martin Liska * analyze_brprob.py: Cover new dump output format. gcc/ChangeLog: 2016-06-01 Martin Liska * predict.c (dump_prediction): Add new argument. (enum predictor_reason): New enum.

[PATCH 0/2] Deduplicate edge predictors

Hi. Following small series makes deduplication of predictors in following 2 ways: 1) remove duplicate prediction that is guessed with the same probability (different than 1/2) to both edges 2) remove duplicates for a prediction that belongs with the same probability to a single

[PATCH 1/2] Introduce filtering for edge_predictions.

gcc/ChangeLog: 2016-05-31 Martin Liska * predict.c (filter_predictions): New function. (remove_predictions_associated_with_edge): Use the filter function. (equal_edge_p): New function. --- gcc/predict.c | 38

Re: Ping Re: Add option for whether ceil etc. can raise "inexact", adjust x86 conditions

On Thu, 2 Jun 2016, Jan Hubicka wrote: > > Ping. This patch > > is pending > > review (for the non-x86-specific parts). > The inliner bits looks fine to me. Of course it is easy to check whether the > function actually calls floor/ceil

Re: [PATCH] Warn about return with a void expression with -Wreturn-type.

2016-06-02 4:51 GMT+02:00 Martin Sebor : >> So here's my shot at fixing this in the documentation. Does that look >> okay? >> > > It looks good to me. Just one minor point below. > >> @@ -4055,8 +4055,12 @@ Warn whenever a function is defined with a >> return type that defaults

Re: [PATCH 00/21] Add -fself-test framework for fast, early unit-testing (unittests v5)

On 06/01/2016 11:19 PM, David Malcolm wrote: This is effectively v5 of the unittests proposal; for the earlier versions see: In general: nice to see this moving forward. There are some issues with the patch kit, some patches seem to fix issues with earlier ones (#3 or #13). One patch adds a

Re: [PATCH v4] gcov: Runtime configurable destination output

On 05/31/16 11:04, Aaron Conole wrote: Nathan Sidwell writes: On 05/26/16 13:08, Aaron Conole wrote: The previous gcov behavior was to always output errors on the stderr channel. This is fine for most uses, but some programs will require stderr to be untouched by libgcov for

Re: Fix up dg-set-compiler-env-var

On Thu, Jun 02, 2016 at 02:26:28PM +0200, Bernd Schmidt wrote: > On 06/02/2016 02:19 PM, Jakub Jelinek wrote: > >The problem is that in cleanup-after-saved-dg-test, there are missing global > >directives, so when it tests/uses the set_compiler_env_var and > >saved_compiler_env_var vars to see if

[PATCH] Fix first match heuristics

Hi. Following patch fixes a typo in first match heuristics that blocks selection of best first match heuristics. It's questionable whether to add a spacial test-case for that? I'm bit concerned that it can be a bit fragile. Bootstrapped and regtested on x86_64-linux. Ready for trunk? Thanks,

Re: Ping Re: Add option for whether ceil etc. can raise "inexact", adjust x86 conditions

> Ping. This patch > is pending > review (for the non-x86-specific parts). The inliner bits looks fine to me. Of course it is easy to check whether the function actually calls floor/ceil and thus is affected by this flag. Do you expect

Re: [PATCH] Support for SPARC M7 and VIS 4.0

> I will also fix all the other points you raised. > Thanks! You're welcome. And if you want to have commit rights to the SVN repository, you can put me as your sponsor (Eric Botcazou ). I just sent a request. Thank you.

Re: [PATCH] Improve *vec_concatv2si_sse4_1

On Thu, May 26, 2016 at 9:24 PM, Jakub Jelinek wrote: > On Thu, May 26, 2016 at 07:39:01PM +0200, Uros Bizjak wrote: >> On Thu, May 26, 2016 at 7:05 PM, Jakub Jelinek wrote: >> > Hi! >> > >> > This patch adds an avx512dq alternative (EVEX vpinsrd requires

[ipa-comdats] create a new comdat group for symbol if it's referenced from multiple comdat groups

Hi, I was trying to address first TODO from ipa-comdats.c (attached patch) TODO: When symbol is used only by comdat symbols, but from different groups, it would make sense to produce a new comdat group for it with anonymous name. The patch simply puts symbol in a new comdat group and makes symbol

[PR c/71381] C/C++ OpenACC cache directive rejects valid syntax (was: [gomp4] OpenACC cache directive for C.)

Hi! On Wed, 05 Nov 2014 17:29:19 +0100, I wrote: > In r217145, I applied Jim's patch to gomp-4_0-branch: > > commit 4361f9b6b2c74c2961c3a5290a4945abe2d7a444 > Author: tschwinge > Date: Wed Nov 5 16:26:47 2014 + > > OpenACC cache

Re: Ping Re: Add option for whether ceil etc. can raise "inexact", adjust x86 conditions

On Thu, 2 Jun 2016, Bernd Schmidt wrote: > On 06/02/2016 02:00 PM, Jan Hubicka wrote: > > > Ping. This patch > > > is pending > > > review (for the non-x86-specific parts). > > The inliner bits looks fine to me. > > In case that leaves

Re: move increase_alignment from simple to regular ipa pass

> Richard Biener wrote: >> "This would mean the pass should get its own non-Optimization flag >> initialized by targets where section anchors are usually used" >> IIUC should we add a new option -fno-increase_alignment and gate the >> pass on it ? Um sorry I didn't understand why targets >>

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On Thu, Jun 02, 2016 at 04:49:59PM +0200, Christophe Lyon wrote: > I'm also seeing that the new gcc.dg/cpp/source_date_epoch-1.c fails because > the output pattern finishes with '\n' instead of the usual '(\n|\r\n|\r)' > > Can I add this as obvious? Some remote test invocations even eat the

[PATCH][ARM][wwwdocs] Mention some arm port changes for GCC 6

Hi all, A bit belatedly, but here are some arm-related entries for the changes.html page for GCC 6. Is this ok to commit? Thanks, Kyrill Index: htdocs/gcc-6/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v

Re: [Patch, ARM] PR71061, length pop* pattern in epilogue correctly

On 31/05/16 15:15, Kyrill Tkachov wrote: +/* Compute the atrribute "length" of insn. Currently, this function is used + for "*load_multiple_with_writeback", "*pop_multiple_with_return" and + "*pop_multiple_with_writeback_and_return". */ + s/atrribute/attribute/ Also, please add a

[PATCH][wwwdocs][AArch64] Mention -mcpu=qdf24xx support for GCC 6

Hi all, As discussed some time ago with Jim, here's the AArch64 note mentioning the support for Qualcomm QDF24xx that was added in GCC 6. Ok to commit? Thanks, Kyrill Index: htdocs/gcc-6/changes.html === RCS file:

Re: [PATCH] Improve *vec_concatv4si

On Thu, May 26, 2016 at 7:07 PM, Jakub Jelinek wrote: > Hi! > > Both vpunpcklqdq and vmovhps are available with XMM EVEX args in AVX512VL. > > Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? > > 2016-05-26 Jakub Jelinek > > *

[C++ PATCH] Fix cp_fold dropping TREE_THIS_VOLATILE flag (PR c++/71372)

Hi! When cp_fold is called on INDIRECT_REF and ARRAY*_REF and any of the arguments change in the recursive calls, we fail to copy TREE_THIS_VOLATILE flag. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/6.2? 2016-06-02 Jakub Jelinek

Re: Ping Re: Add option for whether ceil etc. can raise "inexact", adjust x86 conditions

On 06/02/2016 02:00 PM, Jan Hubicka wrote: Ping. This patch is pending review (for the non-x86-specific parts). The inliner bits looks fine to me. In case that leaves anything unapproved, the remaining parts are OK too, modulo one

Re: move increase_alignment from simple to regular ipa pass

On Thu, 2 Jun 2016, David Edelsohn wrote: > > Richard Biener wrote: > > >> "This would mean the pass should get its own non-Optimization flag > >> initialized by targets where section anchors are usually used" > >> IIUC should we add a new option -fno-increase_alignment and gate the > >>

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On 2 June 2016 at 15:01, Christophe Lyon wrote: > On 1 June 2016 at 18:59, Matthias Klose wrote: >> On 01.06.2016 18:29, Bernd Schmidt wrote: >>> On 05/13/2016 07:09 PM, Dhole wrote: +pfile->source_date_epoch =

Re: [PATCH] c++/60760 - arithmetic on null pointers should not be allowed in constant expressions

On 06/01/2016 10:49 PM, Martin Sebor wrote: On 06/01/2016 01:20 PM, Jason Merrill wrote: On 06/01/2016 02:44 PM, Martin Sebor wrote: The new code in cxx_eval_component_reference diagnoses the following problem that's not detected otherwise: struct S { const S *s; }; constexpr S s = { 0

Re: [C++ PATCH] Fix cp_fold dropping TREE_THIS_VOLATILE flag (PR c++/71372)

On Thu, Jun 2, 2016 at 8:13 AM, Jakub Jelinek wrote: > When cp_fold is called on INDIRECT_REF and ARRAY*_REF and any of the > arguments change in the recursive calls, we fail to copy TREE_THIS_VOLATILE > flag. > > PR c++/71372 > * cp-gimplify.c (cp_fold): For

Re: move increase_alignment from simple to regular ipa pass

> On Thu, 2 Jun 2016, David Edelsohn wrote: > > > > Richard Biener wrote: > > > > >> "This would mean the pass should get its own non-Optimization flag > > >> initialized by targets where section anchors are usually used" > > >> IIUC should we add a new option -fno-increase_alignment and

Re: [middle-end][PATCH] Update alignment_for_piecewise_move

On Tue, Apr 26, 2016 at 11:39 AM, H.J. Lu wrote: > On Tue, Apr 26, 2016 at 11:31 AM, Bernd Schmidt wrote: >> On 04/26/2016 08:21 PM, Richard Sandiford wrote: >>> >>> "H.J. Lu" writes: I am working a patch to enable SSE,

Re: [PATCH, OpenACC] Make reduction arguments addressable

On 2016/6/1 09:38 PM, Jakub Jelinek wrote: >> construct_clauses.lists[OMP_LIST_REDUCTION] = NULL; >> >oacc_clauses = gfc_trans_omp_clauses (, _clauses, >> >code->loc); >> > + for (tree c = oacc_clauses; c; c = OMP_CLAUSE_CHAIN (c)) >> > +

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On Thu, Jun 02, 2016 at 03:01:07PM +0200, Christophe Lyon wrote: > On 1 June 2016 at 18:59, Matthias Klose wrote: > > On 01.06.2016 18:29, Bernd Schmidt wrote: > >> On 05/13/2016 07:09 PM, Dhole wrote: > >>> +pfile->source_date_epoch = > >>>

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On Thu, Jun 02, 2016 at 03:21:03PM +0200, Christophe Lyon wrote: > Ha, thanks, I had missed it. > > But why do I see "random" failures? > Looking at your patch, I have the impression that the test should always fail? Only with make -j1 -k check it should always FAIL. With -j2 and above, it

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On 1 June 2016 at 18:59, Matthias Klose wrote: > On 01.06.2016 18:29, Bernd Schmidt wrote: >> On 05/13/2016 07:09 PM, Dhole wrote: >>> +pfile->source_date_epoch = pfile->cb.get_source_date_epoch(pfile); >> >> Space before paren. Ok with that change. >> >>> *

Re: [PATCH 00/21] Add -fself-test framework for fast, early unit-testing (unittests v5)

On Wed, 2016-06-01 at 15:20 -0600, Sandra Loosemore wrote: > On 06/01/2016 03:19 PM, David Malcolm wrote: > > This is effectively v5 of the unittests proposal; for the earlier > > versions see: > > * v1: https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00765.html > > * v2:

Re: [PATCH, OpenACC] Make reduction arguments addressable

On Thu, Jun 02, 2016 at 09:55:05PM +0800, Chung-Lin Tang wrote: > fortran/ > * trans-openmp.c (gfc_trans_omp_clauses): Mark OpenACC reduction > arguments as addressable when async clause exists. Wouldn't it be better to pass either a bool openacc_async flag, or whole clauses,

Re: [PATCH, i386, AVX-512] Add vectorizer support builtins

On Mon, May 23, 2016 at 19:11:53 +0300, Ilya Verbin wrote: > This patch adds missed 512-bit rounding builtins for vectorization. > Regtested on x86_64-linux and i686-linux. OK for trunk? > > gcc/ > * config/i386/i386-builtin-types.def: Add V16SI_FTYPE_V16SF, > V8DF_FTYPE_V8DF_ROUND,

Re: Allow embedded timestamps by C/C++ macros to be set externally (3)

On 2 June 2016 at 15:05, Jakub Jelinek wrote: > On Thu, Jun 02, 2016 at 03:01:07PM +0200, Christophe Lyon wrote: >> On 1 June 2016 at 18:59, Matthias Klose wrote: >> > On 01.06.2016 18:29, Bernd Schmidt wrote: >> >> On 05/13/2016 07:09 PM, Dhole wrote: >> >>> +

Re: [PATCH, libstdc++/testsuite, ping] 29_atomics/atomic/65913.cc: require atomic-builtins rather than specific target

Ping? On Thursday 26 May 2016 14:00:55 Thomas Preudhomme wrote: > [Sorry for the large recipient list, I wasn't sure who of C++ and x86 > maintainers should approve this] > > Hi, > > 29_atomics/atomic/65913.cc test in libstdc++ is a runtime test that only > rely on atomic and gnu++11 support.

Re: [PATCH 00/21] Add -fself-test framework for fast, early unit-testing (unittests v5)

On Thu, 2016-06-02 at 12:29 +0200, Bernd Schmidt wrote: > On 06/01/2016 11:19 PM, David Malcolm wrote: > > This is effectively v5 of the unittests proposal; for the earlier > > versions see: > > In general: nice to see this moving forward. Thanks. > There are some issues with the patch kit,

  1   2   >