Re: [PATCH] Delete GCJ

> On 10 Oct 2016, at 05:03, Matthias Klose wrote: > > On 07.10.2016 10:30, Iain Sandoe wrote: >> >>> On 7 Oct 2016, at 00:58, Matthias Klose wrote: >>> >>> On 06.10.2016 20:00, Mike Stump wrote: On Oct 6, 2016, at 9:56 AM, Rainer Orth wrote: > I wouldn't hard-fail, but complet

[PATCH][store merging][RFA] Re-implement merging code

Hi Richard, As I mentioned, here is the patch applying to the main store merging patch to re-implement encode_tree_to_bitpos to operate on the bytes directly. This works fine on little-endian but breaks on big-endian, even for merging bitfields within a single byte. Consider the code snippet f

[PATCH][v5] GIMPLE store merging pass

Hi all, This is another revision of the pass addressing Richard's feedback [1] I believe I've addressed all of it and added more comments to the code where needed. The output_merged_store function now uses the new split_group helper to break up the merged store into multiple regular-sized stores

Re: [PATCH][simplify-rtx] Zero-initialise local array in simplify_immed_subreg

On 07/10/16 18:56, Andrew Pinski wrote: On Fri, Oct 7, 2016 at 10:55 AM, Andrew Pinski wrote: On Fri, Oct 7, 2016 at 7:08 AM, Kyrill Tkachov wrote: Hi all, I've encountered another wrong-code bug with the store merging pass. This time it's in RTL. The test gcc.target/aarch64/aapcs64/test_27

Re: [PATCH] Fix PR77826

On Fri, 7 Oct 2016, Richard Biener wrote: On Thu, 6 Oct 2016, Marc Glisse wrote: On Wed, 5 Oct 2016, Richard Biener wrote: The following will fix PR77826, the issue that in match.pd matching up two things uses operand_equal_p which is too lax about the type of the toplevel entity (at least f

[Ada] Fix wrong code with biased subtype

This is a regression present on all active branches for a subtype of a biased type declared with explicit constraints, which is itself biased. The compiler generates code that computes a wrong value for the conversion to an integer. Tested on x86_64-suse-linux, applied on all active branches.

Re: [AArch64][11/14] ARMv8.2-A FP16 testsuite selector

On Thu, Jul 07, 2016 at 05:18:41PM +0100, Jiong Wang wrote: > ARMv8.2-A adds support for scalar and vector FP16 instructions to ARM and > AArch64. This patch adds support for testing code for AArch64 targets > using the new instructions. It is based on the target-support code for > ARMv8.2-A added

Re: [patch] Fix GC issue triggered by arithmetic overflow checking

On Sat, Oct 8, 2016 at 8:56 PM, Eric Botcazou wrote: > Hi, > > adding patterns for unsigned arithmetic overflow checking in a back-end can > have unexpected fallout because of a latent GC issue: when they are present, > GIMPLE optimization passes can create complex (math. sense) types at will by >

Re: [VRP] Allocate bitmap before copying

On Sat, Oct 8, 2016 at 9:34 PM, kugan wrote: > Hi, > > In vrp_intersect_ranges_1, when !vr0->equiv, we are copying vr1->equiv > without allocating bitmap. This patch fixes this. > > Bootstrap and regression testing are ongoing. Is this OK if no new > regressions? Ok for trunk and branches. Richa

Re: [RFC][VRP] Improve intersect_ranges

On Sat, Oct 8, 2016 at 9:38 PM, kugan wrote: > Hi Richard, > > Thanks for the review. > On 07/10/16 20:11, Richard Biener wrote: >> >> On Fri, Oct 7, 2016 at 12:00 AM, kugan >> wrote: >>> >>> Hi, >>> >>> In vrp intersect_ranges, Richard recently changed it to create integer >>> value >>> ranges w

[Ada] Fix inter-unit inlining failure

This is a regression present on the mainline and 6 branch: the compiler fails to inline across units a function declared with pragma Inline_Always because the middle-end detects a type mismatch for an argument, after gimplification removed a conversion. The fix is to make the conversion more ro

Re: Compile-time improvement for if conversion.

On Wed, Oct 5, 2016 at 3:22 PM, Yuri Rumyantsev wrote: > Hi All, > > Here is implementation of Richard proposal: > > < For general infrastructure it would be nice to expose a (post-)dominator > < compute for MESE (post-dominators) / SEME (dominators) regions. I believe > < what makes if-conversio

Re: [PATCH, PR77558] Remove RECORD_TYPE special-casing in std_canonical_va_list_type

On Sun, Sep 25, 2016 at 11:08 AM, Tom de Vries wrote: > Hi, > > this patch fixes PR77558, an ice-on-invalid-code 6/7 regression. > > The fix for PR71602 introduced the invalid-code test-case > c-c++-common/va-arg-va-list-type.c: > ... > __builtin_va_list *pap; > > void > fn1 (void) > { > __built

Re: [AArch64][12/14] ARMv8.2-A testsuite for new data movement intrinsics

On Thu, Jul 07, 2016 at 05:19:09PM +0100, Jiong Wang wrote: > This patch contains testcases for those new scalar intrinsics which are only > available for AArch64. OK. Thanks, James > > gcc/testsuite/ > 2016-07-07 Jiong Wang > > * gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h

Re: PING: [PATCH] Be more conservative in early inliner if FDO is enabled

On Mon, Oct 10, 2016 at 4:23 AM, Yuan, Pengfei wrote: > Hi, > > What is the decision on this patch? > https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01041.html Honza approved the patch already. Richard. > Regards, > Yuan, Pengfei > >> A new patch for trunk is attached. >> >> Regards, >> Yuan, Pe

Re: [AArch64][13/14] ARMv8.2-A testsuite for new vector intrinsics

On Thu, Jul 07, 2016 at 05:19:25PM +0100, Jiong Wang wrote: > This patch contains testcases for those new vector intrinsics which are only > available for AArch64. OK. Thanks, James > gcc/testsuite/ > 2016-07-07 Jiong Wang > > * gcc.target/aarch64/advsimd-intrinsics/vdiv_f16_1.c: New

Re: [AArch64][14/14] ARMv8.2-A testsuite for new scalar intrinsics

On Thu, Jul 07, 2016 at 05:19:37PM +0100, Jiong Wang wrote: > This patch contains testcases for those new scalar intrinsics which are only > available for AArch64. OK. Thanks, James > gcc/testsuite/ > 2016-07-07 Jiong Wang > > * gcc.target/aarch64/advsimd-intrinsics/unary_scalar_op.in

[Ada] Fix type checking failure with pragma Volatile_Full_Access

The problem is that we put an alias set on a variant that is not the main one. Tested on x86_64-suse-linux, applied on the mainline. 2016-10-10 Eric Botcazou * gcc-interface/decl.c (gnat_to_gnu_entity): Put volatile qualifier on types at the very end of the processing.

Re: [PATCH][store merging][RFA] Re-implement merging code

On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > Hi Richard, > > As I mentioned, here is the patch applying to the main store merging patch to > re-implement encode_tree_to_bitpos > to operate on the bytes directly. > > This works fine on little-endian but breaks on big-endian, even for merging > bi

Re: [PATCH][store merging][RFA] Re-implement merging code

On Mon, 10 Oct 2016, Richard Biener wrote: > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > Hi Richard, > > > > As I mentioned, here is the patch applying to the main store merging patch > > to > > re-implement encode_tree_to_bitpos > > to operate on the bytes directly. > > > > This works fi

Re: [patch] Fix GC issue triggered by arithmetic overflow checking

> I believe the rule is that you might only depend on the order of objects > with respect to their DECL_UID, not the actual value of the DECL_UID. > As var-tracking shouldn't look at TYPE_DECLs (?) it's probably a latent > var-tracking bug as well. It presumably doesn't look at TYPE_DECLs, simply

Re: [AArch64][0/14] ARMv8.2-A FP16 extension support

On Wed, Oct 05, 2016 at 05:44:08PM +0100, Jiong Wang wrote: > On 27/09/16 17:03, Jiong Wang wrote: > > > > Now as ARM patches have gone in around r240427, I have done a > quick confirmation > > on the status of these four pending testsuite patches: > > > > https://gcc.gnu.org/ml/gcc-patches/2016-

Re: [patch] Fix GC issue triggered by arithmetic overflow checking

On Mon, Oct 10, 2016 at 12:38 PM, Eric Botcazou wrote: >> I believe the rule is that you might only depend on the order of objects >> with respect to their DECL_UID, not the actual value of the DECL_UID. >> As var-tracking shouldn't look at TYPE_DECLs (?) it's probably a latent >> var-tracking bug

Re: [patch] Fix GC issue triggered by arithmetic overflow checking

On Mon, Oct 10, 2016 at 12:38 PM, Eric Botcazou wrote: >> I believe the rule is that you might only depend on the order of objects >> with respect to their DECL_UID, not the actual value of the DECL_UID. >> As var-tracking shouldn't look at TYPE_DECLs (?) it's probably a latent >> var-tracking bug

Fix invalid doloop setup on ia64 (PR target/77738)

On ia64 the doloop pattern can only work with DImode, so it should reject any other mode. Bootstrapped and regtested on ia64-suse-linux. Andreas. PR target/77738 * config/ia64/ia64.md ("doloop_end"): Reject if mode of loop pseudo is not DImode. --- gcc/confi

Re: PING: [PATCH] Be more conservative in early inliner if FDO is enabled

> On Mon, Oct 10, 2016 at 4:23 AM, Yuan, Pengfei wrote: > > Hi, > > > > What is the decision on this patch? > > https://gcc.gnu.org/ml/gcc-patches/2016-09/msg01041.html > > Honza approved the patch already. > > Richard. Do I need to sign a copyright assignment for the patch? Moreover, I do not

Re: [PATCH 2/3] Fold __builtin_memchr (version 2)

On 10/07/2016 01:21 PM, Wilco Dijkstra wrote: > Hi, > >> -static int >> +int >> target_char_cast (tree cst, char *p) > >> + if (target_char_cast (arg2, &c)) >> +return false; > > I believe target_char_cast is incorrect if the host/target chars are not > identical > (depending on how co

Re: Fix invalid doloop setup on ia64 (PR target/77738)

On 10/10/2016 12:51 PM, Andreas Schwab wrote: On ia64 the doloop pattern can only work with DImode, so it should reject any other mode. Bootstrapped and regtested on ia64-suse-linux. Andreas. PR target/77738 * config/ia64/ia64.md ("doloop_end"): Reject if mode of loop

Re: [PATCH][store merging][RFA] Re-implement merging code

On 10/10/16 11:22, Richard Biener wrote: On Mon, 10 Oct 2016, Kyrill Tkachov wrote: Hi Richard, As I mentioned, here is the patch applying to the main store merging patch to re-implement encode_tree_to_bitpos to operate on the bytes directly. This works fine on little-endian but breaks on bi

Re: [PATCH][store merging][RFA] Re-implement merging code

On 10/10/16 12:06, Kyrill Tkachov wrote: On 10/10/16 11:22, Richard Biener wrote: On Mon, 10 Oct 2016, Kyrill Tkachov wrote: Hi Richard, As I mentioned, here is the patch applying to the main store merging patch to re-implement encode_tree_to_bitpos to operate on the bytes directly. This w

Re: [PATCH][store merging][RFA] Re-implement merging code

On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > On 10/10/16 11:22, Richard Biener wrote: > > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > > > Hi Richard, > > > > > > As I mentioned, here is the patch applying to the main store merging patch > > > to > > > re-implement encode_tree_to_bitpos >

Re: [PATCH][store merging][RFA] Re-implement merging code

On Mon, 10 Oct 2016, Richard Biener wrote: > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > > > On 10/10/16 11:22, Richard Biener wrote: > > > On Mon, 10 Oct 2016, Kyrill Tkachov wrote: > > > > > > > Hi Richard, > > > > > > > > As I mentioned, here is the patch applying to the main store mer

Re: [PATCH] Implement C++17 node extraction and insertion (P0083R5)

On 21/09/16 14:48 +0100, Jonathan Wakely wrote: This implements container node extraction/insertion, and merging. The patch includes Debug Mode support and pretty printers for the node handles. Most of the changes are fairly straightforward, with two things worth pointing out. There's a FIXME i

Re: [PATCH 2/3] Fold __builtin_memchr (version 2)

Martin Liška wrote: > On 10/07/2016 01:21 PM, Wilco Dijkstra wrote: > > > I believe target_char_cast is incorrect if the host/target chars are not > > identical > > (depending on how constant strings are created there may be signed/unsigned > > mismatches too). I recently added target_char_cst_p

Re: [PATCH 2/3] Fold __builtin_memchr (version 2)

On 10/10/2016 01:28 PM, Wilco Dijkstra wrote: > Martin Liška wrote: >> On 10/07/2016 01:21 PM, Wilco Dijkstra wrote: >> >>> I believe target_char_cast is incorrect if the host/target chars are not >>> identical >>> (depending on how constant strings are created there may be signed/unsigned >>> mi

RE: [PATCH] [ARC] Disable compact casesi patterns for arcv2

> > gcc/ > > 2016-05-09 Claudiu Zissulescu > > > > * common/config/arc/arc-common.c > (arc_option_optimization_table): > > Remove compact casesi option. > > * config/arc/arc.c (arc_override_options): Use compact casesi > > option only for pre-ARCv2 cores. > > * doc/invoke.te

Re: Compile-time improvement for if conversion.

Thanks Richard for your comments. I'd like to answer on your last comment regarding use split_edge() instead of creating fake post-header. I started with this splitting but it requires to fix-up closed ssa form by creating additional phi nodes, so I decided to use only cfg change without updating s

[PATCH] Add noexcept to enable_shared_from_this::weak_from_this

I missed out the "noexcept" on these new functions. * include/bits/shared_ptr.h (enable_shared_from_this::weak_from_this): Add noexcept. * include/bits/shared_ptr_base.h (__enable_shared_from_this::weak_from_this): Likewise. * testsuite/20_util/enable_share

Re: Compile-time improvement for if conversion.

On Mon, Oct 10, 2016 at 1:42 PM, Yuri Rumyantsev wrote: > Thanks Richard for your comments. > I'd like to answer on your last comment regarding use split_edge() > instead of creating fake post-header. I started with this splitting > but it requires to fix-up closed ssa form by creating additional

[PATCH] Define std::allocator::is_always_equal

I somehow only added the is_always_equal nested typedef to the allocator specialization, not the primary template. All the containers still do the right thing, because they use allocator_traits>::is_always_equal which gives the right answer, but we still need to provide allocator::is_always_equal

[PATCH] LWG 2733, LWG 2759 reject bool in gcd and lcm

These DRs are only in Tentatively Ready status, but they're not controversial so implementing them immediately seems sensible. The deleted function is sufficient, but the static assertions are more user-friendly (and are only tested once, not in every recursive call to __gcd or __lcm). *

[PATCH] Implement LWG 2192 and LWG 2294 for std::abs

It looks like I forgot to send this patch to the lists last month. This implements the requirements that all overloads of std::abs are declared by either of or . This ensures that including only one of those headers and calling std::abs doesn't cause conversions from integers to floating point t

Re: [PATCH, ARM 5/7] Add support for MOVT/MOVW to ARMv8-M Baseline

Hi Thomas, On 13 July 2016 at 17:34, Thomas Preudhomme wrote: > On Wednesday 13 July 2016 17:14:52 Christophe Lyon wrote: >> Hi Thomas, > > Hi Christophe, > >> >> I'm seeing: >> gcc.target/arm/pr42574.c: syntax error in target selector >> "arm_thumb1_ok && { ! arm_thumb1_movt_ok }" for " dg-do 1

[PATCH] [ARC] New option handling, refurbish multilib support.

Hi Andrew, This is updated patch of the original sent to mailing list some while ago. What is new: - Do not use MULTILIB_REUSE as its semantic changed, and the old one was causing issues while building. - Update invoke.texi documentation adding nps400 option to mcpu. This patch is imp

[patch,avr] Use avr-passes.def to register passes.

This is a code clean-up using the new -passes.def feature in order to register avr target passes and to get -fdump-xxx etc. to work for such passes. Ok for trunk? Johann * config/avr/avr-passes.def: New file. * config/avr/t-avr (PASSES_EXTRA): Add avr-passes.def. * conf

Re: Compile-time improvement for if conversion.

Richard, If "fake" exit or entry block is created in dominance how we can determine what is its the only predecessor or successor without using a notion of loop? 2016-10-10 15:00 GMT+03:00 Richard Biener : > On Mon, Oct 10, 2016 at 1:42 PM, Yuri Rumyantsev wrote: >> Thanks Richard for your comm

[avr,committed] Include string.h in gen-avr-mmcu-texi.c

https://gcc.gnu.org/r240925 https://gcc.gnu.org/r240926 https://gcc.gnu.org/r240927 gen-avr-mmcu-texi.c missed the inclusion of string.h (for strcmp). Applied as obvious. Johann * config/avr/gen-avr-mmcu-texi.c (string.h): Include. Index: config/avr/gen-avr-mmcu-texi.c

[patch, fortran, committed] Fix PR 77915

Hello world, I have committed the attached patch to trunk as obvious after regression-testing. Will commit to gcc6 soon. Regards Thomas 2016-10-10 Thomas Koenig PR fortran/77915 * frontend-passes.c (inline_matmul_assign): Return early if inside a FORALL sta

Re: [PATCH v4 0/6] Separate shrink-wrapping

Ping. On Mon, Oct 03, 2016 at 01:48:17PM +, Segher Boessenkool wrote: > I updated according to Jeff's latest comments (importantly, we cannot > move a *logue in front of a move in general), and added some testcases. > > Bootstrapping is in progress on today's trunk, powerpc64-linux and > powe

PING! Re: [PATCH, Fortran] Extension: COTAN and degree-valued trig intrinsics with -fdec-math

https://gcc.gnu.org/ml/fortran/2016-09/msg00163.html [original] https://gcc.gnu.org/ml/fortran/2016-09/msg00183.html [latest] On Wed, Sep 28, 2016 at 4:14 PM, Fritz Reese wrote: > Attached is a patch extending the GNU Fortran front-end to support > some additional math intrinsics, enabled with a

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

On 09/25/2016 07:25 PM, Arnaud Charlet wrote: int __gnat_get_maximum_file_name_length (void) { +#if defined (__DJGPP__) + return (_use_lfn(".")) ? -1 : 8; +#else return -1; +#endif } Is the above change really necessary? Would be nice to get rid of this extra code. The rest looks OK

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

> >> int > >> __gnat_get_maximum_file_name_length (void) > >> { > >>+#if defined (__DJGPP__) > >>+ return (_use_lfn(".")) ? -1 : 8; > >>+#else > >>return -1; > >>+#endif > >> } > >Is the above change really necessary? Would be nice to get rid of this > >extra code. The rest looks OK to me.

[PATCH] Implement constexpr std::addressof for C++17

Thank to the new __builtin_addressof that Jakub added we can do this now. * doc/xml/manual/intro.xml: Document DR 2296 status. * doc/xml/manual/status_cxx2017.xml: Update status. * include/bits/move.h (__addressof): Add _GLIBCXX_CONSTEXPR and call __builtin_address

[Committed] S/390: Wrap more macro args into ()

Turned out that there where a few () around macro args uses missing. One real problem with it was detected with the int-in-bool-context in the definition of DBX_REGISTER_NUMBER. But while being at it I've also tried to fix other places where brackets might be missing. gcc/ChangeLog: 2016-10-10 A

[PING][PATCH 1/4][Ada,DJGPP] Ada support for DJGPP

I'd like to ping this patch. Last version of the patch together with Changelog entry can be found in mailing list archive: https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01229.html Andris

Re: [Committed] S/390: Wrap more macro args into ()

On Okt 10 2016, Andreas Krebbel wrote: > @@ -491,7 +491,7 @@ extern const char *s390_host_detect_local_cpu (int argc, > const char **argv); >s390_hard_regno_mode_ok ((REGNO), (MODE)) > > #define HARD_REGNO_RENAME_OK(FROM, TO) \ > - s390_hard_regno_rename_ok (FROM, TO) > + s390_

[PING][PATCH 3/4][Ada,DJGPP] Ada support for DJGPP

I'd like to ping patch https://gcc.gnu.org/ml/gcc-patches/2016-09/msg00164.html Additional comments about using ZCX_By_Default := true are in https://gcc.gnu.org/ml/gcc-patches/2016-09/msg00845.html Andris

Re: [PATCH 4/4][Ada,DJGPP] Ada support for DJGPP

On 10/10/2016 06:22 PM, Arnaud Charlet wrote: PS. What about last versions of other 2 not yet approved patches (1 and 3)? There have been many back and forth and many updates, so I do not know where we are on these. I'm pretty sure I OKed one of the other parts, but best to resubmit them clean

[hsa-branch 4/9] Add expansion of reciprocal of square root

Hi, this patch is a simple addition of reciprocal of square root gimple function into its HSAIL equivalent. Committed to the branch, queued for merge to trunk soon. Thanks, Martin 2016-10-03 Martin Jambor * hsa-gen.c (gen_hsa_insn_for_internal_fn_call): Also handle IFN_RSQRT. --- g

[hsa-branch 2/9] Lastprivate lowering for gridified kernels

Hi, this patch implements the lastprivate data sharing clauses of gridified OpenMP looping constructs. It adds code to construct a special condition to identify he "last" loop iteration using special HSA instructions, because that way we do not need information about all HSA dimensions conveyed f

[hsa-branch 3/9] Handle simds within gridified loops gracefully

Hi, this patch deals with simd constructs in gridified OpenMP loops. Standalone simds are dealt with by forcing the gridified copy to have OMP_CLAUSE_SAFELEN_EXPR of one, while simds which are a part of a combined construct with the gridified parallel loop are simply discarded. Committed to the b

[hsa-branch 6/9] Expand FMA_EXPR to HSAIL

Hi, the following patch adds expansion of fused multiply and add to HSAIL. The scalar variant is straightforwardly converted to an HSAIL equivalent while any vector instance is expanded into separate multiplication and additions. Committed to the branch, queued for merge to trunk soon. Thanks, M

[hsa-branch 1/9] Builtins for gridsize and currentworkgroupsize

Hi, the patch below makes the griddim and currentworkgroupsize special HSA instructions available for omp lowering through a builtin. They are then used by subsequent patch to implement conditions determining the last iteration for the lastprivate OpenMP sharing clause. Committed to the branch,

[hsa-branch 5/9] Properly detect variadic arguments

Hi, this patch from Martin properly detects some variadic calls which we have failed to detect before during expansion to HSAIL. Committed to the branch, queued for merge to trunk soon. Thanks, Martin 2016-10-03 Martin Liska Martin Jambor * hsa-gen.c (verify_function_a

[hsa-branch 7/9] Ignore prefetch builtin

Hi, this patch makes HSAIL expansion ignore prefetch built-ins. It is a bit less straightforward because we also need to handle cases where the call does not pass gimple_call_builtin_p test because of argument type mismatches. Committed to the branch, queued for merge to trunk soon. Thanks, Mar

[hsa-branch 8/9] Fail instead of calling an unknown GOMP builtin

Hi, this patch is a bit of a hack to make sure we do not emit calls to libgomp run-time functions which are not available at the HSA GPU side, such as run-time loop scheduling routines. If we fail at the caller side, we avoid issues with finalizer looking at calls to non-existing functions. Comm

[hsa-branch 9/9] Fix another finalizer type complaint

Hi, the subsequent patch deals with a finalizer error issued when we ave a register-register move of an HSAIL vector type. Apparently, such a move must obey the same rules as vector loads and stores. Committed to the branch, queued for merge to trunk soon. Thanks, Martin 2016-10-03 Martin Jam

Re: [PATCH] Improve performance of list::reverse

On 09/10/16 16:23 +0100, Elliot Goodrich wrote: Hi, If we unroll the loop so that we iterate both forwards and backwards, we can take advantage of memory-level parallelism when chasing pointers. This means that reverse takes 35% less time when nodes are randomly scattered in memory and about the

[PATCH] Minor simplification to std::_Bind_result helpers

We don't need to define new class templates for the SFINAE helpers in _Bind_result, we can just use alias templates. This also moves where the helpers are used to the return types, instead of as a defaulted argument. * include/std/functional (_Bind_result::__enable_if_void): Use alias

Go patch committed: remove GCC-specific linemap usage

This patch by Than McIntosh removes a GCC-specific use of the linemap code to retrieve the line number. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian 2016-10-10 Than McIntosh * go-linemap.cc (Gcc_linemap::location_line): New method. Index: gcc/go/go-li

Re: [v3 PATCH] Make any's copy assignment operator exception-safe, don't copy the underlying value when any is moved, make in_place constructors explicit.

On 08/10/16 16:07 +0300, Ville Voutilainen wrote: Tested on Linux-x64. 2016-10-08 Ville Voutilainen Make any's copy assignment operator exception-safe, don't copy the underlying value when any is moved, make in_place constructors explicit. * include/std/any (any(in_place_type_t<_

Re: [PATCH] Implement new hook for max_align_t_align

Attached is an updated version using the new builtin __MAX_ALIGN_T_ALIGN__. This simplifies the declaration of max_align_t and ensures it is always the same as max_align_t_align(). Tested on hppa-unknown-linux-gnu. Okay for trunk? Dave -- John David Anglin dave.ang...@bell.net 2016-10

Re: [v3 PATCH] Make any's copy assignment operator exception-safe, don't copy the underlying value when any is moved, make in_place constructors explicit.

On 10/10/16 19:19 +0100, Jonathan Wakely wrote: On 08/10/16 16:07 +0300, Ville Voutilainen wrote: Tested on Linux-x64. 2016-10-08 Ville Voutilainen Make any's copy assignment operator exception-safe, don't copy the underlying value when any is moved, make in_place constructors explici

[PATCH] Use noexcept instead of _GLIBCXX_USE_NOEXCEPT

This file is compiled with -std=gnu++11 so there's no need to use the macro, we can use noexcept directly. * libsupc++/eh_ptr.cc (exception_ptr): Replace _GLIBCXX_USE_NOEXCEPT with noexcept. Tested powerpc64le-linbux, committed to trunk. commit f80e14a01697a34b835638a303967c0a7

[PATCH] Correct C++11 implementation status docs

The std::list allocator status and the note about timed mutexes are out of date, those are both completely implemented now (there's a fallback timed mutex for targets without _POSIX_TIMEOUTS). * doc/xml/manual/status_cxx2011.xml: Correct C++11 status. Committed to trunk. I'll backport th

Re: [PATCH 10/16] Introduce class function_reader (v3)

David Malcolm writes: > On Wed, 2016-10-05 at 18:00 +0200, Bernd Schmidt wrote: >> On 10/05/2016 06:15 PM, David Malcolm wrote: >> >* errors.c: Use consistent pattern for bconfig.h vs config.h >> >includes. >> >(progname): Wrap with #ifdef GENERATOR_FILE. >> >(error): Likewise. Ad

[PATCH] Fix PR77824

Hi, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77824 reports unreachable code where MODIFY_EXPR is being tested instead of SSA_NAME to identify RHS's for copies. This patch corrects that. I instrumented the compiler to identify copies being added to the candidate table, and found that this n

[PATCH] Update docs on libstdc++ source-code layout

Self-explanatory updates to the docs, and regenerating after the various recent changes. * doc/xml/manual/appendix_contributing.xml (contrib.organization): Describe other subdirectories and add markup. Remove outdated reference to check-script target. * doc/html/*:

Re: [v3 PATCH] Make any's copy assignment operator exception-safe, don't copy the underlying value when any is moved, make in_place constructors explicit.

On 10 October 2016 at 21:19, Jonathan Wakely wrote: > I prefer to put "explicit" on a line of its own, as we do for return > types, but I won't complain if you leave it like this. Changed. >> + any(__rhs).swap(*this); > > > I was trying to avoid the "redundant" xfer operations that the s

Re: [PATCH] 77864 Fix noexcept conditions for map/set default constructors

On 09/10/2016 17:14, Jonathan Wakely wrote: On 08/10/16 22:55 +0200, François Dumont wrote: On 06/10/2016 23:34, Jonathan Wakely wrote: On 06/10/16 22:17 +0200, François Dumont wrote: Another approach is to rely on existing compiler ability to compute conditional noexcept when defaulting imple

Re: PING! Re: [PATCH, Fortran] Extension: COTAN and degree-valued trig intrinsics with -fdec-math

On 10/10/2016 08:06 AM, Fritz Reese wrote: > https://gcc.gnu.org/ml/fortran/2016-09/msg00163.html [original] > https://gcc.gnu.org/ml/fortran/2016-09/msg00183.html [latest] > > On Wed, Sep 28, 2016 at 4:14 PM, Fritz Reese wrote: >> Attached is a patch extending the GNU Fortran front-end to suppor

Move OVERRIDE/FINAL from gcc/coretypes.h to include/ansidecl.h (was: Re: [PATCH 1/2] Add OVERRIDE and FINAL macros to coretypes.h)

Please find below a patch moving the FINAL/OVERRIDE macros to include/ansidecl.h, as I was suggesting in the earlier discussion: On 05/06/2016 07:33 PM, Trevor Saunders wrote: > On Fri, May 06, 2016 at 07:10:33PM +0100, Pedro Alves wrote: >> On 05/06/2016 06:56 PM, Pedro Alves wrote: >> I was goi

New Swedish PO file for 'gcc' (version 6.2.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-6.2.0.sv.po', has just

[libgo] Silence compiler error message

Hi, on Solaris the configuration of the library yields an ugly: checking whether linker supports split/non-split linked together... cc1: error: '-fsplit-stack' is not supported by this compiler configuration xgcc: error: conftest1.o: No such file or directory no Tested on x86-64/Linux and SPARC

Re: PING! Re: [PATCH, Fortran] Extension: COTAN and degree-valued trig intrinsics with -fdec-math

On Mon, Oct 10, 2016 at 12:29:32PM -0700, Jerry DeLisle wrote: > On 10/10/2016 08:06 AM, Fritz Reese wrote: > > https://gcc.gnu.org/ml/fortran/2016-09/msg00163.html [original] > > https://gcc.gnu.org/ml/fortran/2016-09/msg00183.html [latest] > > > > On Wed, Sep 28, 2016 at 4:14 PM, Fritz Reese wr

[PATCH], PR 77924, Fix PowerPC breakage on AIX

I accidently broke AIX with my patch on October 6th. That patch split -mfloat128 into -mfloat128-type and -mfloat128 under PowerPC Linux. This patch fixes that issue. I bootstrapped it on PowerPC Linux with no regressions, and David Edelsohn reports that it fixes the problem on AIX. Is it ok to

Fwd: C++ PATCH for c++/77890, 77912 (C++17 class deduction issues)

77890: we were losing the CLASS_PLACEHOLDER_TEMPLATE when reducing the level of a TEMPLATE_TYPE_PARM. 77912: after 77890 was fixed, we were complaining about an undefined deduction guide; set cp_unevaluated_operand to prevent that. Tested x86_64-pc-linux-gnu, applying to trunk. commit 927a7c6142c

Re: [PATCH] 77864 Fix noexcept conditions for map/set default constructors

Trying again...with a few edits. > On Mon, Oct 10, 2016 at 3:24 PM, François Dumont > wrote: > > @@ -602,24 +612,32 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION > struct _Rb_tree_impl : public _Node_allocator > { >_Key_compare _M_key_compare; > - _Rb_tree_node_base _M_header; > + _

[tree-optimization/71947] Avoid unwanted propagations

So if we have an equality conditional between A & B, we record into our const/copy tables A = B and B = A. This helps us discover some of the more obscure equivalences. But it also creates problems with an expression like A ^ B Where we might cprop the first operand generating B ^ B The

[patch] aarch64-*-freebsd* support for gcc.

Hi all, the attached patch brings support for the aarch64-*-freebsd* target. Bootstraped and tested, results on the list. Not that many results due to board instabilities I lack a cavium ;) Ok for main? And if yes, how far can I backport? Down to 5.4? TIA, Andreas libgcc: 2016-10-10 A

Re: [patch] aarch64-*-freebsd* support for gcc.

On 10/10/2016 03:07 PM, Andreas Tobler wrote: Hi all, the attached patch brings support for the aarch64-*-freebsd* target. Bootstraped and tested, results on the list. Not that many results due to board instabilities I lack a cavium ;) Ok for main? And if yes, how far can I backport? Down

Re: [PATCH 4/5] shrink-wrap: Shrink-wrapping for separate components

On 09/30/2016 04:34 AM, Segher Boessenkool wrote: [ whoops, message too big, resending with the attachment compressed ] On Tue, Sep 27, 2016 at 03:14:51PM -0600, Jeff Law wrote: With transposition issue addressed, the only blocker I see are some simple testcases we can add to the suite. They d

Re: PATCH to introduce c-family/c-warn.c

On 10/10/2016 10:36 AM, Marek Polacek wrote: As outlined recently, this patch creates a new c-warn.c file, where various diagnostic routines should reside, making c-common.c a little bit shorter. There are no function changes though. While at it, I fixed all tabs/space problems in those function

Re: [patch] aarch64-*-freebsd* support for gcc.

On 10.10.16 23:10, Jeff Law wrote: On 10/10/2016 03:07 PM, Andreas Tobler wrote: Hi all, the attached patch brings support for the aarch64-*-freebsd* target. Bootstraped and tested, results on the list. Not that many results due to board instabilities I lack a cavium ;) Ok for main? And i

Re: [PATCH, C++] Warn on redefinition of builtin functions (PR c++/71973)

On 10/06/16 22:37, Bernd Edlinger wrote: > On 10/06/16 16:14, Kyrill Tkachov wrote: >> >> @@ -1553,7 +1588,7 @@ duplicate_decls (tree newdecl, tree olddecl, bool >> >> /* Whether or not the builtin can throw exceptions has no >>bearing on this declarator. */ >> - TREE_NOTHROW

Re: [v3 PATCH] Make any's copy assignment operator exception-safe, don't copy the underlying value when any is moved, make in_place constructors explicit.

On 10/10/16 22:21 +0300, Ville Voutilainen wrote: This code was all pretty carefully written to avoid any redundant operations. Does this change buy us anything except simpler code? As discussed, destroying the value but leaving the manager non-null will do bad things. Oops again on my part!

Re: [PATCH 4/5] shrink-wrap: Shrink-wrapping for separate components

On Mon, Oct 10, 2016 at 03:21:31PM -0600, Jeff Law wrote: > On 09/30/2016 04:34 AM, Segher Boessenkool wrote: > >[ whoops, message too big, resending with the attachment compressed ] > > > >On Tue, Sep 27, 2016 at 03:14:51PM -0600, Jeff Law wrote: > >>With transposition issue addressed, the only bl

Re: [PATCH] Update docs on libstdc++ source-code layout

On 10/10/16 19:57 +0100, Jonathan Wakely wrote: Self-explanatory updates to the docs, and regenerating after the various recent changes. * doc/xml/manual/appendix_contributing.xml (contrib.organization): Describe other subdirectories and add markup. Remove outdated refere

Re: Always support float128 on ia64 (PR target/77586)

On 10/04/2016 10:46 AM, Joseph Myers wrote: Bug 77586, and previously , reports ia64-elf failing to build because of float128_type_node being NULL, but being used by the back end for __float128. The global float128_type_node is only availabl

Re: [PATCH] Improve performance of list::reverse

I haven't yet but I will try and sort it out tomorrow. If we're replacing the current method with one that takes a size parameter when _GLIBCXX_USE_CXX11_ABI is defined, is this going to cause any issues with ABI compatibility? If not, then I agree that we should go with the #if version. On 10 Oc

  1   2   >