[PATCH] PR c/70281: C FE: fix uninitialized range for __builtin_types_compatible_p

PR c/70281 reports another case where Valgrind identified an uninitialized src_range in a c_expr in the C frontend, this time in the parsing of __builtin_types_compatible_p. For gcc 7 I hope to fix this more robustly (via poisoning the values in a c_expr ctor), but for now, this patch fixes the sp

Re: [PATCH, aarch64] Fix 70048

On Wed, Mar 16, 2016 at 02:25:27PM -0700, Richard Henderson wrote: > PR target/70048 > * config/aarch64/aarch64.c (virt_or_elim_regno_p): New. > (aarch64_classify_address): Use it. > (aarch64_legitimize_address): Force all subexpressions of PLUS > into regist

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

On 03/17/2016 04:13 PM, H.J. Lu wrote: On Thu, Mar 17, 2016 at 8:09 AM, Bernd Schmidt wrote: On 03/17/2016 04:06 PM, H.J. Lu wrote: This is the patch I am going to check in. That still mentions darwin which I imagine might not be an exhaustive test. We can add an effective target, somethin

[C++ PATCH] Diagnose invalid _Jv_AllocObject prototype (PR c++/70267)

Hi! _Jv_AllocObject returns a pointer, and as the testcase below shows, we easily ICE if a wrong prototype is provided for it instead. There is already other diagnostics (e.g. when it is missing, or when it is overloaded function), so this ensures at least the return type is sane. Wonder about al

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

On 03/17/2016 02:59 PM, H.J. Lu wrote: On Fri, Mar 11, 2016 at 9:09 AM, H.J. Lu wrote: We can't set flag_pie to the default when flag_pic == 0, which may be set by -fno-pic or -fno-PIC, since the default value of flag_pie is non-zero when GCC is configured with --enable-default-pie. We need to

[i386] Support .lbss etc. sections with Solaris as (PR target/59407)

gcc.target/i386/pr58218.c currently FAILs on 64-bit Solaris/x86 with the native assembler: FAIL: gcc.target/i386/pr58218.c (test for excess errors) Excess errors: Assembler: pr58218.c "/var/tmp//cciHFIO7.s", line 3 : Section attributes do not match .section.lbss,"aw",@nob

Re: [PATCH] Fix PR c++/70218 (illegal access to private field succeeds)

OK. Jason

[AArch64] Add precision choices for the reciprocal square root approximation

Add precision choices for the reciprocal square root approximation Allow a target to prefer such operation depending on the FP precision. gcc/ * config/aarch64/aarch64-protos.h (AARCH64_EXTRA_TUNE_APPROX_RSQRT): New macro. * config/a

Re: [PATCH] Fix PR c++/70218 (illegal access to private field succeeds)

On Wed, Mar 16, 2016 at 5:20 PM, Matthias Klose wrote: > On 13.03.2016 21:03, Patrick Palka wrote: >> >> Here we are mishandling the deferred_access_stack by not coherently >> pushing/popping from it. In cp_parser_lambda_expression we are calling >> (in order): >> >>push_deferring_access_chec

[gomp-nvptx 7/7] nvptx backend: define STACK_SIZE_MODE

Default definition of STACK_SIZE_MODE is word_mode, which is DImode on NVPTX. However, stack pointer mode matches pointer mode, so needs to be SImode on 32-bit NVPTX ABI. Define it to Pmode to fix 32-bit code generation. * config/nvptx/nvptx.h (STACK_SIZE_MODE): Define. --- gcc/ChangeLog

Re: [PATCH] Add debug_varinfo and debug_varmap

On Wed, 16 Mar 2016, Tom de Vries wrote: > [ was: Re: [RFC] dump_varmap in tree-ssa-structalias.c ] > On 10/03/16 10:07, Richard Biener wrote: > > On Thu, 10 Mar 2016, Tom de Vries wrote: > > > > > Hi, > > > > > > I wrote attached patch to print the actual contents of the varmap variable > > > i

Re: [AArch64] Emit square root using the Newton series

On 03/17/16 09:55, James Greenhalgh wrote: On Wed, Mar 16, 2016 at 02:45:37PM -0500, Evandro Menezes wrote: On 03/08/16 16:08, Evandro Menezes wrote: On 02/16/16 14:56, Evandro Menezes wrote: On 12/08/15 15:35, Evandro Menezes wrote: Emit square root using the Newton series 2015-12-03 Ev

Re: [PATCH] c++/65579 - set readonly bit on static constexpr members of templates

On 03/17/2016 08:07 AM, Jason Merrill wrote: On 03/09/2016 05:09 PM, Martin Sebor wrote: While going through constexpr bugs looking for background on one I'm currently working on I came across bug 65579 - [C++11] gcc requires definition of a static constexpr member even though it is not odr-used

Re: [PATCH V3]PR other/70268: map one directory name (old) to another (new) in __FILE__

On March 18, 2016 6:16:46 AM GMT+01:00, Hongxu Jia wrote: +/* Perform user-specified mapping of __FILE__ prefixes. Return + the new name corresponding to filename. */ + +const char * +remap_file_filename (const char *filename) +{ + file_prefix_map *map; + char *s; + const char *name; +

[PATCH, PR70269] Set dump_file to NULL in cgraph_node::get_body

Hi, this patch fixes PR70269, an 5/6 regression. When compiling with "-O2 -fipa-pta -fdump-ipa-pta-graph" we try to initialize a graph dump file for ipa-cp, while the dump file is not enabled, which causes an ICE because dump_file_name is NULL. This condition in pass_init_dump_file enables t

Re: [PATCH, PR70161] Fix fdump-ipa-all-graph

On 15/03/16 12:37, Richard Biener wrote: On Mon, 14 Mar 2016, Tom de Vries wrote: Hi, this patch fixes PR70161, a 4.9/5/6 regression. Currently when using -fdump-ipa-all-graph, the compiler ICEs in execute_function_dump when testing for pass->graph_dump_initialized, because pass == NULL. The

Re: [PATCH] Use simplify_replace_rtx instead of replace_rtx for DEBUG_INSNs in reload

On Fri, 18 Mar 2016, Jakub Jelinek wrote: > Hi! > > This patch fixes one of the spots that use replace_rtx, as this changes > debug insns, it clearly wants to replace just based on regno, not on pointer > equality, and for debug insns simplification is always desirable too. > The other place in r

Re: [C PATCH] Fix up composite_types (PR c/70280)

On Thu, 17 Mar 2016, Jakub Jelinek wrote: > Hi! > > Zdenek reported a compare debug issue, where it is dumping used function > prototypes and there is a difference between -g0 and -g in > -2: static int BIO_vsnprintf (char *, size_t, const char *, struct *, > void, ...); > +2: static int BI

Patch ping

Hi! I'd like to ping a C++ PR70144 patch: http://gcc.gnu.org/ml/gcc-patches/2016-03/msg00653.html Since then, Zdenek has filed another PR with __builtin_constant_p (foo) where foo is FUNCTION_DECL, and only the above version fixes that, not the other one. Jakub

Re: [PATCH] Fix PR64764

On Wed, Mar 16, 2016 at 9:35 AM, Tom de Vries wrote: > On 16/03/16 17:15, H.J. Lu wrote: >> >> On Wed, Mar 16, 2016 at 9:12 AM, H.J. Lu wrote: > > >>> Any particular reason why this test was changed to DOS format? > > > FWIW, the test was in DOS format from the start. > > DOS format was introduc

[PATCH, aarch64] Fix 70048

This fixes only the regression described in the PR. There was quite a bit of follow-up that points to new work that ought to be done during the gcc7 cycle, but isn't really appropriate now. Tested on aarch64-linux; committed as reviewed in the PR. r~ PR target/70048 * config/

[PATCH, i386, AVX-512] Emit vpbroadcastq instead if non-existent vbroadcastsd.

Hello, Intel spec [1] states that there're almost all broadcasting intructions variants available, except for (p. 2-4) vbroadcastsd %xmm, %xmm It is safe to emit vpbroadcastq %xmm, %xmm instead. I was uable to extract a testcase, but if this insn is generated - we'll got asm error.

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. Done. + bitmap_set_bit (seen_insns, INSN_UID (i

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

On Tue, 5 Jan 2016, Richard Biener wrote: On January 5, 2016 2:20:42 PM GMT+01:00, Bernd Edlinger wrote: On 05.01.2016 13:58, Bernd Schmidt wrote: On 01/05/2016 09:44 AM, Bernd Edlinger wrote: Using asm code is generally not desirable for in-tree mpfr builds. Why not? for the same reason

Re: [patch, driver] Ignore -ftree-parallelize-loops={0,1}

Hi! On Tue, 14 Jul 2015 10:36:25 +0200, Tom de Vries wrote: > On 14/07/15 06:54, Jeff Law wrote: > > On 07/13/2015 04:58 AM, Tom de Vries wrote: > >> On 07/07/15 09:53, Tom de Vries wrote: > >>> currently, we have these spec strings in gcc/gcc.c involving > >>> ftree-parallelize-loops: > >>> ...

[AArch64] Emit division using the Newton series

Emit division using the Newton series 2016-03-17 Evandro Menezes gcc/ * config/aarch64/aarch64-tuning-flags.def (AARCH64_EXTRA_TUNE_APPROX_DIV_{SF,DF}: New tuning macros. * config/aarch64/aarch64-protos.h (AARCH64_EXTRA_T

Re: C PATCH for c/70093 (ICE with nested-function returning VM type)

Ping. On Wed, Mar 09, 2016 at 04:55:23PM +0100, Marek Polacek wrote: > On Wed, Mar 09, 2016 at 04:31:37PM +0100, Jakub Jelinek wrote: > > No, I meant: > > switch (n) > > { > > struct S x; > > case 1: > > fn (); > > break; > > case 2: > > fn2 (); > > brea

[hsa branch] Use an obstack instead of multiple alloc pools

Hi, when I started working on expansion to HSAIL almost three years ago, I decided to allocate memory for most of the structures from various alloc-pools for reasons that never materialized and the number of pools later grew to unreasonable numbers. So after an internal discussion, Martin Liska w

Re: [PATCH, PR70185] Only finalize dot files that have been initialized

On Thu, Mar 17, 2016 at 10:19 AM, Tom de Vries wrote: > On 16/03/16 12:34, Richard Biener wrote: >> >> On Wed, Mar 16, 2016 at 11:57 AM, Tom de Vries >> wrote: >>> >>> Hi, >>> >>> Atm, using fdump-tree-all-graph produces invalid dot files: >>> ... >>> $ rm *.c.* ; gcc test.c -O2 -S -fdump-tree-al

Re: [PATCH][SPARC] sparc: switch -fasynchronous-unwind-tables on by default.

On 02/29/2016 07:50 AM, Jose E. Marchesi wrote: > The backtrace(3) implementation for sparc contains a simple unwinder > that works well in most cases, but that unwinder is not used if > libgcc_s.so can be dlopened and it provides _Unwind_Backtrace. There's no reason that simple unwinder can't be

[PATCH, PR70183] Propagate dump flags in pass_manager::register_pass

Hi, atm dumpfile vzeroupper is not influenced by the flags in -fdump-rtl-all-flags. The patch fixes this by copying the flags in pass_manager::register_pass. OK for stage1 if bootstrap and reg-test succeeds? Thanks, - Tom Propagate dump flags in pass_manager::register_pass 2016-03-16 Tom

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On Wed, Mar 16, 2016 at 9:58 AM, Jason Merrill wrote: > On 03/16/2016 08:38 AM, H.J. Lu wrote: >> >> FAIL: g++.dg/abi/pr60336-1.C scan-assembler jmp[\t >> ]+[^$]*?_Z3xxx9true_type >> FAIL: g++.dg/abi/pr60336-5.C scan-assembler jmp[\t >> ]+[^$]*?_Z3xxx9true_type >> FAIL: g++.dg/abi/pr60336-6.C

Re: [PATCH V3]PR other/70268: map one directory name (old) to another (new) in __FILE__

On 03/18/2016 03:56 PM, Bernhard Reutner-Fischer wrote: On March 18, 2016 6:16:46 AM GMT+01:00, Hongxu Jia wrote: +/* Perform user-specified mapping of __FILE__ prefixes. Return + the new name corresponding to filename. */ + +const char * +remap_file_filename (const char *filename) +{ +

Re: [PATCH, PR70161] Fix fdump-ipa-all-graph

On Thu, 17 Mar 2016, Tom de Vries wrote: > On 15/03/16 12:37, Richard Biener wrote: > > On Mon, 14 Mar 2016, Tom de Vries wrote: > > > > > Hi, > > > > > > this patch fixes PR70161, a 4.9/5/6 regression. > > > > > > Currently when using -fdump-ipa-all-graph, the compiler ICEs in > > > execute_fu

Re: [PATCH]PR other/70268: map one directory name (old) to another (new) in __FILE__

On Thu, 17 Mar 2016, Hongxu Jia wrote: > + if (add_file_prefix_map(arg) < 0) Bad formatting (missing space before '('). Likewise elsewhere in this patch. > +@item -ffile-prefix-map=@var{old}=@var{new} > +@opindex ffile-prefix-map > +When parsing __FILE__, __BASE_FILE__ and __builtin_FILE(

Re: [PATCH PR69489/01]Improve tree ifcvt by storing/tracking DR against its innermost loop bahavior if possible

On Wed, Mar 16, 2016 at 12:20 PM, Richard Biener wrote: > > On Wed, Mar 16, 2016 at 10:59 AM, Bin Cheng wrote: > > Hi, > > .. > > Bootstrap and test on x86_64 and AArch64. Is it OK, not sure if it's GCC 7? > > Hmm. Hi, Thanks for reviewing. > > + equal_p = true; > + if (e1->base_address &&

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/18/2016 08:14 PM, Jeff Law wrote: I also added a blurb to the dump file when we create these equivalences and included a test to verify the code fires. I verified it fired on x86 and x86-64. It may or may not fire on other targets, so I left the test in the i386 specific subdirectory. T

Re: C++ PATCH to fix missing warning (PR c++/70194)

While I would find the warning less misleading if it simply said in all three cases: "the address of 'x' will always evaluate as ‘true’" I think it would be even more accurate if it said "the address of 'x' may be assumed to evaluate to ‘true’" That avoids making claims about whether or not it ac

Re: [AArch64] Emit square root using the Newton series

On Wed, Mar 16, 2016 at 02:45:37PM -0500, Evandro Menezes wrote: > On 03/08/16 16:08, Evandro Menezes wrote: > >On 02/16/16 14:56, Evandro Menezes wrote: > >>On 12/08/15 15:35, Evandro Menezes wrote: > >>>Emit square root using the Newton series > >>> > >>> 2015-12-03 Evandro Menezes > >>> > >

Re: [PATCH, PR tree-optimization/70251] Disable VEC_COND_EXPR transformation into VIEW_CONVERT_EXPR for scalar mask case

On Thu, Mar 17, 2016 at 11:23 AM, Ilya Enkovich wrote: > Hi, > > This patch disables two match.pd patterns which transform > VEC_COND_EXPR into simple conversion in case it uses a scalar mask. > The patch was bootstrapped and regtested on x86_64-pc-linux-gnu + > separate check for new test on SDE.

Re: [PATCH] Retry to emit global variables in HSA (PR hsa/70234)

Hi, On Tue, Mar 15, 2016 at 12:59:03PM +0100, Martin Liska wrote: > Hi. > > As emission of a HSAIL function can fail for various reason (-Whsa), > we must guarantee that a global variable is declared and at maximum once. > > Following patch does that, patch can survive make check-target-libgomp

Re: C PATCH for c/70093 (ICE with nested-function returning VM type)

On Wed, Mar 16, 2016 at 04:11:56PM +0100, Marek Polacek wrote: > > 2016-03-09 Marek Polacek > > > > PR c/70093 > > * c-typeck.c (build_function_call_vec): Create a TARGET_EXPR for > > nested functions returning VM types. > > > > * cgraphunit.c (cgraph_node::expand_thunk): Also

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On Tue, Mar 15, 2016 at 12:32 PM, Jason Merrill wrote: > On 03/15/2016 12:00 PM, H.J. Lu wrote: >> >> On Tue, Mar 15, 2016 at 8:35 AM, Jason Merrill wrote: >>> >>> I'm concerned about how this patch changes both target-independent code >>> and >>> target-specific code, with a passing remark that

Re: C++ PATCH to fix missing warning (PR c++/70194)

On 03/17/2016 10:45 AM, Jason Merrill wrote: On 03/16/2016 08:43 PM, Martin Sebor wrote: @@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, return build3 (VEC_COND_EXPR, type, cmp, minus_one_vec, zero_vec); } +/* Possibly warn about an address never being NULL. */ + +static

Re: [PATCH] Fix rs6000 vector builtin macro handling if it is followed by a fn-like macro without arguments (PR target/70296)

On Fri, Mar 18, 2016 at 5:34 PM, Jakub Jelinek wrote: > Hi! > > The following testcase is diagnosed as errorneous, because the preprocessor > mishandles > > #define c(x) x > vector c; > > and > > #define int(x) x > vector int n; > > The thing is if a function-like macro is not followed by (, then

Re: [C PATCH] Prevent -Wunused-value warning with __atomic_fetch_* (PR c/69407)

On 03/14/2016 05:48 AM, Marek Polacek wrote: Ping. On Fri, Mar 04, 2016 at 07:03:09PM +0100, Marek Polacek wrote: On Fri, Mar 04, 2016 at 06:41:26PM +0100, Jakub Jelinek wrote: I'm ok with it for gcc6. Cool. But IMHO you should add dg-bogus directives here. Ok, version with dg-bogus: Bo

Re: C++ PATCH to fix missing warning (PR c++/70194)

On 03/17/2016 10:48 AM, Patrick Palka wrote: On Thu, Mar 17, 2016 at 12:27 PM, Jeff Law wrote: On 03/16/2016 06:43 PM, Martin Sebor wrote: @@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, return build3 (VEC_COND_EXPR, type, cmp, minus_one_vec, zero_vec); } +/* Possibl

Fix 70278 (LRA split_regs followup patch)

This fixes an oversight in my previous patch here. I used biggest_mode in the assumption that if the reg was used in the function, it would be set to something other than VOIDmode, but that fails if we have a multiword access - only the first hard reg gets its biggest_mode assigned in that case

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On Wed, Mar 16, 2016 at 2:45 AM, Bernhard Reutner-Fischer wrote: > On March 16, 2016 3:17:20 AM GMT+01:00, "H.J. Lu" wrote: > >>> Where is the current definition of empty types you're proposing for >>use in >>> GCC? Is the behavior of this case clear from that definition? >> >>https://gcc.gnu.or

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On Fri, Mar 18, 2016 at 04:05:08PM -0400, David Malcolm wrote: > On Fri, 2016-03-18 at 13:20 -0600, Jeff Law wrote: > > On 03/18/2016 01:16 PM, Bernd Schmidt wrote: > > > On 03/18/2016 08:14 PM, Jeff Law wrote: > > > > I also added a blurb to the dump file when we create these > > > > equivalences

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

On 03/14/2016 04:13 PM, Jakub Jelinek wrote: On Mon, Mar 14, 2016 at 03:25:07PM -0600, Martin Sebor wrote: PR c++/67376 - [5/6 regression] Comparison with pointer to past-the-end of array fails inside constant expression PR c++/70170 - [6 regression] bogus not a constant expression error

Re: [PATCH] Fix PR c++/70121 (premature folding of const var that was implicitly captured)

On Thu, Mar 10, 2016 at 6:06 PM, Patrick Palka wrote: > On Thu, Mar 10, 2016 at 5:58 PM, Patrick Palka wrote: >> Within a lambda we should implicitly capture an outer const variable >> only if it's odr-used in the body of the lambda. But we are currently >> making the decision of whether to capt

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On Tue, Mar 15, 2016 at 7:51 PM, Jason Merrill wrote: > On 03/15/2016 08:25 PM, Joseph Myers wrote: >> >> On Tue, 15 Mar 2016, H.J. Lu wrote: >> >>> On Tue, Mar 15, 2016 at 3:34 PM, Joseph Myers >>> wrote: On Tue, 15 Mar 2016, H.J. Lu wrote: > On Tue, Mar 15, 2016 at 2:39 PM, J

C++ PATCH for c++/70147 (-fsanitize=vptr, -flifetime-dse, and virtual bases)

The first patch factors out testing of current_in_charge_parm from various places in the compiler into a new build_if_in_charge function. The second patch implements Bernd's suggestion for modifying -flifetime-dse so that when we have virtual bases, we clobber the entire object, but only when

[PATCH, PR tree-optimization/70251] Disable VEC_COND_EXPR transformation into VIEW_CONVERT_EXPR for scalar mask case

Hi, This patch disables two match.pd patterns which transform VEC_COND_EXPR into simple conversion in case it uses a scalar mask. The patch was bootstrapped and regtested on x86_64-pc-linux-gnu + separate check for new test on SDE. OK for trunk? Thanks, Ilya -- gcc/ 2016-03-17 Ilya Enkovich

Re: [PATCH][ARM][testsuite][committed] Do not override -mcpu in no-volatile-in-it.c

On Fri, Mar 18, 2016 at 10:31 AM, Andre Vieira (lists) wrote: > On 16/07/15 16:31, Kyrill Tkachov wrote: >> Hi all, >> >> This scan-assembler test was failing for me when testing with an >> explicit /-march=armv7-a variant because >> it clashed with the -mcpu=cortex-m7 and overrode it. >> >> This

Scan for parallelization of the oacc kernels test-cases in gfortran.dg/goacc (was: [PATCH, 15/16] Add libgomp.oacc-c-c++-common/kernels-*.c)

Hi! On Wed, 9 Mar 2016 10:17:28 +0100, Tom de Vries wrote: > [Should have cited > > instead of the C/C++ tests] > Retested on current trunk. > > Committed, minus the kernels-parallel-loop-data-enter-exit.f95 te

Re: [PATCH][ARM] Split out armv7ve effective target check

On Wed, Mar 2, 2016 at 1:32 PM, Kyrill Tkachov wrote: > Hi all, > > I'm seeing the fails: > FAIL: gcc.target/arm/atomic_loaddi_2.c scan-assembler-times ldrd\tr[0-9]+, > r[0-9]+, \\[r[0-9]+\\] 1 > FAIL: gcc.target/arm/atomic_loaddi_5.c scan-assembler-times ldrd\tr[0-9]+, > r[0-9]+, \\[r[0-9]+\\] 1

Rename GOMP_MAP_FORCE_DEALLOC to GOMP_MAP_DELETE (was: [gomp4.1] map clause parsing improvements)

Hi! On Thu, 17 Mar 2016 15:37:04 +0100, Jakub Jelinek wrote: > On Thu, Mar 17, 2016 at 03:34:09PM +0100, Thomas Schwinge wrote: > > That's simple enouch; OK to commit? (I'm also including the related > > change, to rename the Fortran OMP_MAP_FORCE_DEALLOC to OMP_MAP_DELETE, > > because I think t

Re: [PATCH, PR70269] Set dump_file to NULL in cgraph_node::get_body

On Thu, 17 Mar 2016, Tom de Vries wrote: > Hi, > > this patch fixes PR70269, an 5/6 regression. > > When compiling with "-O2 -fipa-pta -fdump-ipa-pta-graph" we try to initialize > a graph dump file for ipa-cp, while the dump file is not enabled, which causes > an ICE because dump_file_name is NU

Re: [PATCH] Fix PR64764

On 16/03/16 17:15, H.J. Lu wrote: On Wed, Mar 16, 2016 at 9:12 AM, H.J. Lu wrote: Any particular reason why this test was changed to DOS format? FWIW, the test was in DOS format from the start. Thanks - Tom

Re: [PATCH] Fix PR c++/70218 (illegal access to private field succeeds)

On 13.03.2016 21:03, Patrick Palka wrote: Here we are mishandling the deferred_access_stack by not coherently pushing/popping from it. In cp_parser_lambda_expression we are calling (in order): push_deferring_access_checks (dk_no_deferred); cp_parser_start_tentative_firewall (parser); .

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On Fri, 2016-03-18 at 13:20 -0600, Jeff Law wrote: > On 03/18/2016 01:16 PM, Bernd Schmidt wrote: > > On 03/18/2016 08:14 PM, Jeff Law wrote: > > > I also added a blurb to the dump file when we create these > > > equivalences > > > and included a test to verify the code fires. I verified it > > >

New French PO file for 'cpplib' (version 6.1-b20160131)

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

Re: [C++ PATCH] Fix -flifetime-dse bug in dtors too (PR c++/70272)

OK. Jason

Re: [PATCH] PR testsuite/70150: Check non-pic/ia32 in stackprotectexplicit2.C

"H.J. Lu" writes: > For ia32, __stack_chk_fail isn't called in PIC. We need to check > non-pic or non-ia32 before scanning for __stack_chk_fail. > > OK for trunk? > > H.J. > --- > PR testsuite/70150 There's already PR c++/66400 (and a bunch of others for several related issues) which I'd

[PATCH] Use simplify_replace_rtx instead of replace_rtx for DEBUG_INSNs in reload

Hi! This patch fixes one of the spots that use replace_rtx, as this changes debug insns, it clearly wants to replace just based on regno, not on pointer equality, and for debug insns simplification is always desirable too. The other place in reload1 that modifies DEBUG_INSNs also uses simplify_rep

[PATCH][PR rtl-optimization/70024] Fix argument to CROSSING_JUMP_P

As noted in the BZ, we were passing a SEQUENCE to CROSSING_JUMP_P, which triggers an RTL checking failure. It's pretty obvious that we should have been passing in "delay_jump_insn" and doing so, of course, fixes the failure. I haven't been able to put together a sparc64 system for testing u

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

On 03/17/2016 04:06 PM, H.J. Lu wrote: This is the patch I am going to check in. That still mentions darwin which I imagine might not be an exhaustive test. Bernd

Re: [PATCH] PR testsuite/70150: Check non-pic/ia32 in stackprotectexplicit2.C

On Thu, Mar 17, 2016 at 6:48 AM, Rainer Orth wrote: > "H.J. Lu" writes: > >> For ia32, __stack_chk_fail isn't called in PIC. We need to check >> non-pic or non-ia32 before scanning for __stack_chk_fail. >> >> OK for trunk? >> >> H.J. >> --- >> PR testsuite/70150 > > There's already PR c++/

<    1   2