[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

[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. *

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

[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

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 && { !

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

[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. *

[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

[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).

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

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

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

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

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.

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

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

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.

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

[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

[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

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. >> >

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

[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

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. *

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

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

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

[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

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

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

[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

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

[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. *

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

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

[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

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

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

Re: [RFC][VRP] Improve intersect_ranges

Hi Richard, On 10/10/16 20:13, Richard Biener wrote: 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:

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

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; > -

[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

[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

[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

[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

[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,

[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

[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,

[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-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.

[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

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

[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

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

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

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.

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] 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

[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

[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

[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: [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 >

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] 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

Go patch committed: move Backend/Linemap creation out of frontend

This patch by Than McIntosh moves the calls that create the GCC-specific Backend and Linemap objects out of the Go frontend into the gccgo interface code. This allows for more flexibility creating those objects. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline.

[PING] [PATCH] fix outstanding -Wformat-length failures (pr77735 et al.)

I'm looking for a review of the patch below: https://gcc.gnu.org/ml/gcc-patches/2016-10/msg00043.html The patch should clean up the remaining test suite failures on ILP32 targets and also fixes up some remaining issues in the gimple-ssa-sprintf pass that stand in the way of re-enabling the

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

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

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

Go patch committed: copy print code from Go 1.7 runtime

This patch copies the code that implements the print and println predeclared functions from the Go 1.7 runtime. The compiler is changed to use the new names, and to call the printlock and printunlock functions around a sequence of print calls. The writebuf field in the g struct changes to a

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

[Patch, x86_64] Fix znver1 imov/imovx load reservations.

Hi Maintainers, The below patch fixes integer load type reservations for -march=znver1. Bootstrapped and regtested on x86_64-pc-linux-gnu. Ok to commit to trunk ? (-Snip) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 9659fbf..19b4066 100644 --- a/gcc/ChangeLog +++

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 > > *

[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

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 >

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

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: > > > >

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

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

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 > > *

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. >> >>

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 >

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 > > *

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, )) >> +return false; > > I believe target_char_cast is incorrect if the host/target chars are not > identical > (depending on how

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 > <

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

[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

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

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. ---

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?

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 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

[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

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 >

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

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

[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

  1   2   >