RE: [PATCH] MIPS: Prevent the p5600-bonding.c test from being run for the n32 and 64 ABIs

> This is OK now. Committed as SVN 232980. Regards, Andrew

[PATCH] Fix wide_int unsigned division (PR tree-optimization/69546)

Hi! As the testcase shows, wide_int unsigned division is broken for > 64bit precision division of unsigned dividend which have 63rd bit set, and all higher bits cleared (thus is normalized as 2 HWIs, first with MSB set, the second 0) and divisor of 1, we return just a single HWI, which is

[PATCH][AArch64] PR target/69161: Don't use special predicate for CCmode comparisons in expressions that require matching modes

Hi all, In this PR we ICE during combine when trying to propagate a comparison into a vec_duplicate, that is we end up creating the rtx: (vec_duplicate:V4SI (eq:CC_NZ (reg:CC_NZ 66 cc) (const_int 0 [0]))) The documentation for vec_duplicate says: "The output vector mode must have the

[committed] Fix SSE1 V4SImode vector insert (PR target/69551)

Hi! The following patch fixes a bug in the V4SImode ix86_expand_vector_set SSE1 handling, before we recurse, we need to copy the original target to the temporary, otherwise we set just the single element and leave the rest of the elements uninitialized. Bootstrapped/regtested on x86_64-linux and

Re: [PATCH] Fix PR69547

On Fri, Jan 29, 2016 at 09:40:48AM +0100, Richard Biener wrote: > I am testing the following patch to fix a regression that we no longer > remove some empty loops. Doesn't this mean that DCE will remove the clobbers as unnecessary, even when they aren't in empty loops? Jakub

Re: [PATCH] S/390: Require a hardware vector support for test to succeed.

On Wed, Jan 27, 2016 at 11:04:32AM +0100, Dominik Vogt wrote: > gcc/testsuite/ChangeLog > > * gcc.dg/tree-ssa/ssa-dom-cse-2.c: Require a hardware vector support for > test to succeed. Applied. Thanks! -Andreas-

Re: [PATCH] s390: Add -fsplit-stack support

Hi Marcin, sorry for the late feedback. A few comments regarding the split stack implementation: The GNU coding style requires to replace every 8 leading blanks on a line with a tab. There are many lines in your patch violating this. In case you are an emacs user `whitespace-cleanup' will fix

[committed] Add testcase for PR66137

Hi! This PR has been fixed by the PR68701, I've committed the testcase as obvious to trunk. 2016-01-29 Jakub Jelinek PR target/66137 * gcc.target/i386/pr66137.c: New test. --- gcc/testsuite/gcc.target/i386/pr66137.c.jj 2016-01-29 15:05:19.804958974 +0100

[hsa] Atomic assess memory model fixes

Hi, this is a followup to comments by Jakub and Richi on handling of memory models in HSA atomic operations: - I have made user-visible diagnostics lower case simple words, rather than constant identifiers. - I have added masking by MEMMODEL_BASE_MASK where appropriate. - I have made sure

Re: [off-list] Re: [PATCH PR68542]

Uros, Here is update patch which includes (1) couple changes proposed by Richard in tree-vect-loop.c and (2) the changes in back-end proposed by you. Is it OK for trunk? Bootstrap and regression testing dis not show any new failures. ChangeLog: 2016-01-29 Yuri Rumyantsev

[PATCH][ARM] PR target/69161: Don't ignore mode when matching comparison operator in cstore-like patterns

Hi all, Similar to aarch64, the arm port also suffers from PR target/69161 when combine tries to propagate a CCmode comparison into a vec_duplicate, creating invalid RTL that ICEs. Please refer to the PR and the aarch64 fix for more info. The fix for arm is very similar. We define a new

Re: [PATCH] Remove PTX link option

On Mon, 11 Jan 2016, Alexander Monakov wrote: > On Mon, 11 Jan 2016, Thomas Schwinge wrote: > > Alexander, would you please also submit a fix for that for nvptx-tools' > > nvptx-run.c? (Or want me to do that?) > > I can do that, along with another small change I used for -mgomp testing. I have

Re: Is it OK for rtx_addr_can_trap_p_1 to attempt to compute the frame layout? (was Re: [PATCH] Skip re-computing the mips frame info after reload completed)

On 29.01.2016 02:09, Bernd Schmidt wrote: > On 01/28/2016 12:36 AM, Eric Botcazou wrote: >>> [cc-ing Eric as RTL maintainer] >> >> Sorry for the delay, the message apparently bounced] >> >>> IMO the problem is that rtx_addr_can_trap_p_1 duplicates a large >>> bit of LRA/reload logic: >>> >>> [...]

Re: Default compute dimensions

On Thu, Jan 28, 2016 at 10:38:51AM -0500, Nathan Sidwell wrote: > This patch adds default compute dimension handling. Users rarely specify > compute dimensions, expecting the toolchain to DTRT. More savvy users would > like to specify global defaults. This patch permits both. Isn't it better

Re: [gomp4, PR68977, Committed] Don't gimplify in ssa mode if seen_error in oacc_xform_loop

On 01/29/16 02:48, Richard Biener wrote: I see. Is it possible to simply scrub the whole OACC region in this case instead? Do you mean jettison the body of the offloaded fn, or something else? I guess the former's doable. (Throwing away the fn entirely could result in unresolved symbol

Re: [PATCH] New flag for dumping information about constexpr function calls memoization (GCC 5.2.0)

2016-01-28 17:54 GMT-03:00 Joseph Myers : > Any patch adding a new option needs to add documentation for it to > invoke.texi (both substantive documentation, and inclusion in the summary > lists of options). > > -- > Joseph S. Myers > jos...@codesourcery.com Hi, Thanks

Re: [PING, PATCH] Reduce accuracy of bessel_6.f90.

On Mon, Jan 11, 2016 at 03:40:56PM +0100, Dominik Vogt wrote: > Another patch reducing the accuracy required in the bessel_6 test. Fixes the test case for S/390. Can this be committed? > gcc/testsuite/ChangeLog > > * gfortran.dg/bessel_6.f90: Reduce accuracy for S/390. > >From

Re: [PATCH] PR other/69006: S/390: Fix extra newlines after diagnostics.

On Wed, Jan 27, 2016 at 12:01:26PM +0100, Dominik Vogt wrote: > gcc/ChangeLog > > PR other/69006 > * config/s390/s390-c.c (s390_resolve_overloaded_builtin): Remove > trailing blank line from error message. Applied. Thanks! -Andreas-

Fix some i386 testcases for -frename-registers

This patch corrects some tests that can fail with -frename-registers. The problems typically are of the form "xmm[0-7]+", disallowing registers 8 and 9, and "xmm[0-9]". disallowing numbers higher than 9. Most the patch was automatically generated, but there were some other cases as well.

[Patch] Update GCC Internals: remove section Preserving virtual SSA form

Hi, The section "12.3.2 Preserving the virtual SSA form" in GCC Internals is outdated. The two functions it documents push_stmt_changes and pop_stmt_changes have been removed. The functionality have been replaced with update_stmt. update_stmt is documented elsewhere in internals. I therefore

Fix c/69522, memory management issue in c-parser

Let's say we have struct a { int x[1]; int y[1]; } x = { 0, { 0 } }; ^ When we reach the marked brace, we call into push_init_level, where we notice that we have implicit initializers (for x[]) lying around that we should deal with now that we've seen another open brace. The

Re: [C++ PATCH] Fix -Wunused-function (PR debug/66869)

On Fri, Jan 29, 2016 at 11:35:07AM +0100, Jakub Jelinek wrote: > I can try to stick there an assert whether for FUNCTION_DECL > (DECL_INITIAL (decl) == 0) == DECL_EXTERNAL (decl). Tried that, but cancelled that quickly, I see lots of cases where DECL_INITIAL is non-NULL, but DECL_EXTERNAL is set,

Re: [C++ PATCH] Fix -Wunused-function (PR debug/66869)

On 01/29/2016 11:35 AM, Jakub Jelinek wrote: On Thu, Jan 28, 2016 at 09:51:34PM -0500, Jason Merrill wrote: On 01/28/2016 03:15 PM, Jakub Jelinek wrote: + if (TREE_CODE (decl) == FUNCTION_DECL + && DECL_INITIAL (decl) == 0 + && DECL_EXTERNAL (decl) + &&

[patch] libstdc++/69506 Fix Cygwin bootstrap error due to TM symbols

Another target that doesn't have the necessary weak ref support for the TM-aware exception-handling. Bootrapped successfully by the reporter, committed to trunk. commit b7e2e38ab1d938ee19280eba11ed3643a140f86d Author: Jonathan Wakely Date: Fri Jan 29 10:38:45 2016 +

Re: Fix some i386 testcases for -frename-registers

Hello! > * gcc.target/i386/avx512bw-vptestmb-1.c: Correct [xyz]mm register > number scans. > * gcc.target/i386/avx512bw-vptestmw-1.c: Likewise. > * gcc.target/i386/avx512bw-vptestnmb-1.c: Likewise. > * gcc.target/i386/avx512bw-vptestnmw-1.c: Likewise. > *

Re: [PATCH] PR c++/69462: Provide FLT_EVAL_METHOD and DECIMAL_DIG in float.h.

On 28/01/16 15:42 +0100, Jakub Jelinek wrote: On Thu, Jan 28, 2016 at 01:32:18PM +, Jonathan Wakely wrote: On 28/01/16 13:40 +0100, Dominik Vogt wrote: >The attached patch (written by Jonathan, not me) makes >FLT_EVAL_METHOD and DECIMAL_DIG available in C++-11 as they should >be. >

Re: [C++ PATCH] Fix -Wunused-function (PR debug/66869)

On Thu, Jan 28, 2016 at 09:51:34PM -0500, Jason Merrill wrote: > On 01/28/2016 03:15 PM, Jakub Jelinek wrote: > >+if (TREE_CODE (decl) == FUNCTION_DECL > >+&& DECL_INITIAL (decl) == 0 > >+&& DECL_EXTERNAL (decl) > >+&& !TREE_PUBLIC (decl) > >+&& !DECL_ARTIFICIAL

Re: Martin Jambor appointed HSA Maintainer

Hi, On Fri, Dec 18, 2015 at 08:41:41AM -0500, David Edelsohn wrote: > I am pleased to announce that the GCC Steering Committee has > appointed Martin Jambor as HSA maintainer. > > Please join me in congratulating Martin on his new role. > Martin, please update your listing in the

Re: [PATCH] PR other/69006: S/390: Fix extra newlines after diagnostics.

On Wed, Jan 27, 2016 at 09:22:19AM -0500, David Malcolm wrote: > On Wed, 2016-01-27 at 12:01 +0100, Dominik Vogt wrote: > > The attached patch removes a blank line after an error message. > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69006 > > Presumably this was exposed by the stricter

[PATCH] Fix PR69547

I am testing the following patch to fix a regression that we no longer remove some empty loops. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2016-01-19 Richard Biener PR tree-optimization/69547 * tree-ssa-dce.c

[Patch,microblaze]: Better register allocation to minimize the spill and fetch.

This patch improves the allocation of registers in the given function. The allocation is optimized for the conditional branches. The temporary register used in the conditional branches to store the comparison results and use of temporary in the conditional branch is optimized. Such temporary

Re: [PATCH] PR c++/69462: Provide FLT_EVAL_METHOD and DECIMAL_DIG in float.h.

On Fri, Jan 29, 2016 at 09:27:46AM +0100, Dominik Vogt wrote: > On Thu, Jan 28, 2016 at 03:41:29PM +0100, Jakub Jelinek wrote: > > On Thu, Jan 28, 2016 at 01:40:12PM +0100, Dominik Vogt wrote: > > > -#if defined (__STDC_VERSION__) && __STDC_VERSION__ >= 199901L > > > +#if (defined

Re: [PATCH] S/390: Use %qs in error messages.

On Fri, Jan 29, 2016 at 10:06:47AM +0100, Dominik Vogt wrote: > gcc/ChangeLog > > * config/s390/s390-c.c (s390_resolve_overloaded_builtin): Format > declaration name with %qs and print it in both error messages. Also > fix indentation. Applied. Thanks! -Andreas-

Re: [PATCH] PR c++/69462: Provide FLT_EVAL_METHOD and DECIMAL_DIG in float.h.

On Thu, Jan 28, 2016 at 03:41:29PM +0100, Jakub Jelinek wrote: > On Thu, Jan 28, 2016 at 01:40:12PM +0100, Dominik Vogt wrote: > > -#if defined (__STDC_VERSION__) && __STDC_VERSION__ >= 199901L > > +#if (defined (__STDC_VERSION__) && __STDC_VERSION__ >= 199901L) \ > > + || (defined (__cplusplus)

[PATCH] S/390: Use %qs in error messages.

The attached patch replaces %qs instead of %s in an error message, adds %qs to another and fixes indentation in one of the messages. Compiled and checked that no tests rely on the changed error messages on a zEC12. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog *

Re: [PATCH] PR c++/69462: Provide FLT_EVAL_METHOD and DECIMAL_DIG in float.h.

On Fri, Jan 29, 2016 at 09:27:46AM +0100, Dominik Vogt wrote: > gcc/ChangeLog > > PR c++/69462 > * ginclude/float.h: Also provide FLT_EVAL_METHOD and DECIMAL_DIG for > C++-11. Applied. Thanks! -Andreas-

Re: [PATCH] Fix use of declare'd vars by routine procedures.

On Thu, Jan 28, 2016 at 12:26:38PM -0600, James Norris wrote: > I think the attached change is what you had in mind with > regard to doing the check at gimplification time. Nope, this is still a wrong location for that. If you look at the next line after the block you've added, you'll see if

Re: [PATCH] Fix PR69547

On Fri, 29 Jan 2016, Jakub Jelinek wrote: > On Fri, Jan 29, 2016 at 09:40:48AM +0100, Richard Biener wrote: > > I am testing the following patch to fix a regression that we no longer > > remove some empty loops. > > Doesn't this mean that DCE will remove the clobbers as unnecessary, even > when

Re: [C++ PATCH] Fix -Wunused-function (PR debug/66869)

On 01/28/2016 03:15 PM, Jakub Jelinek wrote: + if (TREE_CODE (decl) == FUNCTION_DECL + && DECL_INITIAL (decl) == 0 + && DECL_EXTERNAL (decl) + && !TREE_PUBLIC (decl) + && !DECL_ARTIFICIAL (decl) + && !TREE_NO_WARNING (decl)) Do we need to

Re: [C++ patch] report better diagnostic for static following '[' in parameter declaration

On 29 January 2016 at 05:03, Marek Polacek wrote: > On Fri, Jan 29, 2016 at 04:46:56AM +0530, Prathamesh Kulkarni wrote: >> @@ -19016,10 +19017,22 @@ cp_parser_direct_declarator (cp_parser* parser, >> cp_lexer_consume_token (parser->lexer); >> /* Peek at the

[PATCH PR67921]Convert pointer expr to proper type before negating it

Hi, Function fold_binary_loc calls split_tree to split a tree into constant, literal and variable parts. Function split_tree deals with minus_expr by negating different parts into NEXGATE_EXPR. Since tree exprs fed to split_tree are with NOP conversions stripped, this could result in illegal

Re: Default compute dimensions

On 01/29/16 10:18, Jakub Jelinek wrote: On Thu, Jan 28, 2016 at 10:38:51AM -0500, Nathan Sidwell wrote: This patch adds default compute dimension handling. Users rarely specify compute dimensions, expecting the toolchain to DTRT. More savvy users would like to specify global defaults. This

Re: Is it OK for rtx_addr_can_trap_p_1 to attempt to compute the frame layout? (was Re: [PATCH] Skip re-computing the mips frame info after reload completed)

On 01/29/2016 04:41 PM, Jakub Jelinek wrote: On Fri, Jan 29, 2016 at 02:09:25AM +0100, Bernd Schmidt wrote: I think a better approach might be to just mark accesses at known locations in the frame, or arg pushes, as MEM_NOTRAP_P, and consider accesses with non-constant or calculated offsets

[PATCH] [graphite] document that isl-0.16 is supported

* config/isl.m4: Add comments about isl-0.16. * configure: Regenerate. gcc/ * doc/install.texi: Document that isl-0.16 is supported. --- config/isl.m4| 6 +++--- configure| 12 ++-- gcc/doc/install.texi | 2 +- 3 files changed, 10

Re: [PATCH, RFC] New memory usage statistics infrastructure

On Thu, May 28, 2015 at 1:07 PM, Jeff Law wrote: > On 05/28/2015 06:29 AM, Martin Liška wrote: > >>> >> >> Hello. >> >> Thank you for pointing about missing copyright. >> Following patch adds that. >> >> Ready for trunk? > > Yes. > jeff > It looks like this patch was never

[RS6000] ABI_V4 init of toc section

Since 4c4a180d, LTO has turned off flag_pic when linking a fixed position executable. This results in flag_pic being zero in rs6000_file_start, and no definition of ".LCTOC1". However, when we get to actually emitting code, flag_pic may be on again, and references made to ".LCTOC1". How

Re: Is it OK for rtx_addr_can_trap_p_1 to attempt to compute the frame layout? (was Re: [PATCH] Skip re-computing the mips frame info after reload completed)

On Fri, Jan 29, 2016 at 02:09:25AM +0100, Bernd Schmidt wrote: > I'm on the fence; I do think the original problem is an issue we should fix, > but I'm also not terribly happy with the implementation we have right now. The fact that it has been only reported from generated testcases only means we

Re: [off-list] Re: [PATCH PR68542]

On Fri, Jan 29, 2016 at 3:13 PM, Yuri Rumyantsev wrote: > Uros, > > Here is update patch which includes (1) couple changes proposed by > Richard in tree-vect-loop.c and (2) the changes in back-end proposed > by you. > > Is it OK for trunk? > Bootstrap and regression testing

Re: [PATCH] s390: Add -fsplit-stack support

On 29/01/16 14:33, Andreas Krebbel wrote: Hi Marcin, sorry for the late feedback. A few comments regarding the split stack implementation: The GNU coding style requires to replace every 8 leading blanks on a line with a tab. There are many lines in your patch violating this. In case you are

Re: [PATCH] s390: Add -fsplit-stack support

On 01/29/2016 04:43 PM, Marcin Kościelnicki wrote: > The testsuite with -fsplit-stack already hits all of them, and checking > them manually is rather tricky (I don't know if it could be done in > target-independent way at all), but I think it'd be reasonable to make > assembly testcases

[PING] Re: [RFC][PATCH, ARM 8/8] Added support for ARMV8-M Security Extension cmse_nonsecure_caller intrinsic

On 26/12/15 01:59, Thomas Preud'homme wrote: [Sending on behalf of Andre Vieira] Hello, This patch adds support ARMv8-M's Security Extension's cmse_nonsecure_caller intrinsic. This intrinsic is used to check whether an entry function was called from a non-secure state. See Section 5.4.3 of

[PATCHv2] Re: [RFC][PATCH, ARM 7/8] ARMv8-M Security Extension's cmse_nonsecure_call: use __gnu_cmse_nonsecure_call]

On 19/01/16 15:28, Andre Vieira (lists) wrote: On 16/01/16 14:49, Senthil Kumar Selvaraj wrote: User-agent: mu4e 0.9.13; emacs 24.5.1 Hi, Apologies for the bad posting style (I don't have the original email handy), but shouldn't _gnu_cmse_nonsecure_call be defined with the .global directive

Enabling -frename-registers?

So PR57193 has an example of sub-optimal code generation, with some unnecessary register moves left after LRA. These seem to be difficult to prevent, but last year Robert Suchanek made some modifications to regrename that allow it to clean up such cases. Enabling -frename-registers removes one

Re: RFA: patch to fix PR69299

On 01/28/2016 09:07 AM, Vladimir Makarov wrote: On 01/28/2016 08:05 AM, Jakub Jelinek wrote: On Wed, Jan 27, 2016 at 04:01:23PM -0500, Vladimir Makarov wrote: The following patch fixes PR69299. The details of the problem is described on

Re: [Patch,microblaze]: Better register allocation to minimize the spill and fetch.

On 01/29/2016 02:31 AM, Ajit Kumar Agarwal wrote: This patch improves the allocation of registers in the given function. The allocation is optimized for the conditional branches. The temporary register used in the conditional branches to store the comparison results and use of temporary in the

[Patch, fortran, pr67451, v1] [5/6 Regression] ICE with sourced allocation from coarray

Hi all, attached is a patch to fix a regression in current gfortran when a coarray is used in the source=-expression of an allocate(). The ICE was caused by the class information, i.e., _vptr and so on, not at the expected place. The patch fixes this. The patch also fixes pr69418, which I will

[PING] Re: [RFC][PATCH, ARM 5/8] ARMv8-M Security Extension's cmse_nonsecure_entry: clear registers

On 26/12/15 01:54, Thomas Preud'homme wrote: [Sending on behalf of Andre Vieira] Hello, This patch extends support for the ARMv8-M Security Extensions 'cmse_nonsecure_entry' attribute to safeguard against leak of information through unbanked registers. When returning from a nonsecure entry

[PING] Re: [RFC][PATCH, ARM 6/8] Handling ARMv8-M Security Extension's cmse_nonsecure_call attribute

On 26/12/15 01:55, Thomas Preud'homme wrote: [Sending on behalf of Andre Vieira] Hello, This patch adds support for the ARMv8-M Security Extensions 'cmse_nonsecure_call' attribute. This attribute may only be used for function types and when used in combination with the '-mcmse' compilation

[PING] Re: [RFC][PATCH, ARM 3/8] Handling ARMv8-M Security Extension's cmse_nonsecure_entry attribute

On 26/12/15 01:47, Thomas Preud'homme wrote: [Sending on behalf of Andre Vieira] Hello, This patch adds support for the ARMv8-M Security Extensions 'cmse_nonsecure_entry' attribute. In this patch we implement the attribute handling and diagnosis around the attribute. See Section 5.4 of

Re: [RFC][PATCH , ARM 2/8] Add RTL patterns for thumb1 push/pop

On 26/12/15 01:45, Thomas Preud'homme wrote: [Sending on behalf of Andre Vieira] Hello, This patch adds RTL patterns for the push and pop instructions for thumb1. These are needed by subsequent patches in the series. *** gcc/ChangeLog *** 2015-10-27 Andre Vieira

Re: Is it OK for rtx_addr_can_trap_p_1 to attempt to compute the frame layout? (was Re: [PATCH] Skip re-computing the mips frame info after reload completed)

On 28.01.2016 23:17, Richard Sandiford wrote: > Bernd Edlinger writes: >> On 26.01.2016 22:18, Richard Sandiford wrote: >>> [cc-ing Eric as RTL maintainer] >>> >>> Matthew Fortune writes: Bernd Edlinger

Re: [PATCH, rs6000] Fix PR65546

On Thu, Jan 28, 2016 at 5:41 PM, Bill Schmidt wrote: > Hi, > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65546 identifies a failure > in gcc.dg/vect/costmodel/ppc/costmodel-vect-31a.c. The test case hasn't > kept up with changes in the vectorizer, so it's looking

Re: [RS6000] ABI_V4 init of toc section

On Fri, Jan 29, 2016 at 11:38 AM, Alan Modra wrote: > Since 4c4a180d, LTO has turned off flag_pic when linking a fixed > position executable. This results in flag_pic being zero in > rs6000_file_start, and no definition of ".LCTOC1". > > However, when we get to actually

[PATCHv2] Re: [RFC][PATCH, ARM 1/8] Add support for ARMv8-M's Security Extensions flag and intrinsics

On 05/01/16 14:38, Andre Vieira wrote: On 31/12/15 20:54, Joseph Myers wrote: On Sat, 26 Dec 2015, Thomas Preud'homme wrote: +#define CMSE_TT_ASM(flags) \ +{ \ + cmse_address_info_t result; \ + __asm__ ("tt" # flags " %0,%1" \ + : "=r"(result) \ + : "r"(p) \ + :

[PING] Re: [RFC][PATCH, ARM 4/8] ARMv8-M Security Extension's cmse_nonsecure_entry: __acle_se label and bxns return

On 26/12/15 01:52, Thomas Preud'homme wrote: [Sending on behalf of Andre Vieira] Hello, This patch extends support for the ARMv8-M Security Extensions 'cmse_nonsecure_entry' attribute in two ways: 1) Generate two labels for the function, the regular function name and one with the function's

[commited, PATCH] PR target/69530: [6 Regression] ICE: SIGSEGV

in ix86_split_long_move (i386.c:24353) with -fno-split-wide-types -mavx Reply-To: "H.J. Lu" r229087, which caused PR 69530, was supposed to fix PR 67609. r229458 has made r229087 unnecessary. Approved by Vladimir in PR 69530. Checked into trunk. H.J. --- gcc/

[wwwdocs] fortran/index.html - remove local styles

These local styles feel a bit odd to begin with, and if we skip the ... within ..., the originally perceived/ addresses issue should go away. Unless there are objections from the Fortran side, I plan on committing this in a couple of days. Gerald Index: fortran/index.html

[wwwdocs] Use external CSS for the News and Status panes on the main page

This fixes the worst of what the new server settings broke; our main page should look quite more reasonable with this. And it simplifies things a little. Applied. Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v

Re: [PATCH] Fix up _Pragma GCC diagnostics regressions (PR preprocessor/69543, PR c/69558)

On Fri, 2016-01-29 at 20:50 +0100, Jakub Jelinek wrote: > Hi! > > This patch reverts one tiny change from r228049 changes (which hasn't > been > mentioned in the ChangeLog or patch description). We definitely need > to > revisit this for GCC 7, but stage4 is probably not the right time for >

[wwwdocs] Avoid local styles in the standard footer.

Our standard footer also got hit by the (needlessly for for us) stricter server settings. This again gets rid of extra vertical space in that box. Installed, and afterwards I rebuilt our website using the /www/gcc/bin/preprocess script on gcc.gnu.org. Gerald Index: gcc.css

Re: Enabling -frename-registers?

On January 29, 2016 6:34:50 PM GMT+01:00, Bernd Schmidt wrote: >So PR57193 has an example of sub-optimal code generation, with some >unnecessary register moves left after LRA. These seem to be difficult >to >prevent, but last year Robert Suchanek made some modifications to

patch to fix PR69299

The following patch is the final version of the patch for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69299 The patch was approved by Richard Henderson and Jakub. Committed as rev. 232993. Index: ChangeLog === ---

Re: [C++ patch] report better diagnostic for static following '[' in parameter declaration

On 29/01/16 17:01, Prathamesh Kulkarni wrote: Thanks for the review. AFAIK the type-qualifiers would be const, restrict, volatile and _Atomic (n1570 p 6.7.3) ? I added a check for those and for variable length array. I am having issues with writing the test-case, some cases pass with -std=c++11

Re: [C PATCH] Clear C_TYPE_INCOMPLETE_VARS even on variant types (PR debug/69518)

On Fri, Jan 29, 2016 at 08:41:18PM +0100, Jakub Jelinek wrote: > Hi! > > We ICE on the following testcase, because the C FE abuses TYPE_VFIELD > for its FE stuff, but may leak it to the middle-end. > We clear it for TYPE_MAIN_VARIANT (and only use it for that), but > for the other variants it

[C PATCH] Clear C_TYPE_INCOMPLETE_VARS even on variant types (PR debug/69518)

Hi! We ICE on the following testcase, because the C FE abuses TYPE_VFIELD for its FE stuff, but may leak it to the middle-end. We clear it for TYPE_MAIN_VARIANT (and only use it for that), but for the other variants it could be non-NULL, because build_variant_type* would just copy that field

[PATCH] Fix up _Pragma GCC diagnostics regressions (PR preprocessor/69543, PR c/69558)

Hi! This patch reverts one tiny change from r228049 changes (which hasn't been mentioned in the ChangeLog or patch description). We definitely need to revisit this for GCC 7, but stage4 is probably not the right time for that, and the patch fixes e.g. tons of warnings (or with -Werror errors on

Re: Is it OK for rtx_addr_can_trap_p_1 to attempt to compute the frame layout? (was Re: [PATCH] Skip re-computing the mips frame info after reload completed)

On 29.01.2016 16:47 Bernd Schmidt wrote: > On 01/29/2016 04:41 PM, Jakub Jelinek wrote: >> On Fri, Jan 29, 2016 at 02:09:25AM +0100, Bernd Schmidt wrote: > >>> I think a better approach might be to just mark accesses at known >>> locations >>> in the frame, or arg pushes, as MEM_NOTRAP_P, and

[Patch, MIPS] Fix PR target/68273, passing args in wrong regs

This is a patch for PR 68273, where MIPS is passing arguments in the wrong registers. The problem is that when passing an argument by value, mips_function_arg_boundary was looking at the type of the argument (and not just the mode), and if that argument was a variable with extra alignment info