[Bug jit/88747] [9 Regression] jit testsuite failures: test-sum-of-squares.c.exe (and test-combination)

2019-01-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88747 --- Comment #2 from David Malcolm --- Author: dmalcolm Date: Tue Jan 8 01:39:09 2019 New Revision: 267671 URL: https://gcc.gnu.org/viewcvs?rev=267671&root=gcc&view=rev Log: Fix jit test case (PR jit/88747) Amongst other changes, r266077 update

[Bug jit/88747] [9 Regression] jit testsuite failures: test-sum-of-squares.c.exe (and test-combination)

2019-01-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88747 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c/87488] hyperlink filenames in diagnostics

2019-01-08 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87488 --- Comment #7 from David Malcolm --- I'm unconvinced that doing it for filenames is a good idea (based on the objections in comment #3), but I think that there could be other good uses tagging URLs into the output. For example, a static analysi

[Bug tree-optimization/88763] Better Output for Loop Unswitching

2019-01-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88763 --- Comment #4 from David Malcolm --- (In reply to Marius Messerschmidt from comment #3) > Sorry but I do not fully understand what you mean. Do you suggest using > different command line arguments? I believe Richard is referring to the internal

[Bug tree-optimization/88763] Better Output for Loop Unswitching

2019-01-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88763 --- Comment #5 from David Malcolm --- Marius: do you have a simple testcase which demonstrates an area where the log could be improved? [I'm testing a patch right now which ports things to the dump_* API, and thus should make the existing dump m

[Bug c++/88779] No fix-it hints for misspelled member initializers

2019-01-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88779 David Malcolm changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug c++/88779] New: No fix-it hints for misspelled member initializers

2019-01-09 Thread dmalcolm at gcc dot gnu.org
Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- Consider: class test

[Bug tree-optimization/88763] Better Output for Loop Unswitching

2019-01-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88763 --- Comment #6 from David Malcolm --- Candidate patch for porting to the dump_* API: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg00512.html

[Bug tree-optimization/88763] Better Output for Loop Unswitching

2019-01-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88763 --- Comment #10 from David Malcolm --- Created attachment 45406 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45406&action=edit Followup patch to try to dump why a condition can't be unswitched within a loop

[Bug tree-optimization/88763] Better Output for Loop Unswitching

2019-01-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88763 --- Comment #11 from David Malcolm --- Thanks for the testcase. (In reply to David Malcolm from comment #10) > Created attachment 45406 [details] > Followup patch to try to dump why a condition can't be unswitched within a > loop This is a foll

[Bug c++/88699] [9 Regression] tree check fail: expected function_decl, have using_decl in add_method, at cp/class.c:1137

2019-01-11 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88699 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug c++/88699] [9 Regression] tree check fail: expected function_decl, have using_decl in add_method, at cp/class.c:1137

2019-01-11 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88699 --- Comment #6 from David Malcolm --- (In reply to David Malcolm from comment #5) [...] > This patch stops the crash for comment #4 (though maybe it's papering over > the problem): [...] It also fixes the ICE from the original reproducer.

[Bug rtl-optimization/88423] [9 Regression] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-11 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88423 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug c++/88294] [9 Regression] ICE on (invalid) C++11 code: in tsubst_copy, at cp/pt.c:15391

2019-01-11 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88294 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug fortran/88810] gcc/fortran/dependency.c:2200: possible cut'n'paste error ?

2019-01-14 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88810 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug c++/88795] ICE on class-template argument deduction if non-type parameter has indirection

2019-01-15 Thread dmalcolm at gcc dot gnu.org
||dmalcolm at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #2 from David Malcolm --- ICE occurs at gcc/tree.c:6811 here: 6809case FUNCTION_TYPE: 6810 for (tree t = TYPE_ARG_TYPES (type); t; t

[Bug c++/88795] ICE on class-template argument deduction if non-type parameter has indirection

2019-01-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88795 --- Comment #3 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg00865.html

[Bug target/88861] [9 Regression] ICE in calc_dfs_tree, at dominance.c:458

2019-01-15 Thread dmalcolm at gcc dot gnu.org
||2019-01-15 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #2 from David Malcolm --- Confirmed (with target==ppc64le-redhat-linux)

[Bug target/88861] [9 Regression] ICE in calc_dfs_tree, at dominance.c:458

2019-01-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88861 --- Comment #3 from David Malcolm --- Assertion fails in dom_info::calc_dfs_tree: 457 /* This aborts e.g. when there is _no_ path from ENTRY to EXIT at all. */ 458 gcc_assert (m_nodes == (unsigned int) m_n_basic_blocks - 1); (gdb)

[Bug c++/88795] ICE on class-template argument deduction if non-type parameter has indirection

2019-01-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88795 --- Comment #4 from David Malcolm --- Author: dmalcolm Date: Tue Jan 15 23:29:15 2019 New Revision: 267957 URL: https://gcc.gnu.org/viewcvs?rev=267957&root=gcc&view=rev Log: Fix ICE on class-template argument deduction (PR c++/88795) PR c++/887

[Bug c++/88795] ICE on class-template argument deduction if non-type parameter has indirection

2019-01-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88795 --- Comment #5 from David Malcolm --- Should be fixed on trunk by r267957.

[Bug target/88861] [9 Regression] ICE in calc_dfs_tree, at dominance.c:458

2019-01-15 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #4 from David Malcolm --- Am testing a fix.

[Bug target/88861] [9 Regression] ICE in calc_dfs_tree, at dominance.c:458

2019-01-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88861 --- Comment #6 from David Malcolm --- (In reply to Segher Boessenkool from comment #5) > Cool, thanks! Is the plan to simply not allow something that can throw to be > recognised as noop move? Candidate patch: https://gcc.gnu.org/ml/gcc-patches

[Bug rtl-optimization/88879] [9 Regression] ICE in sel_target_adjust_priority, at sel-sched.c:3332

2019-01-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88879 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug target/88861] [9 Regression] ICE in calc_dfs_tree, at dominance.c:458

2019-01-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88861 --- Comment #7 from David Malcolm --- Author: dmalcolm Date: Wed Jan 16 20:13:23 2019 New Revision: 267984 URL: https://gcc.gnu.org/viewcvs?rev=267984&root=gcc&view=rev Log: Fix ICE due to "combine" creating unreachable EH blocks (PR target/8886

[Bug target/88861] [9 Regression] ICE in calc_dfs_tree, at dominance.c:458

2019-01-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88861 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/88699] [9 Regression] tree check fail: expected function_decl, have using_decl in add_method, at cp/class.c:1137

2019-01-16 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug rtl-optimization/88423] [9 Regression] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88423 --- Comment #4 from David Malcolm --- The code_label is for a tablejump_p, immediately before the jump_table_data.

[Bug c++/88699] [9 Regression] tree check fail: expected function_decl, have using_decl in add_method, at cp/class.c:1137

2019-01-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88699 --- Comment #7 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg00962.html

[Bug c++/88699] [9 Regression] tree check fail: expected function_decl, have using_decl in add_method, at cp/class.c:1137

2019-01-17 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88699 --- Comment #8 from David Malcolm --- Author: dmalcolm Date: Thu Jan 17 17:07:20 2019 New Revision: 268041 URL: https://gcc.gnu.org/viewcvs?rev=268041&root=gcc&view=rev Log: C++: Fix ICE when adding overloaded operator via using_decl (PR c++/886

[Bug c++/88699] [9 Regression] tree check fail: expected function_decl, have using_decl in add_method, at cp/class.c:1137

2019-01-17 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88699 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug rtl-optimization/88423] [9 Regression] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-18 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #5 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg01084.html

[Bug driver/88911] No "did you mean" for incorrect -dumpspecs option

2019-01-18 Thread dmalcolm at gcc dot gnu.org
||2019-01-18 CC||dmalcolm at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #2 from David Malcolm --- Confirmed; am

[Bug driver/88911] No "did you mean" for incorrect -dumpspecs option

2019-01-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88911 --- Comment #3 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg01311.html

[Bug rtl-optimization/88423] [9 Regression] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88423 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug rtl-optimization/88347] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88347 --- Comment #2 from David Malcolm --- *** Bug 88423 has been marked as a duplicate of this bug. ***

[Bug rtl-optimization/88347] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-22 Thread dmalcolm at gcc dot gnu.org
||dmalcolm at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #3 from David Malcolm --- Segher's fix appears superior to mine; am testing it now...

[Bug rtl-optimization/88347] ICE in begin_move_insn, at sched-ebb.c:175

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88347 David Malcolm changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|dmalcolm at gcc

[Bug driver/89014] New: Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- Host: aarch64 Target: aarch64 This downstream report: https://bugzilla.redhat.com/show_bug.cgi?id=1668631 describes a problem with -march=native on

[Bug driver/89014] Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89014 --- Comment #1 from David Malcolm --- Reproducer: $ valgrind ./xgcc -B. -c test.c -march=native -v (with a dummy test.c; see with today's trunk e.g. r268186)

[Bug driver/89014] Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89014 --- Comment #3 from David Malcolm --- The obvious fix: diff --git a/gcc/config/aarch64/driver-aarch64.c b/gcc/config/aarch64/driver-aarch64.c index 2bf1f9a8c13..100e0c3529c 100644 --- a/gcc/config/aarch64/driver-aarch64.c +++ b/gcc/config/aarch6

[Bug driver/89014] Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89014 --- Comment #5 from David Malcolm --- (In reply to ktkachov from comment #4) > Yeah, the fix looks right. > For the rest of the assembler errors we need the patch at: > https://gcc.gnu.org/ml/gcc-patches/2019-01/msg00562.html Thanks; Tamar's pat

[Bug driver/89014] Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89014 --- Comment #6 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg01381.html

[Bug driver/89014] Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89014 --- Comment #7 from David Malcolm --- Author: dmalcolm Date: Wed Jan 23 16:36:46 2019 New Revision: 268189 URL: https://gcc.gnu.org/viewcvs?rev=268189&root=gcc&view=rev Log: aarch64: fix use-after-free in -march=native (PR driver/89014) Running

[Bug driver/89014] Use-after-free in aarch64 -march=native

2019-01-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89014 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/89036] ICE if destructor has a requires

2019-01-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89036 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug c++/89036] [8/9 Regression] ICE if destructor has a requires

2019-01-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89036 David Malcolm changed: What|Removed |Added Keywords||ice-on-valid-code Status|UNC

[Bug c++/89036] [8/9 Regression] ICE if destructor has a requires

2019-01-24 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug c/89045] [9 Regression] ICE in get_parm_info, at c/c-decl.c:7518

2019-01-24 Thread dmalcolm at gcc dot gnu.org
||2019-01-25 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed. It's unexpectedly reaching this case in get_parm_decl with a VAR_DECL:

[Bug c++/89036] [8/9 Regression] ICE if destructor has a requires

2019-01-25 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89036 --- Comment #3 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg01513.html

[Bug c++/89055] wrong location with predefined macros

2019-01-25 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89055 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug middle-end/89091] ICE: Segmentation fault (in tree_class_check)

2019-01-28 Thread dmalcolm at gcc dot gnu.org
||2019-01-28 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed, at -O1 and above. Segfaults reading through NULL "unsigned_type" here in decode_field

[Bug middle-end/89091] ICE: Segmentation fault (in tree_class_check)

2019-01-28 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug middle-end/89099] Have "-fopt-info" show the original source code context

2019-01-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89099 --- Comment #1 from David Malcolm --- Indeed: -fopt-info is currently implemented via writing to up to two FILE * destinations: the dumpfile and the opt-info destination (e.g. stderr). In particular it doesn't go through the diagnostic subsystem

[Bug middle-end/89099] Have "-fopt-info" show the original source code context

2019-01-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89099 --- Comment #2 from David Malcolm --- (In reply to David Malcolm from comment #1) > (a) unify the two, so that -fopt-info messages ..."go through the diagnostics subsystem", I meant to write.

[Bug c/89122] bad fix-it hint for FLT_MAX when is included

2019-01-30 Thread dmalcolm at gcc dot gnu.org
||2019-01-30 CC||dmalcolm at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed. It&#

[Bug c/89122] bad fix-it hint for FLT_MAX when is included

2019-01-30 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89122 --- Comment #2 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-01/msg01756.html

[Bug c/89122] bad fix-it hint for FLT_MAX when is included

2019-01-31 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89122 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c/89122] bad fix-it hint for FLT_MAX when is included

2019-01-31 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89122 --- Comment #3 from David Malcolm --- Author: dmalcolm Date: Thu Jan 31 18:09:29 2019 New Revision: 268426 URL: https://gcc.gnu.org/viewcvs?rev=268426&root=gcc&view=rev Log: Fix bogus fix-it for FLT_MAX (PR c/89122) PR c/89122 reports that we e

[Bug tree-optimization/89134] A missing optimization opportunity for a simple branch in loop

2019-01-31 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89134 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug lto/89084] [9 Regression] ICE in get_partitioning_class, at symtab.c:1892

2019-01-31 Thread dmalcolm at gcc dot gnu.org
||2019-01-31 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed (on godbolt with x86_64 trunk, and a regression relative to gcc 8.2)

[Bug lto/89084] [9 Regression] ICE in get_partitioning_class, at symtab.c:1892

2019-01-31 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89084 --- Comment #2 from David Malcolm --- Fails this assertion: 1892 gcc_checking_assert (vnode->definition); (gdb) p vnode $3 =

[Bug c++/88983] ICE in label_matches, at cp/constexpr.c:4035

2019-01-31 Thread dmalcolm at gcc dot gnu.org
||2019-01-31 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed, *jump_target is a RETURN_EXPR here: 4000 switch (TREE_CODE (*jump_target)) 4001

[Bug ipa/89139] GCC emits code for static functions that aren't used by the optimized code

2019-02-01 Thread dmalcolm at gcc dot gnu.org
||2019-02-01 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #2 from David Malcolm --- Indeed, the last elimination is happening too late. At -O3: .040t.tailr1 converts the tail-recursion in &q

[Bug ipa/89139] GCC emits code for static functions that aren't used by the optimized code

2019-02-01 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89139 --- Comment #3 from David Malcolm --- (In reply to David Malcolm from comment #2) > happens when we're already running cgraph_node::expand on "h" ~~ "when we've already run", I meant to say

[Bug c++/70693] valgrind error in get_visual_column

2019-02-01 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70693 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug jit/63854] Fix memory leaks seen in JIT

2019-02-01 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63854 --- Comment #32 from David Malcolm --- Using: RUN_UNDER_VALGRIND= \ make check-jit \ RUNTESTFLAGS="-v -v -v jit.exp=test-factorial.c" trunk currently shows this: LEAK SUMMARY: definitely lost: 55,908 bytes in 654 blocks indirectly

[Bug lto/88147] [9 Regression] ICE in linemap_line_start, at libcpp/line-map.c:781 starting from r265875

2019-02-02 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88147 --- Comment #10 from David Malcolm --- FWIW I'm able to reproduce this with r265875 and am running a reduction script over this weekend to see if I can isolate what the issue is/was.

[Bug c++/88937] valgrind error in parse_has_include

2019-02-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88937 --- Comment #4 from David Malcolm --- It's clearly wrong to access token->val.node for token->type == CPP_STRING and token->type == CPP_HEADER_NAME. It's effectively casting the length of the header name to a (cpp_hashnode *). For reference, th

[Bug c++/88937] valgrind error in parse_has_include

2019-02-04 Thread dmalcolm at gcc dot gnu.org
dot gnu.org |dmalcolm at gcc dot gnu.org Target Milestone|9.0 |10.0 --- Comment #5 from David Malcolm --- Patch: https://gcc.gnu.org/ml/gcc-patches/2019-02/msg00222.html

[Bug lto/88147] [9 Regression] ICE in linemap_line_start, at libcpp/line-map.c:781 starting from r265875

2019-02-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88147 --- Comment #11 from David Malcolm --- Created attachment 45610 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=45610&action=edit Partially reduced testcase This is 975366 bytes (decompressed) and takes 0.3-0.4 seconds to crash r265875's lt

[Bug c++/71302] [9 Regression] -Wzero-as-null-pointer-constant: misleading caret location for pointer in function call

2019-02-06 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Target Milestone|--- |9.0 Summary|-Wzero-as-null-pointer-cons |[9 Regression] |tant: misleading caret |-Wzero-as-null-pointer-cons |location for pointer in

[Bug tree-optimization/89223] internal compiler error: in int_cst_value, at tree.c:11226

2019-02-06 Thread dmalcolm at gcc dot gnu.org
|UNCONFIRMED |NEW Last reconfirmed||2019-02-06 CC||dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed via godbolt; this crashes GCC 5

[Bug c++/71302] [9 Regression] -Wzero-as-null-pointer-constant: misleading caret location for pointer in function call

2019-02-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71302 --- Comment #4 from David Malcolm --- Author: dmalcolm Date: Wed Feb 6 19:44:52 2019 New Revision: 268589 URL: https://gcc.gnu.org/viewcvs?rev=268589&root=gcc&view=rev Log: Fix locations in conversion_null_warnings (PR c++/71302) PR c++/71302

[Bug c++/71302] [9 Regression] -Wzero-as-null-pointer-constant: misleading caret location for pointer in function call

2019-02-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71302 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/88680] [9 Regression] bogus -Wtype-limits for constant expressions after r267272

2019-02-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88680 --- Comment #2 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-02/msg00363.html

[Bug tree-optimization/89235] [9 Regression] ICE: tree check: expected block, have in inlining_chain_to_json, at optinfo-emit-json.cc:285

2019-02-07 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #2 from David Malcolm --- I needed "-g" in addition to the options in comment #0 to trigger it. Am investigating.

[Bug tree-optimization/89235] [9 Regression] ICE: tree check: expected block, have in inlining_chain_to_json, at optinfo-emit-json.cc:285

2019-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89235 --- Comment #3 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2019-02/msg00402.html

[Bug tree-optimization/89235] [9 Regression] ICE: tree check: expected block, have in inlining_chain_to_json, at optinfo-emit-json.cc:285

2019-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89235 --- Comment #4 from David Malcolm --- Author: dmalcolm Date: Thu Feb 7 23:00:18 2019 New Revision: 268659 URL: https://gcc.gnu.org/viewcvs?rev=268659&root=gcc&view=rev Log: Fix more ICEs in -fsave-optimization-record (PR tree-optimization/89235

[Bug tree-optimization/86637] [9 Regression] ICE: tree check: expected block, have in inlining_chain_to_json, at optinfo-emit-json.cc:293

2019-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86637 --- Comment #13 from David Malcolm --- Author: dmalcolm Date: Thu Feb 7 23:00:18 2019 New Revision: 268659 URL: https://gcc.gnu.org/viewcvs?rev=268659&root=gcc&view=rev Log: Fix more ICEs in -fsave-optimization-record (PR tree-optimization/8923

[Bug tree-optimization/89235] [9 Regression] ICE: tree check: expected block, have in inlining_chain_to_json, at optinfo-emit-json.cc:285

2019-02-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89235 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/85110] Missing underlines for some bad arguments

2018-04-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85110 --- Comment #5 from David Malcolm --- Author: dmalcolm Date: Tue Apr 10 14:37:09 2018 New Revision: 259282 URL: https://gcc.gnu.org/viewcvs?rev=259282&root=gcc&view=rev Log: Show pertinent parameter (PR c++/85110) gcc/cp/ChangeLog: PR c

[Bug c++/85110] Missing underlines for some bad arguments

2018-04-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85110 David Malcolm changed: What|Removed |Added Status|WAITING |RESOLVED Resolution|---

[Bug jit/85384] New: libgccjit does not work if --with-gcc-major-version is used

2018-04-12 Thread dmalcolm at gcc dot gnu.org
Priority: P3 Component: jit Assignee: dmalcolm at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- As noted in this downstream bug: https://bugzilla.redhat.com/show_bug.cgi?id=1566178 libgccjit does not honor --with-gcc-major-version

[Bug jit/85384] libgccjit does not work if --with-gcc-major-version is used

2018-04-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85384 David Malcolm changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug c++/85385] New: [8 Regression] Bogus "macro had not yet been defined" message when macro used with wrong arg count

2018-04-12 Thread dmalcolm at gcc dot gnu.org
NCONFIRMED Keywords: diagnostic Severity: normal Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- $ cat test.c #define MACRO(X,Y) void test () { MACRO(42);

[Bug jit/85384] libgccjit does not work if --with-gcc-major-version is used

2018-04-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85384 --- Comment #2 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2018-04/msg00638.html

[Bug c++/85385] [8 Regression] Bogus "macro had not yet been defined" message when macro used with wrong arg count

2018-04-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85385 --- Comment #1 from David Malcolm --- Author: dmalcolm Date: Thu Apr 12 23:44:09 2018 New Revision: 259360 URL: https://gcc.gnu.org/viewcvs?rev=259360&root=gcc&view=rev Log: Prevent erroneous "macro had not yet been defined" messages (PR c++/853

[Bug c++/85385] [8 Regression] Bogus "macro had not yet been defined" message when macro used with wrong arg count

2018-04-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85385 David Malcolm changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug c++/85515] New: Bogus suggestions from "GCC's leaky abstractions"

2018-04-24 Thread dmalcolm at gcc dot gnu.org
everity: normal Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- Jason Turner's video C++ Weekly - Ep 112 - GCC's Leaky Abstractions https://www.youtube.com/watch?v=S9_mYmvO4Ow

[Bug c++/85515] Bogus suggestions from "GCC's leaky abstractions"

2018-04-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85515 David Malcolm changed: What|Removed |Added CC||barry.revzin at gmail dot com --- Commen

[Bug c++/85502] lambda capture is nameable - leaky abstraction

2018-04-24 Thread dmalcolm at gcc dot gnu.org
||dmalcolm at gcc dot gnu.org Resolution|--- |DUPLICATE --- Comment #1 from David Malcolm --- Thanks; looks like we both filed a bug about the issue in that video. Marking this as a dup of 85515 as that latter one also has the for-loop case

[Bug c++/85515] Bogus suggestions from "GCC's leaky abstractions"

2018-04-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85515 --- Comment #3 from David Malcolm --- Some examples of using the leaky abstractions, adapted from the video: int main () { auto lambda = [val = 2](){ return val; }; //lambda.__val = 4; return lambda(); } On uncommenting the assignment,

[Bug other/84889] Ideas on revamping how we format diagnostics

2018-04-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84889 --- Comment #8 from David Malcolm --- But what about parallel builds, where the errors can get interleaved?

[Bug c++/85515] Bogus suggestions from "GCC's leaky abstractions"

2018-04-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85515 --- Comment #4 from David Malcolm --- Also affects gcc 7 and gcc 6 branches.

[Bug c++/85523] Add fix-it hint for missing return statement in assignment operators

2018-04-25 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85523 --- Comment #3 from David Malcolm --- Note to self: test coverage should also verify += and so on.

[Bug c++/85523] Add fix-it hint for missing return statement in assignment operators

2018-04-25 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85523 --- Comment #4 from David Malcolm --- Note to self: this came out of this ML thread: https://gcc.gnu.org/ml/gcc/2018-04/msg00168.html

[Bug c++/85515] Bogus suggestions from "GCC's leaky abstractions"

2018-04-27 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85515 --- Comment #5 from David Malcolm --- Author: dmalcolm Date: Fri Apr 27 18:39:18 2018 New Revision: 259720 URL: https://gcc.gnu.org/viewcvs?rev=259720&root=gcc&view=rev Log: Don't offer suggestions for compiler-generated variables (PR c++/85515)

[Bug c/81405] [8 Regression] Buffer overflow when consolidating printing of out-of-order fix-it hints

2018-04-30 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81405 --- Comment #5 from David Malcolm --- Author: dmalcolm Date: Mon Apr 30 15:01:56 2018 New Revision: 259768 URL: https://gcc.gnu.org/viewcvs?rev=259768&root=gcc&view=rev Log: Use char_span for return type of location_get_source_line location_get

[Bug c++/85523] Add fix-it hint for missing return statement in assignment operators

2018-04-30 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85523 --- Comment #5 from David Malcolm --- Author: dmalcolm Date: Tue May 1 00:10:10 2018 New Revision: 259783 URL: https://gcc.gnu.org/viewcvs?rev=259783&root=gcc&view=rev Log: Add gcc_rich_location::add_fixit_insert_formatted This patch adds a su

<    1   2   3   4   5   6   7   8   9   10   >