[Bug c++/67825] [concepts] expression constraint bug when taking address of a member function

2020-04-03 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67825 Jason Merrill changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED CC|

Re: [PATCH] c++: Refrain from using replace_placeholders in constexpr evaluation [PR94205]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/3/20 4:07 PM, Patrick Palka wrote: On Fri, 3 Apr 2020, Jason Merrill wrote: On 4/2/20 7:40 PM, Patrick Palka wrote: + /* Prefer the outermost matching object, but don't cross + CONSTRUCTOR_PLACEHOLDER_BOUNDARY constructors. */ + if (ctx->ctor && !CONSTRUCTOR_PLACEHOLDER_BOUNDARY

Re: [PATCH v2] c++: Fix further protected_set_expr_location related -fcompare-debug issues [PR94441]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/3/20 4:12 PM, Jakub Jelinek wrote: On Fri, Apr 03, 2020 at 03:02:58PM -0400, Jason Merrill via Gcc-patches wrote: Or do we want a further wrapper, perhaps C++ FE only, that would do this protected_set_expr_location_if_unset (stmt, locus)? That sounds good to me. So like this if it

Re: [PATCH] c++: Fix constexpr evaluation of self-modifying CONSTRUCTORs [PR94219]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/3/20 4:57 PM, Patrick Palka wrote: On Fri, 3 Apr 2020, Patrick Palka wrote: On Fri, 3 Apr 2020, Jason Merrill wrote: On 4/2/20 2:19 PM, Patrick Palka wrote: On Thu, 2 Apr 2020, Patrick Palka wrote: This PR reveals that cxx_eval_bare_aggregate and cxx_eval_store_expression do not

Re: [PATCH] debug: Improve debug info of c++14 deduced return type [PR94459]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/3/20 6:47 PM, Jakub Jelinek wrote: Hi! On the following testcase, in gdb ptype S::m1 prints long as return type, but all the other methods show void instead. PR53756 added code to add_type_attribute if the return type is auto/decltype(auto), but we actually should look through references,

Re: [PATCH v2] c++: Fix crash in gimplifier with paren init of aggregates [PR94155]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/3/20 9:08 PM, Marek Polacek wrote: On Fri, Apr 03, 2020 at 03:01:37PM -0400, Jason Merrill via Gcc-patches wrote: On 3/30/20 4:28 PM, Marek Polacek wrote: Here we crash in the gimplifier because gimplify_init_ctor_eval doesn't expect null indexes for a constructor: /* ??? Here's

[RFA] skip gcc.target/arm/div64-unwinding.c on vxworks_kernel targets

2020-04-03 Thread Joel Brobecker
Hello, This test verifies, by using a weak reference to _Unwind_RaiseException, that performing division by zero does not cause that symbol to get indirectly pulled into our closure. The testing methodology unfortunately does not work on VxWorks targets when building in kernel mode. This is

Re: [PATCH v2] c++: Fix crash in gimplifier with paren init of aggregates [PR94155]

2020-04-03 Thread Marek Polacek via Gcc-patches
On Fri, Apr 03, 2020 at 03:01:37PM -0400, Jason Merrill via Gcc-patches wrote: > On 3/30/20 4:28 PM, Marek Polacek wrote: > > Here we crash in the gimplifier because gimplify_init_ctor_eval doesn't > > expect null indexes for a constructor: > > > >/* ??? Here's to hoping the front end

Re: [PATCH] rs6000: Don't split constant oprator when add, move to temp register for future optimization

2020-04-03 Thread Alan Modra via Gcc-patches
On Fri, Apr 03, 2020 at 04:11:32PM -0500, Segher Boessenkool wrote: > On Fri, Apr 03, 2020 at 10:43:49PM +1030, Alan Modra wrote: > > Segher probably meant the part I'm working on and haven't posted yet, > > fixing lots of problems with rs6000_rtx_costs. > > I meant PR94393 in fact, but yeah,

[committed] Fix xstormy16 fallout from Jakub's cselib changes

2020-04-03 Thread Jeff Law via Gcc-patches
This fixes more fallout from Jakub's cselib changes. This shows up as an ICE building stdarg-3 from the testsuite after Jakub's changes. However, a reduced testcase will fail miserably all the way back to gcc-7 (as far back as I tested). The xstormy port only allows a subset of the

[Bug rtl-optimization/92264] [10 Regression] Compile time hog in 521.wrf_r with -Ofast -march=znver2 -g since r276318

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92264 --- Comment #46 from CVS Commits --- The master branch has been updated by Jeff Law : https://gcc.gnu.org/g:7f26e60c2600f0a676fc9370390ebf0a3c78f31c commit r10-7548-g7f26e60c2600f0a676fc9370390ebf0a3c78f31c Author: Jeff Law Date: Fri Apr 3

[PATCH] gcc/gcc.c: add outfiles spec

2020-04-03 Thread Rasmus Villemoes
Commit 0b7fb27b698d (Fix and document -r option) broke building of VxWorks modules, at least for our version of VxWorks. A VxWorks module is a relocatable ELF file - the final link is done when the file is linked in to the running kernel. So building with -r is necessary. However, the module also

[PATCH v4 5/5] libgomp/test: Remove a build sysroot fix regression

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Fix a problem with commit c8e759b4215b ("libgomp/test: Fix compilation for build sysroot") that caused a regression in some standalone test environments where testsuite/libgomp-test-support.exp is used, but the compiler is expected to be determined by `[find_gcc]', and set the GCC_UNDER_TEST

[PATCH v4 GCC 3/5] libffi/test: Make `libffi-init' use $CC_FOR_TARGET

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Update code in `libffi-init' to use $CC_FOR_TARGET in determining the value of $ld_library_path, as using a different compiler location from one actually used in testing may have odd consequences. As this obviously loses the setting of $gccdir provide a replacement way to determine the

[PATCH v4 4/5] libgo/test: Complement compilation fix for build sysroot

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Complement commit b72813a68c94 ("libgo: fix DejaGNU testsuite compiler when using build sysroot") and move testsuite/libgo-test-support.exp.in to testsuite/libgo-site-extra.exp.in. Update testsuite/lib/libgo.exp to handle the `--tool_exec' option to `runtest' as with other top-level GCC

[PATCH v4 GCC 2/5] libffi/test: Fix compilation for build sysroot

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Fix a problem with the libffi testsuite using a method to determine the compiler to use resulting in the tool being different from one the library has been built with, and causing a catastrophic failure from the inability to actually choose any compiler at all in a cross-compilation

[PATCH v4 1/5] libatomic/test: Fix compilation for build sysroot

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Fix a problem with the libatomic testsuite using a method to determine the compiler to use resulting in the tool being different from one the library has been built with, and causing a catastrophic failure from the lack of a suitable `--sysroot=' option where the `--with-build-sysroot='

[PATCH v4 0/5] Fix library testsuite compilation for build sysroot

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Hi, This is v4 of patch series, originally posted here: v2

Re: [PATCH] debug: Improve debug info of c++14 deduced return type [PR94459]

2020-04-03 Thread Hannes Domani via Gcc-patches
Am Samstag, 4. April 2020, 00:47:27 MESZ hat Jakub Jelinek via Gcc-patches Folgendes geschrieben: > Hi! > > On the following testcase, in gdb ptype S::m1 prints long as return > type, but all the other methods show void instead. > PR53756 added code to add_type_attribute if the return type is

[PATCH libffi 3/4] Make `libffi-init' use $CC_FOR_TARGET

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Update code in `libffi-init' to use $CC_FOR_TARGET in determining the value of $ld_library_path, as using a different compiler location from one actually used in testing may have odd consequences. As this obviously loses the setting of $gccdir provide a replacement way to determine the

[PATCH libffi 4/4] Correct indentation throughout `libffi-init'

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
--- testsuite/lib/libffi.exp |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) libffi-init-whitespace.diff Index: libffi/testsuite/lib/libffi.exp === --- libffi.orig/testsuite/lib/libffi.exp +++

[PATCH libffi 2/4] Use a documented way to pass $compiler_vendor to DejaGNU

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Use Autoconf substitution in the template used for extra DejaGNU site configuration, which is a documented supported way to pass information from the `configure' script, rather than resorting to a hack with extracting an undocumented internal setting from `config.log' to pass the compiler

[PATCH libffi 1/4] Use a template to pass $CC and $CXX to DejaGNU

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Use an Autoconf template rather an inline piece of scriptery to set DejaGNU's $CC_FOR_TARGET and $CXX_FOR_TARGET variables from $CC and $CXX respectively, making it easier to maintain and making it take advantage of Automake's dependency and rule generation. Relocate the generated `local.exp'

[PATCH libffi 0/4] Robustify compiler and library path selection in the testsuite

2020-04-03 Thread Maciej W. Rozycki via Gcc-patches
Hi, In the course of a discussion at one of the GCC mailing lists here: I have realised that some parts related to libffi testing have not been merged to GCC, and they are needed to choose the right compiler in a

[Bug c++/94477] ICE in tsubst_expr, at cp/pt.c:18181

2020-04-03 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94477 --- Comment #3 from Jakub Jelinek --- Fixed on the trunk so far.

[PATCH] debug: Improve debug info of c++14 deduced return type [PR94459]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
Hi! On the following testcase, in gdb ptype S::m1 prints long as return type, but all the other methods show void instead. PR53756 added code to add_type_attribute if the return type is auto/decltype(auto), but we actually should look through references, pointers and qualifiers. Haven't included

Re: [PATCH] cselib: Don't consider SP_DERIVED_VALUE_P values as useless [PR94468]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
On Fri, Apr 03, 2020 at 07:42:10PM +0200, Jakub Jelinek via Gcc-patches wrote: > Or, to avoid the repetitive code, should I introduce > static bool > cselib_useless_value_p (cselib_val *v) > { > return (v->locs == 0 > && !PRESERVED_VALUE_P (v->val_rtx) > && !SP_DERIVED_VALUE_P

[PATCH] i386: Simplify {,v}ph{add,sub{,s}{w,d} insn patterns [PR94460]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
Hi! As mentioned in the previous PR94460 patch, the RTL patterns look too large/complicated, we can simplify them by just performing two 2 arg permutations to move the arguments into the right spots and then just doing the plus/minus (or signed saturation version thereof). Bootstrapped/regtested

[Bug target/94482] New: Inserting into vector with optimization enabled on x86 generates incorrect result

2020-04-03 Thread e...@coeus-group.com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94482 Bug ID: 94482 Summary: Inserting into vector with optimization enabled on x86 generates incorrect result Product: gcc Version: 9.3.0 Status: UNCONFIRMED

[committed] openmp: Fix ICE on #pragma omp parallel master in template [PR94477]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase ICEs, because for parallel combined with some other construct we initialize the omp_parallel_combined_clauses pointer and expect the construct combined with it to clear it after it no longer needs it, but OMP_MASTER didn't do that. Bootstrapped/regtested on

[Bug c++/94477] ICE in tsubst_expr, at cp/pt.c:18181

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94477 --- Comment #2 from CVS Commits --- The master branch has been updated by Jakub Jelinek : https://gcc.gnu.org/g:0c809f727cd2a6c70c307d9dd53d26dc84bf292a commit r10-7547-g0c809f727cd2a6c70c307d9dd53d26dc84bf292a Author: Jakub Jelinek Date:

gcc-8-20200403 is now available

2020-04-03 Thread GCC Administrator via Gcc
Snapshot gcc-8-20200403 is now available on https://gcc.gnu.org/pub/gcc/snapshots/8-20200403/ and on various mirrors, see http://gcc.gnu.org/mirrors.html for details. This snapshot has been generated from the GCC 8 git branch with the following options: git://gcc.gnu.org/git/gcc.git branch

Re: [PATCH] Fix some incorrect Changelog dates

2020-04-03 Thread Jakub Jelinek via Gcc-patches
On Fri, Apr 03, 2020 at 03:53:47PM -0600, Zackery Spytz via Gcc-patches wrote: > Hello, > > It seems that some dates in gcc/ChangeLog-2019 and > gcc/testsuite/ChangeLog-2019 are incorrect. If we want to fix these, there are many more suspect cases. I guess we'd need to look at each of those,

[Bug libbacktrace/92916] elf_add freeing strtab_view in fail, even though it shouldn't

2020-04-03 Thread ian at airs dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92916 Ian Lance Taylor changed: What|Removed |Added CC||ian at airs dot com --- Comment #2

[Bug libbacktrace/92916] elf_add freeing strtab_view in fail, even though it shouldn't

2020-04-03 Thread marc.streckfuss at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92916 --- Comment #1 from Marc Streckfuß --- This is fixed in https://github.com/ianlancetaylor/libbacktrace, if the changes are also upstream, this bug can be closed.

libgcc patch committed: Avoid hooks in split-stack code

2020-04-03 Thread Ian Lance Taylor via Gcc-patches
The split-stack code invokes mmap and munmap with a limited amount of stack space. That is fine when the functions just make a system call, but it's not fine when programs use LD_PRELOAD or linker tricks to add hooks to mmap/munmap. Those hooks may use too much stack space, leading to an obscure

[PATCH] Fix some incorrect Changelog dates

2020-04-03 Thread Zackery Spytz via Gcc-patches
Hello, It seems that some dates in gcc/ChangeLog-2019 and gcc/testsuite/ChangeLog-2019 are incorrect. diff --git a/gcc/ChangeLog-2019 b/gcc/ChangeLog-2019 index 9f5782fd458..1acf57d49b9 100644 --- a/gcc/ChangeLog-2019 +++ b/gcc/ChangeLog-2019 @@ -1,10 +1,10 @@ -2020-12-31 Richard Sandiford

Re: [PATCH] rs6000: Don't split constant oprator when add, move to temp register for future optimization

2020-04-03 Thread Segher Boessenkool
On Fri, Apr 03, 2020 at 10:43:49PM +1030, Alan Modra wrote: > On Fri, Apr 03, 2020 at 02:13:06PM +0800, luoxhu via Gcc-patches wrote: > > Seems PR94393? Yes, rs6000_emit_set_const calls rs6000_emit_set_long_const > > for DImode. > > I tried unsigned long like 0xabcd87654321, 0xabcd87654321

Re: [PATCH] rs6000: Don't split constant oprator when add, move to temp register for future optimization

2020-04-03 Thread Segher Boessenkool
Hi! On Fri, Apr 03, 2020 at 02:13:06PM +0800, luoxhu wrote: > Currently 286r.split2 will split "12:%9:DI=0x87654321" to lis+ori by > rs6000_emit_set_const of define_split, do you mean add new > define_insn_and_split > to do the split? Another patch to do this after this one goes upstream in >

Re: [PATCH] c++: Fix constexpr evaluation of self-modifying CONSTRUCTORs [PR94219]

2020-04-03 Thread Patrick Palka via Gcc-patches
On Fri, 3 Apr 2020, Patrick Palka wrote: > On Fri, 3 Apr 2020, Jason Merrill wrote: > > > On 4/2/20 2:19 PM, Patrick Palka wrote: > > > On Thu, 2 Apr 2020, Patrick Palka wrote: > > > > > > > This PR reveals that cxx_eval_bare_aggregate and > > > > cxx_eval_store_expression > > > > do > > > >

Re: [PATCH] c++: Fix constexpr evaluation of self-modifying CONSTRUCTORs [PR94219]

2020-04-03 Thread Patrick Palka via Gcc-patches
On Fri, 3 Apr 2020, Jason Merrill wrote: > On 4/2/20 2:19 PM, Patrick Palka wrote: > > On Thu, 2 Apr 2020, Patrick Palka wrote: > > > > > This PR reveals that cxx_eval_bare_aggregate and cxx_eval_store_expression > > > do > > > not anticipate that a constructor element's initializer could mutate

[Bug c++/94481] New: ICE: Segmentation fault (in cp_parser_placeholder_type_specifier)

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94481 Bug ID: 94481 Summary: ICE: Segmentation fault (in cp_parser_placeholder_type_specifier) Product: gcc Version: 10.0 Status: UNCONFIRMED Keywords:

[Bug c++/94480] New: ICE in cp_parser_abort_tentative_parse, at cp/parser.c:31858

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94480 Bug ID: 94480 Summary: ICE in cp_parser_abort_tentative_parse, at cp/parser.c:31858 Product: gcc Version: 10.0 Status: UNCONFIRMED Keywords:

Re: subversion status on gcc.gnu.org

2020-04-03 Thread Frank Ch. Eigler via Gcc
Hi - > > > https://gcc.gnu.org/r2000 > > > maps to a search and you get thousands of hits for SVN: r2000** > > > > The gcc git svn-rev alias handles that (and also handles release and other > > branches) using approx. > > git log --all --grep="From-SVN: r$rev\b" | head -n 1 | awk '{print $2}' >

[PATCH v2] c++: Fix further protected_set_expr_location related -fcompare-debug issues [PR94441]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
On Fri, Apr 03, 2020 at 03:02:58PM -0400, Jason Merrill via Gcc-patches wrote: > > Or do we want a further wrapper, perhaps C++ FE only, that would do this > > protected_set_expr_location_if_unset (stmt, locus)? > > That sounds good to me. So like this if it passes bootstrap/regtest? 2020-04-03

[Bug tree-optimization/93435] [8/9 Regression] Hang with -O2 on innocuous looking code with GCC 8.3

2020-04-03 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93435 --- Comment #13 from Martin Jambor --- The problematic behavior of SRA is now fixed on master and both opened release branches so I consider my work done here. I'm leaving the bug opened in case Jeff wants to add some DSE limiter like he wrote

Re: [PATCH] c++: Refrain from using replace_placeholders in constexpr evaluation [PR94205]

2020-04-03 Thread Patrick Palka via Gcc-patches
On Fri, 3 Apr 2020, Jason Merrill wrote: > On 4/2/20 7:40 PM, Patrick Palka wrote: > > + /* Prefer the outermost matching object, but don't cross > > + CONSTRUCTOR_PLACEHOLDER_BOUNDARY constructors. */ > > + if (ctx->ctor && !CONSTRUCTOR_PLACEHOLDER_BOUNDARY (ctx->ctor)) > > +if (tree

[Bug c++/94453] [10 Regression] ICE in make_decl_rtl since r10-3591

2020-04-03 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94453 Jason Merrill changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |jason at gcc dot gnu.org

[Bug c++/91966] [8 Regression] pack expansion for Cartesian product breaks if certain indirections are involved since r6-1487-g1a4cd2cd7e7266b0

2020-04-03 Thread jason at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91966 Jason Merrill changed: What|Removed |Added Summary|[8/9 Regression] pack |[8 Regression] pack

[Bug tree-optimization/93435] [8/9 Regression] Hang with -O2 on innocuous looking code with GCC 8.3

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93435 --- Comment #12 from CVS Commits --- The releases/gcc-8 branch has been updated by Martin Jambor : https://gcc.gnu.org/g:b445ceec81ba3f4afad8c3ead1e58f14f1c2e146 commit r8-10163-gb445ceec81ba3f4afad8c3ead1e58f14f1c2e146 Author: Martin Jambor

[Bug c++/94477] ICE in tsubst_expr, at cp/pt.c:18181

2020-04-03 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94477 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

Re: [PATCH] c++: Fix constexpr evaluation of self-modifying CONSTRUCTORs [PR94219]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/2/20 2:19 PM, Patrick Palka wrote: On Thu, 2 Apr 2020, Patrick Palka wrote: This PR reveals that cxx_eval_bare_aggregate and cxx_eval_store_expression do not anticipate that a constructor element's initializer could mutate the underlying CONSTRUCTOR. Evaluation of the initializer could

[PATCH][gcc][PR94230]provide an option to change the size limitation for -Wmisleading-indent

2020-04-03 Thread Qing Zhao via Gcc-patches
Hi, David and Jakub, Per the discussion we had for PR94230: provide an option to change the size limitation for -Wmisleading-indent https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94230 I come up with the following simple patch per David’s

[Bug c++/94476] NSDMI deferred parse

2020-04-03 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94476 Nathan Sidwell changed: What|Removed |Added Priority|P3 |P4 Status|ASSIGNED

Re: Question about the testresults mailing list

2020-04-03 Thread Jonathan Wakely via Gcc
On Fri, 3 Apr 2020 at 19:40, Bernd Edlinger wrote: > > > > On 4/3/20 7:56 PM, Jonathan Wakely wrote: > > On Fri, 3 Apr 2020 at 18:55, Bernd Edlinger > > wrote: > >> > >> > >> > >> On 4/3/20 7:50 PM, Jonathan Wakely wrote: > >>> On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: > >

[Bug tree-optimization/91322] [10 regression] alias-4 test failure

2020-04-03 Thread wdijkstr at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91322 --- Comment #10 from Wilco --- (In reply to Christophe Lyon from comment #6) > Created attachment 48184 [details] > GCC passes dumps So according to that, in 105t.vrp1 it removes the branch and unconditionally calls abort: Folding statement:

[Bug c/94479] New: NetBSD: internal compiler error: in recompute_tree_invariant_for_addr_expr

2020-04-03 Thread tk at giga dot or.at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94479 Bug ID: 94479 Summary: NetBSD: internal compiler error: in recompute_tree_invariant_for_addr_expr Product: gcc Version: 9.3.0 Status: UNCONFIRMED Severity:

[Bug c++/91966] [8/9 Regression] pack expansion for Cartesian product breaks if certain indirections are involved since r6-1487-g1a4cd2cd7e7266b0

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91966 --- Comment #4 from CVS Commits --- The releases/gcc-9 branch has been updated by Jason Merrill : https://gcc.gnu.org/g:70ae61692245b5002f32454c8447655ad65a8535 commit r9-8450-g70ae61692245b5002f32454c8447655ad65a8535 Author: Jason Merrill

Re: [PATCH] c++: Refrain from using replace_placeholders in constexpr evaluation [PR94205]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/2/20 7:40 PM, Patrick Palka wrote: + /* Prefer the outermost matching object, but don't cross + CONSTRUCTOR_PLACEHOLDER_BOUNDARY constructors. */ + if (ctx->ctor && !CONSTRUCTOR_PLACEHOLDER_BOUNDARY (ctx->ctor)) +if (tree parent_ob = lookup_placeholder (ctx->parent, lval, type))

Re: [PATCH] c++: Fix further protected_set_expr_location related -fcompare-debug issues [PR94441]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 4/2/20 3:48 AM, Jakub Jelinek wrote: Hi! My recent protected_set_expr_location changes work well when that function is called unconditionally, but as the testcase shows, the C++ FE has a few spots that do: if (!EXPR_HAS_LOCATION (stmt)) protected_set_expr_location (stmt, locus); or

Re: [PATCH] c++: Fix crash in gimplifier with paren init of aggregates [PR94155]

2020-04-03 Thread Jason Merrill via Gcc-patches
On 3/30/20 4:28 PM, Marek Polacek wrote: Here we crash in the gimplifier because gimplify_init_ctor_eval doesn't expect null indexes for a constructor: /* ??? Here's to hoping the front end fills in all of the indices, so we don't have to figure out what's missing ourselves.

[Bug target/94467] [10 Regression] wrong code with -mavx and ssse3 builtins

2020-04-03 Thread hjl.tools at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94467 H.J. Lu changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/94467] [10 Regression] wrong code with -mavx and ssse3 builtins

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94467 --- Comment #7 from CVS Commits --- The master branch has been updated by H.J. Lu : https://gcc.gnu.org/g:bbcdf9bb3fd04adc59f41e4e1ff6293c84cbecc4 commit r10-7545-gbbcdf9bb3fd04adc59f41e4e1ff6293c84cbecc4 Author: H.J. Lu Date: Fri Apr 3

[committed] Fix m32r target bug exposed by Jakub's cselib.c changes

2020-04-03 Thread Jeff Law via Gcc-patches
va-arg-22.c started failing on the m32r target (-O1 only) after Jakub's recent cselib changes. The m32r port has this pattern: (define_insn "cpymemsi_internal" [(set (mem:BLK (match_operand:SI 0 "register_operand" "r")) ;; destination (mem:BLK (match_operand:SI 1 "register_operand"

[Bug rtl-optimization/92264] [10 Regression] Compile time hog in 521.wrf_r with -Ofast -march=znver2 -g since r276318

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92264 --- Comment #45 from CVS Commits --- The master branch has been updated by Jeff Law : https://gcc.gnu.org/g:b949f8e2acb49273b2f08ecaa3bc7128baaad850 commit r10-7544-gb949f8e2acb49273b2f08ecaa3bc7128baaad850 Author: Jeff Law Date: Fri Apr 3

Re: Question about the testresults mailing list

2020-04-03 Thread Bernd Edlinger
On 4/3/20 7:56 PM, Jonathan Wakely wrote: > On Fri, 3 Apr 2020 at 18:55, Bernd Edlinger wrote: >> >> >> >> On 4/3/20 7:50 PM, Jonathan Wakely wrote: >>> On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: Hi, I have a question about the gcc-testresults mailing list,

Re: Question about the testresults mailing list

2020-04-03 Thread Bernd Edlinger
On 4/3/20 7:57 PM, Jonathan Wakely wrote: > On Fri, 3 Apr 2020 at 18:54, Bernd Edlinger wrote: >> >> On 4/3/20 7:50 PM, Jonathan Wakely wrote: >>> On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: Hi, I have a question about the gcc-testresults mailing list, that is I

[Bug target/94461] [10 Regression] ICE: in extract_insn, at recog.c:2294 with __builtin_ia32_pmuludq() and -mno-sse2

2020-04-03 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94461 Jakub Jelinek changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/94461] [10 Regression] ICE: in extract_insn, at recog.c:2294 with __builtin_ia32_pmuludq() and -mno-sse2

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94461 --- Comment #15 from CVS Commits --- The master branch has been updated by Jakub Jelinek : https://gcc.gnu.org/g:a13d6ec867e750169af95649235a6681f410464a commit r10-7543-ga13d6ec867e750169af95649235a6681f410464a Author: Jakub Jelinek Date:

[Bug tree-optimization/93435] [8/9 Regression] Hang with -O2 on innocuous looking code with GCC 8.3

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93435 --- Comment #11 from CVS Commits --- The releases/gcc-9 branch has been updated by Martin Jambor : https://gcc.gnu.org/g:a1bb16994caed4dacf8c9ee1a33b177df140e9dc commit r9-8449-ga1bb16994caed4dacf8c9ee1a33b177df140e9dc Author: Martin Jambor

[Bug c++/68394] [concepts] segfault in valid code

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68394 Arseny Solokha changed: What|Removed |Added CC||asolokha at gmx dot com --- Comment #1

[Bug c++/94478] New: [10 Regression] ICE in comptypes, at cp/typeck.c:1486

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94478 Bug ID: 94478 Summary: [10 Regression] ICE in comptypes, at cp/typeck.c:1486 Product: gcc Version: 10.0 Status: UNCONFIRMED Keywords: ice-on-invalid-code Severity:

Re: [stage1][PATCH] Change semantics of -frecord-gcc-switches and add -frecord-gcc-switches-format.

2020-04-03 Thread Egeyar Bagcioglu via Gcc-patches
On 3/18/20 10:05 AM, Martin Liška wrote: On 3/17/20 7:43 PM, Egeyar Bagcioglu wrote: Hi Martin, I like the patch. It definitely serves our purposes at Oracle and provides another way to do what my previous patches did as well. 1) It keeps the backwards compatibility regarding

Re: [PATCH] Improve svn-rev to search for pattern at line beginning.

2020-04-03 Thread Joseph Myers
Thanks, this matches the commit hook (which only rejects From-SVN: at start of line, not elsewhere in a commit message). -- Joseph S. Myers jos...@codesourcery.com

[Bug other/93641] Wrong strncmp and strncasecmp size arguments

2020-04-03 Thread dominik.b.czarnota+bugzilla at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93641 --- Comment #8 from Dominik 'disconnect3d' Czarnota --- (In reply to Iain Sandoe from comment #7) > if the invoke.texi entry does not need a change, then this should now be > fixed, correct? It would be a good practice to fix the invoke.texi

Re: Question about the testresults mailing list

2020-04-03 Thread Jonathan Wakely via Gcc
On Fri, 3 Apr 2020 at 18:54, Bernd Edlinger wrote: > > On 4/3/20 7:50 PM, Jonathan Wakely wrote: > > On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: > >> > >> Hi, > >> > >> I have a question about the gcc-testresults mailing list, > >> that is I see everyone using: > >> > >> [releases/gcc-9

Re: Question about the testresults mailing list

2020-04-03 Thread Jonathan Wakely via Gcc
On Fri, 3 Apr 2020 at 18:55, Bernd Edlinger wrote: > > > > On 4/3/20 7:50 PM, Jonathan Wakely wrote: > > On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: > >> > >> Hi, > >> > >> I have a question about the gcc-testresults mailing list, > >> that is I see everyone using: > >> > >>

Re: Question about the testresults mailing list

2020-04-03 Thread Bernd Edlinger
On 4/3/20 7:50 PM, Jonathan Wakely wrote: > On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: >> >> Hi, >> >> I have a question about the gcc-testresults mailing list, >> that is I see everyone using: >> >> [releases/gcc-9 revision >>

Re: Question about the testresults mailing list

2020-04-03 Thread Bernd Edlinger
On 4/3/20 7:50 PM, Jonathan Wakely wrote: > On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: >> >> Hi, >> >> I have a question about the gcc-testresults mailing list, >> that is I see everyone using: >> >> [releases/gcc-9 revision >> 02a201f71:0f58d3b54:0e66150084aa217811a5c45fb15e98d7ed3e8839]

Re: [PATCH] i386: Fix up handling of OPTION_MASK_ISA_MMX builtins [PR94461]

2020-04-03 Thread Uros Bizjak via Gcc-patches
On Fri, Apr 3, 2020 at 7:30 PM Jakub Jelinek wrote: > > Hi! > > In https://gcc.gnu.org/ml/gcc-patches/2017-10/msg00576.html the builtin > handling was changed so that OPTION_MASK_ISA_MMX | OPTION_MASK_ISA_SSE > etc. in i386-builtin.def means we require both mmx and sse, not just one of > those,

Re: Question about the testresults mailing list

2020-04-03 Thread Jonathan Wakely via Gcc
On Fri, 3 Apr 2020 at 18:45, Bernd Edlinger wrote: > > Hi, > > I have a question about the gcc-testresults mailing list, > that is I see everyone using: > > [releases/gcc-9 revision > 02a201f71:0f58d3b54:0e66150084aa217811a5c45fb15e98d7ed3e8839] > > or > > [master revision >

[pushed] c++: alias template and parameter packs (PR91966).

2020-04-03 Thread Jason Merrill via Gcc-patches
In this testcase, when we do a pack expansion of count_better_mins, nums appears both in the definition of count_better_mins and as its template argument. The intent is that we get a expansion over pairs of elements of the pack, i.e. less<2,2>, less<2,7>, less<7,2>, But if we substitute

[Bug c++/91966] [8/9/10 Regression] pack expansion for Cartesian product breaks if certain indirections are involved since r6-1487-g1a4cd2cd7e7266b0

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91966 --- Comment #3 from CVS Commits --- The master branch has been updated by Jason Merrill : https://gcc.gnu.org/g:bcafd8748cf9eb83a9e71bf67c4e2a7c5c8f513c commit r10-7542-gbcafd8748cf9eb83a9e71bf67c4e2a7c5c8f513c Author: Jason Merrill Date:

[Bug target/94460] wrong code with __builtin_ia32_phsubw256() or __builtin_ia32_phsubd256()

2020-04-03 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94460 --- Comment #5 from Jakub Jelinek --- Fixed on the trunk so far.

[Bug target/94460] wrong code with __builtin_ia32_phsubw256() or __builtin_ia32_phsubd256()

2020-04-03 Thread cvs-commit at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94460 --- Comment #4 from CVS Commits --- The master branch has been updated by Jakub Jelinek : https://gcc.gnu.org/g:b8020a5aafd02af1ccf99372d3d8052c40b59725 commit r10-7541-gb8020a5aafd02af1ccf99372d3d8052c40b59725 Author: Jakub Jelinek Date:

Question about the testresults mailing list

2020-04-03 Thread Bernd Edlinger
Hi, I have a question about the gcc-testresults mailing list, that is I see everyone using: [releases/gcc-9 revision 02a201f71:0f58d3b54:0e66150084aa217811a5c45fb15e98d7ed3e8839] or [master revision 63b2923dc6f:0c89e976db9:1c16f7fc903c1c1c912faf7889b69d83429b7b2e what is the first 2 hashes,

[PATCH] cselib: Don't consider SP_DERIVED_VALUE_P values as useless [PR94468]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase ICEs, because at one point we see the SP_DERIVED_VALUE_P VALUE as useless (not PRESERVED_VALUE_P and no locs) and so expect it to be discarded as useless. But, later on we are adding some new VALUE that is equivalent to it, and when adding the equivalency that that new

[Bug c++/94477] New: ICE in tsubst_expr, at cp/pt.c:18181

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94477 Bug ID: 94477 Summary: ICE in tsubst_expr, at cp/pt.c:18181 Product: gcc Version: 10.0 Status: UNCONFIRMED Keywords: ice-on-valid-code, openmp Severity: normal

[Bug c++/94476] NSDMI deferred parse

2020-04-03 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94476 Nathan Sidwell changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug c++/94476] New: NSDMI deferred parse

2020-04-03 Thread nathan at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94476 Bug ID: 94476 Summary: NSDMI deferred parse Product: gcc Version: 10.0 Status: UNCONFIRMED Severity: normal Priority: P3 Component: c++ Assignee:

[PATCH] i386: Fix up handling of OPTION_MASK_ISA_MMX builtins [PR94461]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
Hi! In https://gcc.gnu.org/ml/gcc-patches/2017-10/msg00576.html the builtin handling was changed so that OPTION_MASK_ISA_MMX | OPTION_MASK_ISA_SSE etc. in i386-builtin.def means we require both mmx and sse, not just one of those, and later on for other option combinations very similar rule has

Re: [PATCH] i386: Fix vph{add, subs?}[wd] 256-bit AVX2 RTL patterns [PR94460]

2020-04-03 Thread Uros Bizjak via Gcc-patches
On Fri, Apr 3, 2020 at 7:06 PM Jakub Jelinek wrote: > > Hi! > > The following testcase is miscompiled, because the AVX2 patterns don't > describe correctly what the insn does. E.g. vphaddd with %ymm* operands > (the second pattern) instruction as per: >

[Bug c++/94475] [9/10 Regression] ICE: tree check: expected class 'type', have 'exceptional' (error_mark) in element_mode, at tree.c:13813

2020-04-03 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94475 Marek Polacek changed: What|Removed |Added Summary|[10 Regression] ICE: tree |[9/10 Regression] ICE: tree

[Bug c++/94475] [10 Regression] ICE: tree check: expected class 'type', have 'exceptional' (error_mark) in element_mode, at tree.c:13813

2020-04-03 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94475 Marek Polacek changed: What|Removed |Added Status|UNCONFIRMED |NEW CC|

[Bug tree-optimization/90356] Missed optimization for variables initialized to 0.0

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90356 Arseny Solokha changed: What|Removed |Added CC||asolokha at gmx dot com --- Comment #8

[Bug c++/94475] New: [10 Regression] ICE: tree check: expected class 'type', have 'exceptional' (error_mark) in element_mode, at tree.c:13813

2020-04-03 Thread asolokha at gmx dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94475 Bug ID: 94475 Summary: [10 Regression] ICE: tree check: expected class 'type', have 'exceptional' (error_mark) in element_mode, at tree.c:13813 Product: gcc

[PATCH] i386: Fix vph{add,subs?}[wd] 256-bit AVX2 RTL patterns [PR94460]

2020-04-03 Thread Jakub Jelinek via Gcc-patches
Hi! The following testcase is miscompiled, because the AVX2 patterns don't describe correctly what the insn does. E.g. vphaddd with %ymm* operands (the second pattern) instruction as per: https://software.intel.com/sites/landingpage/IntrinsicsGuide/#text=_mm256_hadd_epi32=2941 does { a0+a1,

[Bug ipa/93621] [10 Regression] ICE in redirect_call_stmt_to_callee, at cgraph.c:1443 since r10-5567

2020-04-03 Thread jamborm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93621 --- Comment #16 from Martin Jambor --- The following workaround works for the testcase but would need to be generalized for a chain of former_decl_of's to be universal, I'm afraid: diff --git a/gcc/cgraph.c b/gcc/cgraph.c index

Re: [PATCH] x86: Mark scratch operand in ssse3_pshufbv8qi3 as earlyclobber

2020-04-03 Thread Uros Bizjak via Gcc-patches
On Fri, Apr 3, 2020 at 6:51 PM H.J. Lu wrote: > > commit 16ed2601ad0a4aa82f11e9df86ea92183f94f979 > Author: H.J. Lu > Date: Wed May 15 15:26:19 2019 + > > i386: Emulate MMX pshufb with SSE version > > has > > +(define_insn_and_split "ssse3_pshufbv8qi3" > + [(set (match_operand:V8QI 0

[PATCH] x86: Mark scratch operand in ssse3_pshufbv8qi3 as earlyclobber

2020-04-03 Thread H.J. Lu via Gcc-patches
commit 16ed2601ad0a4aa82f11e9df86ea92183f94f979 Author: H.J. Lu Date: Wed May 15 15:26:19 2019 + i386: Emulate MMX pshufb with SSE version has +(define_insn_and_split "ssse3_pshufbv8qi3" + [(set (match_operand:V8QI 0 "register_operand" "=y,x,Yv") + (unspec:V8QI [(match_operand:V8QI

  1   2   3   >