Re: [PATCH][PR66010] Don't take address of ap unless necessary

2015-05-13 Thread Richard Biener
On Wed, 13 May 2015, Andreas Krebbel wrote: On 05/12/2015 01:45 PM, Tom de Vries wrote: On 12-05-15 12:04, Tom de Vries wrote: Committed with comments below added. Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c: ... gimplification failed: ap addr_expr

Re: [patch 26/29] libstdc++-v3: Use automake-1.11.6 (across the tree)

2015-05-13 Thread Jonathan Wakely
On 06/05/15 12:21 +0200, Michael Haubenwallner wrote: Patch for libstdc++-v3. The libstd++ parts are OK. N.B. libstdc++ patches should be sent to the libstdc++ list (which saves you CCing individual maintainers.) (Will this change mean I need to have 1.11.1 and 1.11.6 both installed and

[patch] Update SJLJ buffer after dynamic stack allocation

2015-05-13 Thread Eric Botcazou
Hi, the SJLJ EH schemes manipulate a buffer that contains a few pointers and, in particular, the stack pointer. Therefore, when dynamic stack allocation is also used, you need to update its value in the buffer, otherwise Bad Things will happen if you resume execution where the stack is

Re: [PATCH] combine: Don't create (set (reg:CC) (compare (reg:CC) (const0)))

2015-05-13 Thread Segher Boessenkool
On Mon, May 11, 2015 at 06:13:40AM -0700, Segher Boessenkool wrote: * combine.c (simplify_set): When generating a CC set, if the source already is in the correct mode, do not wrap it in a compare. Simplify the rest of that code. Committed now. Segher

Re: [PATCH] Support vectorizing SLP with mixed plus/minus

2015-05-13 Thread Andreas Schwab
Richard Biener rguent...@suse.de writes: PR tree-optimization/37021 * tree-vectorizer.h (struct _slp_tree): Add two_operators flag. (SLP_TREE_TWO_OPERATORS): New define. * tree-vect-slp.c (vect_create_new_slp_node): Initialize SLP_TREE_TWO_OPERATORS.

Re: [PATCH] Fix va_arg gimplification error for s390

2015-05-13 Thread Richard Biener
On Wed, May 13, 2015 at 9:38 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch fixes a gimplification error of the va_list argument of va_arg for target s390. The error was introduced by r223054, the fix for PR66010. I. consider test-case: ... #include stdarg.h int f1

Re: My patch for GCC 5 directory names

2015-05-13 Thread Matthias Klose
On 05/12/2015 03:42 PM, Richard Biener wrote: I promised to send out my pat^Whack. Before building I introduce gcc/FULL-VER as copy of gcc/BASE-VER and adjust gcc/BASE-VER to just the major number. Then I only need the following small patch (where I don't speak enough tcl for fixing

[PATCH v2] libiberty: cleanup Makefile.in

2015-05-13 Thread Bernhard Reutner-Fischer
* configure.ac (TARGETLIB_PIC, TARGETLIB_NOASAN): New variables. * configure: Regenerate. * maint-tool: Refactor pic/ and noasan/ handling. * Makefile.in: Likewise. Regenerate dependencies. --- The below does the same but attempts to be limited to what POSIX

Re: My patch for GCC 5 directory names

2015-05-13 Thread Richard Biener
On Tue, 12 May 2015, H.J. Lu wrote: On Tue, May 12, 2015 at 9:32 AM, Richard Biener rguent...@suse.de wrote: On May 12, 2015 6:11:45 PM GMT+02:00, H.J. Lu hjl.to...@gmail.com wrote: On Tue, May 12, 2015 at 9:09 AM, Richard Biener richard.guent...@gmail.com wrote: On May 12, 2015 5:58:07 PM

Re: [PATCH, ARM] attribute target (thumb,arm) [2.1/6] respin (5th)

2015-05-13 Thread Kyrill Tkachov
On 13/05/15 09:49, Christian Bruel wrote: 2 parts for maintainers - c-family: machine descriptions export macro definitions into c implementation : need to export 'builtin_define_with_int_value' and ' builtin_define_type_sizeof' +1 on this. It would help me a lot in implementing target

[PATCH] Fix PR66123

2015-05-13 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-05-13 Richard Biener rguent...@suse.de PR tree-optimization/66123 * tree-ssa-dom.c (propagate_rhs_into_lhs): Check if we found a taken edge. * gcc.dg/torture/pr66123.c: New testcase.

[committed] [patch 0/27] Use automake-1.11.6 across the tree

2015-05-13 Thread Michael Haubenwallner
On 05/05/2015 06:03 PM, Michael Haubenwallner wrote: Hello build machinery maintainers, following up http://thread.gmane.org/gmane.comp.gcc.patches/331902/focus=334462 http://thread.gmane.org/gmane.comp.gcc.patches/332160 On 01/25/2015 08:42 PM, Jan-Benedict Glaw wrote: On Sun,

[PATCH 3/7] [D] libiberty: Include type modifiers in demangled function symbols

2015-05-13 Thread Iain Buclaw
Like C++ const and volatile, in D mangled symbols can exist modifiers that represent the const, immutable, inout and shared-ness of the 'this' parameter. This information should be written out in the demangled symbol to show that each variant has a unique identity. --- libiberty/ChangeLog:

[PATCH 2/7] [D] libiberty: Fail if reached end of symbol string

2015-05-13 Thread Iain Buclaw
If a symbol that has so far been valid abruptly ends then we will want to fail the process rather than silently succeed. --- libiberty/ChangeLog 2015-05-13 Iain Buclaw ibuc...@gdcproject.org * d-demangle.c (dlang_call_convention): Return NULL if have reached the end of the symbol, but

[PATCH 1/7] [D] libiberty: Correctly decode white or non-printable characters

2015-05-13 Thread Iain Buclaw
Hi, Started these as separate patches, but as more came out of what I was originally trying to achieve (see Patch 6/7), I thought it better to have it as a running series. These set out to update d-demangle.c for new ABI additions, general bug fixes, and improved template support. --- D

Re: [patch] Implement ISO/IEC TS 18822 C++ File system TS

2015-05-13 Thread Jonathan Wakely
On 08/05/15 13:43 +0200, Rainer Orth wrote: Jonathan Wakely jwak...@redhat.com writes: I've committed the two changes attached (only tested on linux again). patch2.txt should fix the mingw-w64 errors above, as well as the issues Daniel reported, and should fix the error on Solaris 10 Rainer,

Re: [PATCH] Fix PR66110

2015-05-13 Thread Richard Biener
On Tue, 12 May 2015, Richard Biener wrote: The following patch fixes PR66110 - there is no reason why two alias-sets conflict just because one or the other has a child with alias-set zero. We need to paper over strict-aliasing bugs in dfp.c (re-using struct real_value unsigned long sig[]

Re: [patch] Clean up detection of SJLJ exceptions in target libraries

2015-05-13 Thread Jonathan Wakely
On 12/05/15 18:42 +0200, Eric Botcazou wrote: libstdc++-v3/ * acinclude.m4 (GLIBCXX_ENABLE_SJLJ_EXCEPTIONS): Delete. * configure.ac: Remove GLIBCXX_ENABLE_SJLJ_EXCEPTIONS. * config.h.in: Regenerate. * configure: Likewise. * libsupc++/eh_personality.cc:

[PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-13 Thread Iain Buclaw
In my tests, this gives the demangler near-complete support. Of a sample of about 75k symbols pulled from the standard library unittester, all but 20 were successfully parsed. --- libiberty/ChangeLog: 2015-05-13 Iain Buclaw ibuc...@gdcproject.org * d-demangle.c (dlang_symbol_kinds): New

[PATCH 5/7] [D] libiberty: Add support for return parameter and attributes

2015-05-13 Thread Iain Buclaw
The next version of D introduces two new mangle conventions. - Nj to represent methods whose 'this' parameter is also the function return value - Nk to represent a 'ref' parameter that is also the function return value This patch introduces support for these two symbols. ---

[PATCH 4/7] [D] libiberty: Check symbol length before using strncmp

2015-05-13 Thread Iain Buclaw
This addresses a subtle logic error, noticed when I was in the middle of testing out some other tightening up of parsing checks. --- libiberty/ChangeLog 2015-05-13 Iain Buclaw ibuc...@gdcproject.org * d-demangle.c (dlang_identifier): Check encoded length of identifier to verify strncmp

Re: [Patch, fortran, pr65548, 2nd take, v3] [5/6 Regression] gfc_conv_procedure_call

2015-05-13 Thread Andre Vehreschild
Hi Mikael, snip --- 5198,5222 /* In all other cases evaluate the expr3 and create a temporary. */ gfc_init_se (se, NULL); + /* For more complicated expression, the decision when to get the +descriptor and when to get a reference is depending

[PATCH 7/7] [D] libiberty: Add support for cent and ucent types

2015-05-13 Thread Iain Buclaw
The next version of D adds support for cent and ucent for platforms that are able to handle them natively. This adds support for demangling them. --- libiberty/ChangeLog: 2015-05-13 Iain Buclaw ibuc...@gdcproject.org * d-demangle.c (dlang_type): Handle cent and ucent types. *

Re: [PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-13 Thread Richard Biener
On Fri, May 8, 2015 at 5:09 PM, Kyrill Tkachov kyrylo.tkac...@foss.arm.com wrote: On 08/05/15 14:56, Kyrill Tkachov wrote: On 08/05/15 11:18, Richard Biener wrote: On Fri, May 1, 2015 at 6:02 PM, Kyrill Tkachov kyrylo.tkac...@foss.arm.com wrote: Hi all, GCC has some logic to expand

Re: [PATCH PR65447]Improve IV handling by grouping address type uses with same base and step

2015-05-13 Thread Richard Biener
On Fri, May 8, 2015 at 12:47 PM, Bin Cheng bin.ch...@arm.com wrote: Hi, GCC's IVO currently handles every IV use independently, which is not right by learning from cases reported in PR65447. The rationale is: 1) Lots of address type IVs refer to the same memory object, share similar base

Re: Work around PR65873

2015-05-13 Thread Richard Biener
On Wed, May 13, 2015 at 12:12 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch works around PR where we refuse to inline always_inline memcpy into function with explicit Ofast optimization attribute. This is because we think we can not promote -fno-fast-math code to -fast-math code.

Re: [patch] Clean up detection of SJLJ exceptions in target libraries

2015-05-13 Thread Matt Breedlove
This patch fixes an issue preventing mingw-w64 i686 dwarf2-eh bootstrapping described at: http://sourceforge.net/p/mingw-w64/mailman/message/34101954/ I'm assuming this has more to do with switching away from the current sjlj configuration method since configuring gcc with

[patch] Add std::raw_storage_iterator::base() member

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2454 Voted into the WP in Lenexa. Tested powerpc64le-linux, committed to trunk. commit 8cb30e1ad50f1bb277c635d83796c64772b61d68 Author: redi redi@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Wed May 13 13:54:46 2015 +

[patch] LWG 2418 make std:experimental::apply() work with pointers to member

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2418 This was voted into the TS in Lenexa. Tested powerpc64le-linux, committed to trunk. commit ef712e821bc683fe3979251dcc09786bee0233e0 Author: Jonathan Wakely jwak...@redhat.com Date: Wed May 13 13:37:05 2015 +0100 *

[PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Marek Polacek
As discussed in the PR, match.pd happily folds 0 * whatever into 0. That is undesirable from the C/C++ FE POV, since it can make us accept invalid initializers. So fixed in match.pd -- I'd hope there's a better way to do this, but this seems to work. There was some fallout, but nothing

[C PATCH] Fold C_MAYBE_CONST_EXPRs with C_MAYBE_CONST_EXPR_INT_OPERANDS set (PR c/66066)

2015-05-13 Thread Marek Polacek
The following is an attempt to implement what Joseph outlined here: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01015.html. This patch relies on the match.pd I've just posted. It shouldn't reduce e.g. division by zero to pedwarn-if-pedantic (the tests test that). But e.g. enum { A = 1 -5 };

[patch] Make std::string default constructor conditionally noexcept

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2455 Voted into the WP in Lenexa. We already did the right thing for vector, so only basic_string needs to change. Tested powerpc64le-linux, committed to trunk. commit 634ed6e2d2ea4d69a29a8907044e6f68541d88aa Author: Jonathan

[patch] std::packaged_task(allocator_arg_t, const A, F) should not be explicit

2015-05-13 Thread Jonathan Wakely
http://wiki.edg.com/twiki/pub/Wg21lenexa/StrawPolls/N4525.html#2407 Voted into the WP in Lenexa. Tested powerpc64le-linux, committed to trunk. commit f58bb7ab1b098d525359869767286f9c8955acd3 Author: Jonathan Wakely jwak...@redhat.com Date: Wed May 13 14:39:24 2015 +0100 *

Re: [patch 26/29] libstdc++-v3: Use automake-1.11.6 (across the tree)

2015-05-13 Thread Michael Haubenwallner
On 05/13/2015 12:27 PM, Jonathan Wakely wrote: On 06/05/15 12:21 +0200, Michael Haubenwallner wrote: Patch for libstdc++-v3. The libstd++ parts are OK. N.B. libstdc++ patches should be sent to the libstdc++ list (which saves you CCing individual maintainers.) Indeed, thanks! (Will

[PATCH] Fix PR66129

2015-05-13 Thread Richard Biener
The following fixes PR66129. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2015-05-13 Richard Biener rguent...@suse.de PR tree-optimization/66129 * tree-vect-slp.c (vect_build_slp_tree): Make sure all ops are commutative.

[v3 patch] Implement LWG 2466 (allocator_traits::max_size() is stoopid)

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/lwg-active.html#2466 We agreed to move this to Ready in Lenexa. I'd like to see max_size() deprecated, it's useless, but if we can't do that it might as well not give an impossible answer. Tested powerpc64le-linux, committed to trunk. commit

[patch] std::polar requires non-negative rho

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2459 Voted into the WP in Lenexa. Tested powerpc64le-linux, comitted to trunk. commit 9bf3b9ea20334711ecdced656323f69959521a82 Author: Jonathan Wakely jwak...@redhat.com Date: Wed May 13 14:18:03 2015 +0100 *

Re: [patch] std::polar requires non-negative rho

2015-05-13 Thread Daniel Krügler
2015-05-13 15:32 GMT+02:00 Jonathan Wakely jwak...@redhat.com: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2459 Voted into the WP in Lenexa. Tested powerpc64le-linux, comitted to trunk. What about assertions regarding non-NAN rho and finite theta, as decided for by the

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Jakub Jelinek
On Wed, May 13, 2015 at 03:41:11PM +0200, Marek Polacek wrote: As discussed in the PR, match.pd happily folds 0 * whatever into 0. That is undesirable from the C/C++ FE POV, since it can make us accept invalid initializers. So fixed in match.pd -- I'd hope there's a better way to do this,

[patch] LWG 2440 std::seed_seq::size() should be noexcept

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2440 This was resolved as a defect and voted into the working paper in Lenexa. As an extension we can also make the default constructor noexcept. Tested powerpc64le-linux, committed to trunk. commit

Re: [patch] std::polar requires non-negative rho

2015-05-13 Thread Marc Glisse
On Wed, 13 May 2015, Daniel Krügler wrote: 2015-05-13 15:32 GMT+02:00 Jonathan Wakely jwak...@redhat.com: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2459 Voted into the WP in Lenexa. Tested powerpc64le-linux, comitted to trunk. What about assertions regarding

RFA: RL78: Place zero-initialised data into the .bss section

2015-05-13 Thread Nick Clifton
Hi DJ, Currently the RL78 port does not place zero-initialised data in the .bss section. This is because of the rl78_select_section() function which does not handle bss data. The patch below updates the function to handle that and other types of data so that they end up in the

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Marek Polacek
On Wed, May 13, 2015 at 04:11:15PM +0200, Marek Polacek wrote: I don't know how to reliably fix this :(. Except disabling (0 * X) - 0 completely, that is. Marek

Re: [PATCH 3/4] libcpp/input.c: Add a way to visualize the linemaps

2015-05-13 Thread David Malcolm
On Mon, 2015-05-04 at 13:19 -0600, Jeff Law wrote: On 05/01/2015 06:56 PM, David Malcolm wrote: As a relative newcomer to GCC, one of the issues I had was becoming comfortable with the linemap API and its internal representation. To familiarize myself with it, I wrote a dumping routine

[PATCH, alpha]: Some further HWI == 64 improvements

2015-05-13 Thread Uros Bizjak
Hello! 2015-05-13 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_emit_set_long_const): Remove c1 argument. (alpha_extract_integer): Redeclare as static HOST_WIDE_INT. Remove *p0 and *p1 arguments. Rewrite function. (alpha_legitimate_constant_p): Update call to

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Marek Polacek
On Wed, May 13, 2015 at 03:55:10PM +0200, Jakub Jelinek wrote: On Wed, May 13, 2015 at 03:41:11PM +0200, Marek Polacek wrote: As discussed in the PR, match.pd happily folds 0 * whatever into 0. That is undesirable from the C/C++ FE POV, since it can make us accept invalid initializers.

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Joseph Myers
On Wed, 13 May 2015, Marek Polacek wrote: On Wed, May 13, 2015 at 03:55:10PM +0200, Jakub Jelinek wrote: On Wed, May 13, 2015 at 03:41:11PM +0200, Marek Polacek wrote: As discussed in the PR, match.pd happily folds 0 * whatever into 0. That is undesirable from the C/C++ FE POV, since it

Re: [patch] std::polar requires non-negative rho

2015-05-13 Thread Jonathan Wakely
On 13/05/15 15:36 +0200, Daniel Krügler wrote: 2015-05-13 15:32 GMT+02:00 Jonathan Wakely jwak...@redhat.com: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2459 Voted into the WP in Lenexa. Tested powerpc64le-linux, comitted to trunk. What about assertions regarding

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Richard Biener
On Wed, 13 May 2015, Marek Polacek wrote: On Wed, May 13, 2015 at 03:55:10PM +0200, Jakub Jelinek wrote: On Wed, May 13, 2015 at 03:41:11PM +0200, Marek Polacek wrote: As discussed in the PR, match.pd happily folds 0 * whatever into 0. That is undesirable from the C/C++ FE POV, since it

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Richard Biener
On Wed, 13 May 2015, Marek Polacek wrote: On Wed, May 13, 2015 at 04:11:15PM +0200, Marek Polacek wrote: I don't know how to reliably fix this :(. Except disabling (0 * X) - 0 completely, that is. Well, make sure that X has TREE_SIDE_EFFECTS set. That way we'd fold to 0, X. Richard. --

Re: Work around PR65873

2015-05-13 Thread Jan Hubicka
On Wed, May 13, 2015 at 12:12 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch works around PR where we refuse to inline always_inline memcpy into function with explicit Ofast optimization attribute. This is because we think we can not promote -fno-fast-math code to -fast-math

[PATCH] Avoid unnecessary work when -Wmisleading-indentation isn't enabled

2015-05-13 Thread David Malcolm
This might qualify as obvious. OK for trunk? (if it passes bootstrap and regrtest) gcc/c-family/ChangeLog: * c-indentation.c (warn_for_misleading_indentation): Bail out immediately if -Wmisleading-indentation isn't enabled. --- gcc/c-family/c-indentation.c | 6 ++ 1 file

[PATCH 8/7] [D] libiberty: Add support for specialized template parameters

2015-05-13 Thread Iain Buclaw
One last thing that was added in the next D version's ABI (and I subsequently missed). This is a trivial patch to just ignore the new mangle symbol. --- libiberty/ChangeLog: 2015-05-13 Iain Buclaw ibuc...@gdcproject.org * d-demangle.c (dlang_template_args): Skip over specialized template

Re: [PATCH i386] Extend sibcall peepholes to allow source in %eax

2015-05-13 Thread Alexander Monakov
On Mon, 11 May 2015, Jan Hubicka wrote: Yes, to make my original email clear, I think we are safe to remove peep2_reg_dead_p. I would however introduce a check that the call target is not also among parameters of the function. In this case the peephole would remove the load and make the

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Marek Polacek
On Wed, May 13, 2015 at 03:06:00PM +, Joseph Myers wrote: It won't handle e.g. 0 * (unsigned) (1 / 0). I think this illustrates why handling this in the folding-for-optimization is a mistake. For optimization, it's perfectly fine to fold away 0 * (something without side effects),

[gomp4] Prohibit C++ reference types in OpenACC regions

2015-05-13 Thread Thomas Schwinge
Hi! On Wed, 29 Apr 2015 09:27:49 -0700, Cesar Philippidis cesar_philippi...@mentor.com wrote: This patch teaches the c++ front end to error when reference typed variables are used in openacc parallel and kernels regions. The c++ front end changes are pretty straightforward. I did, however,

[gomp4] Basic -misa support for nvptx (was: How to use old GPU (Fermi) in gcc with OpenACC?)

2015-05-13 Thread Thomas Schwinge
Hi! On Sat, 9 May 2015 10:26:22 -0700, Satoshi_OHSHIMA satoshiohsh...@gmail.com wrote: I'm trying to use and evaluate gcc with OpenACC on some NVIDIA GPUs. I succeeded to build gcc with OpenACC by using http://scelementary.com/2015/04/25/openacc-in-gcc.html as a reference. Heh, their build

Re: [PATCH, alpha]: Remove some_operand and some_ni_operand

2015-05-13 Thread Richard Henderson
On 05/13/2015 11:11 AM, Uros Bizjak wrote: We can use general_operand instead of some_operand. 2015-05-13 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.md (extendqidi2): Use general_operand instead of some_operand for operand[1] predicate. (extendhidi2): Ditto.

[gomp4] nvptx libgcc atomic routines

2015-05-13 Thread Thomas Schwinge
Hi! On Fri, 10 Apr 2015 09:20:30 -0700, Cesar Philippidis cesar_philippi...@mentor.com wrote: This patch implements the various atomic __sync_* functions inside libgcc. The original [motivation], were the calls to __sync_val_compare_and_swap_1 by openacc kernels involving boolean

Re: [PATCH 3/7] [D] libiberty: Include type modifiers in demangled function symbols

2015-05-13 Thread Iain Buclaw
On 13 May 2015 at 22:34, Iain Buclaw ibuc...@gdcproject.org wrote: On 13 May 2015 at 10:51, Iain Buclaw ibuc...@gdcproject.org wrote: Like C++ const and volatile, in D mangled symbols can exist modifiers that represent the const, immutable, inout and shared-ness of the 'this' parameter. This

[gomp4] nvptx offloading linking (was: [WIP] OpenMP 4 NVPTX support)

2015-05-13 Thread Thomas Schwinge
Hi! On Wed, 22 Apr 2015 17:08:26 +0200, Bernd Schmidt ber...@codesourcery.com wrote: On 04/21/2015 05:58 PM, Jakub Jelinek wrote: suggests that while it is nice that when building nvptx accel compiler we build libgcc.a, libc.a, libm.a, libgfortran.a (and in the future hopefully

[gomp4] Assorted OpenACC changes (was: Next set of OpenACC changes)

2015-05-13 Thread Thomas Schwinge
Hi! On Mon, 11 May 2015 18:35:12 +0200, I wrote: On Tue, 05 May 2015 10:54:02 +0200, I wrote: In follow-up messages, I'll be posting the separated parts (for easier review) of a next set of OpenACC changes that we'd like to commit. ChangeLog updates not yet written; will do that before

Re: [PATCH 3/7] [D] libiberty: Include type modifiers in demangled function symbols

2015-05-13 Thread Iain Buclaw
On 13 May 2015 at 10:51, Iain Buclaw ibuc...@gdcproject.org wrote: Like C++ const and volatile, in D mangled symbols can exist modifiers that represent the const, immutable, inout and shared-ness of the 'this' parameter. This information should be written out in the demangled symbol to show

Re: Work around PR65873

2015-05-13 Thread Sandra Loosemore
On 05/12/2015 04:12 PM, Jan Hubicka wrote: Hi, this patch works around PR where we refuse to inline always_inline memcpy into function with explicit Ofast optimization attribute. This is because we think we can not promote -fno-fast-math code to -fast-math code. This is of course safe for

[gomp4] libgomp: Cope with DejaGnu having no mechanism to transfer environment variables to remote boards

2015-05-13 Thread Thomas Schwinge
Hi! On Fri, 24 Oct 2014 00:18:10 +0200, I wrote: On Thu, 23 Oct 2014 17:02:20 +0200, I wrote: On Fri, 16 May 2014 20:01:47 +0200, I wrote: On Fri, 16 May 2014 19:02:41 +0200, Tom de Vries tom_devr...@mentor.com wrote: Essentially I want to run the set of testcases in libgomp.oacc-c,

Re: [PATCH] Don't fold away division by zero (PR middle-end/66127)

2015-05-13 Thread Joseph Myers
On Wed, 13 May 2015, Marek Polacek wrote: Rather, how about having an extra argument to c_fully_fold_internal (e.g. for_int_const) indicating that the folding is of an expression with integer constant operands (so this argument would be true in the new case of folding the contents of a

Re: [gomp4] Basic -misa support for nvptx (was: How to use old GPU (Fermi) in gcc with OpenACC?)

2015-05-13 Thread Joseph Myers
On Wed, 13 May 2015, Thomas Schwinge wrote: * config/nvptx/nvptx.opt (ptx_isa, sm_30, sm_35): New enum and its values. (misa=): New option. New options do of course need documenting in invoke.texi. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Avoid unnecessary work when -Wmisleading-indentation isn't enabled

2015-05-13 Thread Marek Polacek
On Wed, May 13, 2015 at 11:25:56AM -0400, David Malcolm wrote: This might qualify as obvious. OK for trunk? (if it passes bootstrap and regrtest) Ok. Marek

Re: [PATCH][tree-ssa-math-opts] Expand pow (x, CONST) using square roots when possible

2015-05-13 Thread Kyrill Tkachov
Hi Richard, On 13/05/15 12:27, Richard Biener wrote: I notice that we don't have a testuite check that the target has a hw sqrt instructions. Would you like me to add one? Or can I make the testcase aarch64-specific? Would be great to have a testsuite check for this. I've committed the

Re: [PATCH, ARM, doc] add missing -mtune options

2015-05-13 Thread Ramana Radhakrishnan
On 07/05/15 23:12, Jim Wilson wrote: I noticed that the list of -mtune options in the arm-cores.def file didn't match the list in the doc/invoke.texi file. There are 3 cores missing: generic-armv7-a, cortex-a17, and cortex-a17.cortex-a7. This patch adds the missing cores to the docs. Jim

Re: [patch] Clean up detection of SJLJ exceptions in target libraries

2015-05-13 Thread Ian Lance Taylor
On Tue, May 12, 2015 at 9:42 AM, Eric Botcazou ebotca...@adacore.com wrote: 6 target libraries in the tree detect whether they are being compiled by a compiler configured for setjmp/longjmp exceptions: libada, libgcc, libgo, libjava, libobjc and libstdc++. They can be divided into 3

Re: [patch] Clean up detection of SJLJ exceptions in target libraries

2015-05-13 Thread Eric Botcazou
This patch fixes an issue preventing mingw-w64 i686 dwarf2-eh bootstrapping described at: http://sourceforge.net/p/mingw-w64/mailman/message/34101954/ I'm assuming this has more to do with switching away from the current sjlj configuration method since configuring gcc with

Re: Fwd: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-05-13 Thread Ramana Radhakrishnan
On 12/05/15 14:01, Martin Galvan wrote: On Tue, May 12, 2015 at 5:49 AM, Ramana Radhakrishnan ramana.radhakrish...@arm.com wrote: That's what I mean when I say email clients munged it : email clients and / or some popular email servers appear to end up munging white spaces and patches don't

Re: [ARM][PR65768] Keep constants in register when expanding

2015-05-13 Thread Kugan
Ping ? Thanks, Kugan On 27/04/15 09:07, Kugan wrote: While I support your goals, I think your approach needs some refinement. In particular, we DO NOT want another function that starts looking at constant values and tries to decide, on a case by case basis, what to do with that value. We

RE: [PATCH, FT32] initial support

2015-05-13 Thread James Bowman
On Tue, May 12, 2015 at 10:17:01PM +, James Bowman wrote: It seems that with whenever a function's frame is bigger than 512 bytes, LRA loops. Likely this causes a problem because there is no actual instruction for subtracting constants more than 512. However, there is a link pattern

[patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-13 Thread Jerry DeLisle
The attached patch reverts a change I made for pr65456 which caused this regression and adds a check for quotes farther down in the function. This avoids treating a '!' in a string as a comment and wiping the rest of the line. I found the added code is needed because an interposing quote was

Re: [PR65768] Check rtx_cost when propagating constant

2015-05-13 Thread Kugan
ping? Thanks, Kugan On 15/04/15 17:53, Kugan wrote: As mentioned in PR65768, ARM gcc generates suboptimal code for constant Uses in loop. Part of the reason is cprop is undoing what loop invariant code motion did. Zhenqiang posted a patch at to fix this based on rtx costs:

[PATCH] [gcc, combine] Backport to GCC 5.0 branch PR46164: Don't combine the insns if a volatile register is contained.

2015-05-13 Thread Hale Wang
-Original Message- From: Terry Guo [mailto:flame...@gmail.com] Sent: Wednesday, April 22, 2015 3:24 PM To: Segher Boessenkool Cc: l...@redhat.com; Hale Wang; GCC Patches; Richard Sandiford Subject: Re: Ping^3 : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile

[patch] Constructing std::shared_ptr from an empty std::unique_ptr

2015-05-13 Thread Jonathan Wakely
http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2015/n4383.html#2415 Voted in to the WP in Lenexa. Tested powerpc64le-linux, committed to trunk. commit 87937e036ae87944f7ac2b26d8ae17fe81525c4f Author: Jonathan Wakely jwak...@redhat.com Date: Wed May 13 15:54:45 2015 +0100 *

[committed] Fix #pragma omp task expansion with noreturn task bodies (PR middle-end/66133)

2015-05-13 Thread Jakub Jelinek
Hi! During taskloop development, I have noticed a bug in the GIMPLE_OMP_TASK ompexp - unlike GIMPLE_OMP_PARALLEL, where if the parallel body is noreturn, the GOMP_parallel call never returns either, for tasks it really depends on whether the task is included or not. If the library (the usual

[PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-13 Thread Segher Boessenkool
This patch changes rs6000_rtx_costs to be closer to reality. It is still not quite right, but at least it handles more cases, so the (set (reg:SI) (eq:SI (reg:SI) (reg:SI))) from the PR isn't taken as costing 1 insn anymore, while setting a reg:DI was cost 3 insns. The mode of the dest

Re: [PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-13 Thread Segher Boessenkool
On Wed, May 13, 2015 at 01:25:16PM -0400, David Edelsohn wrote: PR rtl-optimization/30967 * config/rs6000/rs6000.c (rs6000_rtx_costs): Don't consider destination mode for the cost of scc patterns. Okay. Should this be applied to GCC 5 branch? [ Testing is still

Re: [PATCH, alpha]: Some further HWI == 64 improvements

2015-05-13 Thread Richard Henderson
On 05/13/2015 08:04 AM, Uros Bizjak wrote: Hello! 2015-05-13 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_emit_set_long_const): Remove c1 argument. (alpha_extract_integer): Redeclare as static HOST_WIDE_INT. Remove *p0 and *p1 arguments. Rewrite function.

Re: Fwd: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-05-13 Thread Ramana Radhakrishnan
On 13/05/15 17:37, Ramana Radhakrishnan wrote: On 12/05/15 14:01, Martin Galvan wrote: On Tue, May 12, 2015 at 5:49 AM, Ramana Radhakrishnan ramana.radhakrish...@arm.com wrote: That's what I mean when I say email clients munged it : email clients and / or some popular email servers appear

Re: [patch] std::polar requires non-negative rho

2015-05-13 Thread Daniel Krügler
2015-05-13 16:05 GMT+02:00 Marc Glisse marc.gli...@inria.fr: On Wed, 13 May 2015, Daniel Krügler wrote: What about assertions regarding non-NAN rho and finite theta, as decided for by the LWG 2439? non-NAN rho is already covered by rho = 0. Agreed on that. - Daniel

Re: Fwd: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-05-13 Thread Martin Galvan
On Wed, May 13, 2015 at 1:58 PM, Ramana Radhakrishnan ramana.radhakrish...@arm.com wrote: On 13/05/15 17:37, Ramana Radhakrishnan wrote: I'm not sure what's going on here and couldn't figure out what was going wrong in the 20 minutes I spent on it just now. Bah, I must be going blind. @@

Re: [C frontend] Fix construction of TYPE_STUB_DECL

2015-05-13 Thread Jan Hubicka
Hello, it seems that the discussion here got stuck without arriving to a consensus. I generally see three options here 1) make TYPE_PUBLIC flag of TYPE_STUB_DECL to work consistently across frontends in a sense that types with flag !TYPE_PUBLIC (TYPE_STUB_DECL (t)) can be considered

Re: Fwd: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-05-13 Thread Marek Polacek
On Wed, May 13, 2015 at 02:07:34PM -0300, Martin Galvan wrote: That makes sense. I downloaded the gcc sources from the git mirror at github-- perhaps that's what's munging the patch? That mirror seems to be out-of-date. I suggest using https://gcc.gnu.org/git/?p=gcc.git;a=summary.

Re: [PATCH] rs6000: Improve rtx_costs for EQ a bit (PR30967)

2015-05-13 Thread David Edelsohn
On Wed, May 13, 2015 at 1:15 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: This patch changes rs6000_rtx_costs to be closer to reality. It is still not quite right, but at least it handles more cases, so the (set (reg:SI) (eq:SI (reg:SI) (reg:SI))) from the PR isn't taken

Re: [PATCH, alpha]: Some further HWI == 64 improvements

2015-05-13 Thread Uros Bizjak
On Wed, May 13, 2015 at 6:46 PM, Richard Henderson r...@redhat.com wrote: On 05/13/2015 08:04 AM, Uros Bizjak wrote: Hello! 2015-05-13 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_emit_set_long_const): Remove c1 argument. (alpha_extract_integer): Redeclare as static

[PATCH, GOOGLE] Backport patch r212222 to google 4.9 branch

2015-05-13 Thread Carrot Wei
Hi The more strict devirtualization condition in this patch helps to fix google bug b/19872411. Bootstraped and regression tested on x86-64. OK for google 4.9 branch? patch Description: Binary data

Re: RFA: RL78: Place zero-initialised data into the .bss section

2015-05-13 Thread DJ Delorie
OK to apply ? Ok but.. +case SECCAT_TBSS: + return default_select_section (decl, reloc, align); + +default: + gcc_unreachable (); Would it be better to just default: everything to default_select_section, instead of enumerating everything we know about today?

[PATCH, i386]: Macroize mul{,v} patterns some more

2015-05-13 Thread Uros Bizjak
Hello! Also fixes a couple of operand constraints. 2015-05-13 Uros Bizjak ubiz...@gmail.com * config/i386/i386.md (*mulmode3_1): Merge with *mulhi3_1 using SWIM248 mode iterator. (*mulvmode4): Use x86_64_sext_operand for operand[2] constraint. (*mulvhi4): mark operand[1] as

Re: [PATCH, GOOGLE] Backport patch r212222 to google 4.9 branch

2015-05-13 Thread Xinliang David Li
ok. David On Wed, May 13, 2015 at 11:00 AM, Carrot Wei car...@google.com wrote: Hi The more strict devirtualization condition in this patch helps to fix google bug b/19872411. Bootstraped and regression tested on x86-64. OK for google 4.9 branch?

Re: Fwd: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-05-13 Thread Martin Galvan
Here's the new patch. I downloaded the gcc sources from the SVN repository, removed the extra semicolon from my version of the files and re-generated the patch using svn diff, making sure the context info had all the tabs from the original. I then e-mailed the patch to myself as an attachment,

Re: Fwd: [PING 2][PATCH] libgcc: Add CFI directives to the soft floating point support code for ARM

2015-05-13 Thread Martin Galvan
Forgot to mention: I first did a --dry-run of the patch and then applied it to the gcc sources. On Wed, May 13, 2015 at 3:11 PM, Martin Galvan martin.gal...@tallertechnologies.com wrote: Here's the new patch. I downloaded the gcc sources from the SVN repository, removed the extra semicolon from

Re: [patch] Clean up detection of SJLJ exceptions in target libraries

2015-05-13 Thread Eric Botcazou
The libgo parts are fine, but since libgo is mirrored from an external repository I'll commit those parts myself. Thanks! I assume I can go ahead and commit them now? Yes, you can, the libgo bits are independent. -- Eric Botcazou

[PATCH, alpha]: Remove some_operand and some_ni_operand

2015-05-13 Thread Uros Bizjak
We can use general_operand instead of some_operand. 2015-05-13 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.md (extendqidi2): Use general_operand instead of some_operand for operand[1] predicate. (extendhidi2): Ditto. (cbranchdi4): Use general_operand instead of

Re: [PATCH][MIPS] Enable load-load/store-store bonding

2015-05-13 Thread sameera
Hi Mike, Thanks for your comments. Please find my comments inlined. - Thanks and regards, Sameera D. On Monday 11 May 2015 10:09 PM, Mike Stump wrote: On May 11, 2015, at 4:05 AM, sameera sameera.deshpa...@imgtec.com wrote: +(define_insn *join2_loadhi + [(set (match_operand:SI 0

[PATCH] Fix va_arg gimplification error for s390

2015-05-13 Thread Tom de Vries
Hi, this patch fixes a gimplification error of the va_list argument of va_arg for target s390. The error was introduced by r223054, the fix for PR66010. I. consider test-case: ... #include stdarg.h int f1 (int i, ...) { int res; va_list ap; va_start (ap, i); res = va_arg (ap,

  1   2   >