[Bug c++/84269] More suggestions for missing #include

2018-03-17 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #5 from David Malcolm --- (In reply to David Malcolm from comment #4) > And another, as reported at > https://www.reddit.com/r/cpp/comments/84op5c/usability_improvements_in_gcc_8/ > dvtl76x/ > > > const auto s = strlen("test"); > > w

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

2018-03-17 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84889 --- Comment #7 from David Malcolm --- (In reply to David Malcolm from comment #5) > Note to self: need to look at Elm. http://elm-lang.org/blog/compiler-errors-for-humans http://elm-lang.org/blog/compilers-as-assistants

[Bug c++/53281] poor error message for calling a non-const method from a const object

2018-03-18 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53281 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org

[Bug c++/53281] poor error message for calling a non-const method from a const object

2018-03-18 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53281 --- Comment #7 from David Malcolm --- (In reply to David Malcolm from comment #6) [...] > We should also underline the param, or the "const" token. (actually, if the member function or param is 'const', it's not a problem, I think. -ENOCOFFEE)

[Bug c/84890] Overly verbose notes for missing headers

2018-03-18 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84890 --- Comment #5 from David Malcolm --- A similar sentiment to comment #1: https://www.reddit.com/r/cpp/comments/84op5c/usability_improvements_in_gcc_8/dvu7n5e/ > I find that "did you forget to '#include" is kind of personal message > and it's quit

[Bug c++/84943] [8 Regression] internal compiler error: in build_call_a, at cp/call.c:374

2018-03-19 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84943 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

[Bug c++/84993] New: Combination of fieldnames and accessor suggestions for misspelled private fields

2018-03-20 Thread dmalcolm at gcc dot gnu.org
: normal Priority: P3 Component: c++ Assignee: dmalcolm at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- Consider: class foo { public: double get_ratio() const { return m_ratio; } private: double m_ratio; }; void test(foo

[Bug c++/84994] New: Missing accessor hint for private field at -O1 and above

2018-03-20 Thread dmalcolm at gcc dot gnu.org
Severity: normal Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- Given this: class foo { public: double get_ratio() const { return m_ratio; } private: double m_ratio

[Bug c++/84994] Missing accessor hint for private field at -O1 and above

2018-03-20 Thread dmalcolm at gcc dot gnu.org
||2018-03-20 Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Target Milestone|--- |9.0 Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- [confirmed; mine]

[Bug c++/84994] Missing accessor hint for private field at -O1 and above when -g is enabled

2018-03-20 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84994 David Malcolm changed: What|Removed |Added Summary|Missing accessor hint for |Missing accessor hint for

[Bug c++/85004] ambiguous diagnostic: passing ‘const S’ as ‘this’ argument discards qualifiers

2018-03-20 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85004 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org

[Bug c++/84892] Missing accessor hint for private field

2018-03-20 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84892 David Malcolm changed: What|Removed |Added Keywords||patch Status|NEW

[Bug c++/84994] Missing accessor hint for private field at -O1 and above when -g is enabled

2018-03-20 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84994 David Malcolm changed: What|Removed |Added Keywords||patch Status|NEW

[Bug c++/84892] Missing accessor hint for private field

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

[Bug c++/84892] Missing accessor hint for private field

2018-03-21 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84892 --- Comment #3 from David Malcolm --- Author: dmalcolm Date: Wed Mar 21 13:09:09 2018 New Revision: 258716 URL: https://gcc.gnu.org/viewcvs?rev=258716&root=gcc&view=rev Log: C++: show private field accessor hints for const accesses (PR c++/84892

[Bug c++/85021] New: "Missing #include" hints for unrecognized names that would implicitly be in "std"

2018-03-21 Thread dmalcolm at gcc dot gnu.org
D Severity: normal Priority: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- We provide fix-it hints for the most common "std::" names: void f() { std::cout <&

[Bug c++/85021] "Missing #include" hints for unrecognized names that would implicitly be in "std"

2018-03-21 Thread dmalcolm at gcc dot gnu.org
|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Target Milestone|--- |9.0

[Bug c++/84269] More suggestions for missing #include

2018-03-21 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #6 from David Malcolm --- Another one, affecting C++ only: void f() { printf("test"); } : In function 'void f()': :2:5: error: 'printf' was not declared in this scope printf("test"); ^~ :2:5: note: suggested altern

[Bug c++/84994] Missing accessor hint for private field at -O1 and above when -g is enabled

2018-03-21 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84994 --- Comment #4 from David Malcolm --- Author: dmalcolm Date: Wed Mar 21 18:21:39 2018 New Revision: 258731 URL: https://gcc.gnu.org/viewcvs?rev=258731&root=gcc&view=rev Log: C++: show private field accessor hints with -g and optimization (PR c++

[Bug c++/84994] Missing accessor hint for private field at -O1 and above when -g is enabled

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

[Bug c++/84994] Missing accessor hint for private field at -O1 and above when -g is enabled

2018-03-21 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84994 David Malcolm changed: What|Removed |Added Target Milestone|9.0 |8.0

[Bug c/82967] "did you mean" suggestions are way too suggestive

2018-03-21 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82967 David Malcolm changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug c++/84269] More suggestions for missing #include

2018-03-21 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #7 from David Malcolm --- Another one, from https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82967#c1 #include void test (float pf, float inff) { assert (pf == inff); } : In function 'test': :5:3: warning: implicit declaration of f

[Bug c++/84269] More suggestions for missing #include

2018-03-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #8 from David Malcolm --- From PR 84896: gcc 8 currently emits the following for: std::pair foo; error: 'pair' in namespace 'std' does not name a template type std::pair foo; ^~~~ We ought to suggest including for this,

[Bug c++/84269] More suggestions for missing #include

2018-03-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #9 from David Malcolm --- *** Bug 84896 has been marked as a duplicate of this bug. ***

[Bug c++/84896] Better handling of missing for std::pair

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

[Bug c++/84269] More suggestions for missing #include

2018-03-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 David Malcolm changed: What|Removed |Added Keywords||patch Status|ASSIGNED

[Bug c++/85021] "Missing #include" hints for unrecognized names that would implicitly be in "std"

2018-03-22 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85021 David Malcolm changed: What|Removed |Added Keywords||patch Status|NEW

[Bug c/84183] internal compiler error: in initialize, at alloc-pool.h:257

2018-03-23 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84183 --- Comment #3 from David Malcolm --- (In reply to Andre Pribil from comment #2) Thanks. Sounds like this might be an ld bug then. Thomas: do you know which version of binutils you were using?

[Bug c++/63392] poor error recovery with missing typename

2018-03-28 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63392 David Malcolm changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug c/70618] better error messages for missing/too many arguments

2018-03-28 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70618 David Malcolm changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

[Bug c/70924] Wrong position for "warning: missing braces around initializer [-Wmissing-braces]"

2018-03-28 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70924 David Malcolm changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org

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

2018-03-28 Thread dmalcolm at gcc dot gnu.org
: P3 Component: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- In gcc 8, the C++ FE underlines some bogus arguments to function calls, as of r256448. However we're missing some. Given: void callee (in

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

2018-03-28 Thread dmalcolm at gcc dot gnu.org
||2018-03-28 Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- In build_new_function_call, we have: cand = perform_overload_resolution (fn, *args

[Bug fortran/83319] [7/8 Regression] ICE on use of allocatable component in derived type coarray defined in module

2018-03-28 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83319 David Malcolm changed: What|Removed |Added CC||dmalcolm at gcc dot gnu.org --- Comment

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

2018-03-28 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85110 David Malcolm changed: What|Removed |Added Status|ASSIGNED|WAITING --- Comment #2 from David Malcol

[Bug c++/84993] Combination of fieldnames and accessor suggestions for misspelled private fields

2018-03-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84993 David Malcolm changed: What|Removed |Added Target Milestone|--- |9.0

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

2018-03-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85110 --- Comment #3 from David Malcolm --- Author: dmalcolm Date: Thu Mar 29 14:43:01 2018 New Revision: 258957 URL: https://gcc.gnu.org/viewcvs?rev=258957&root=gcc&view=rev Log: More underlining of bad arguments (PR c++/85110) As of r256448, the C+

[Bug c++/84269] More suggestions for missing #include

2018-03-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #12 from David Malcolm --- Author: dmalcolm Date: Thu Mar 29 20:43:48 2018 New Revision: 258966 URL: https://gcc.gnu.org/viewcvs?rev=258966&root=gcc&view=rev Log: More #include suggestions (PR c++/84269) PR c++/84269 reports a numbe

[Bug c++/85215] New: "gcc_assert (!force_elide);" failure

2018-04-04 Thread dmalcolm at gcc dot gnu.org
mponent: c++ Assignee: unassigned at gcc dot gnu.org Reporter: dmalcolm at gcc dot gnu.org Target Milestone: --- Created attachment 43847 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43847&action=edit Reproducer The downstream bug report: https://bugzilla.redhat.co

[Bug c++/85215] [8 Regression] ICE: "gcc_assert (!force_elide);" failure

2018-04-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85215 --- Comment #1 from David Malcolm --- Started somewhere between r258751 (unaffected) and r258755 (affected)

[Bug c++/85215] [8 Regression] ICE: "gcc_assert (!force_elide);" failure

2018-04-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85215 David Malcolm changed: What|Removed |Added CC||jason at gcc dot gnu.org --- Comment #2

[Bug c++/85228] [8 Regression] ICE with constexpr lambda in template

2018-04-05 Thread dmalcolm at gcc dot gnu.org
|NEW Last reconfirmed||2018-04-05 CC||dmalcolm at gcc dot gnu.org, ||jason at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from

[Bug c++/85210] ICE with broken structured binding in template

2018-04-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85210 --- Comment #2 from David Malcolm --- Confirmed. On gcc-7-branch, with checking enabled, this is: test.cc:5:8: internal compiler error: tree check: expected var_decl, have parm_decl in cp_finish_decomp, at cp/decl.c:7354 7454 SET_DECL

[Bug c++/85210] [7/8 Regression] ICE with broken structured binding in template

2018-04-05 Thread dmalcolm at gcc dot gnu.org
||2018-04-05 CC||dmalcolm at gcc dot gnu.org, ||jason at gcc dot gnu.org, ||nathan at gcc dot gnu.org Summary|ICE with broken structured

[Bug c++/85227] [7/8/ Regression] ICE with structured binding of a forward declared variable

2018-04-05 Thread dmalcolm at gcc dot gnu.org
||2018-04-05 CC||dmalcolm at gcc dot gnu.org, ||jason at gcc dot gnu.org Summary|ICE with structured binding |[7/8/ Regression] ICE with |of a forward declared

[Bug c++/85136] [6/7/8 Regression] ICE with array as template variable

2018-04-05 Thread dmalcolm at gcc dot gnu.org
||2018-04-05 CC||dmalcolm at gcc dot gnu.org Summary|ICE with array as template |[6/7/8 Regression] ICE with |variable|array as template variable Ever confirmed|0

[Bug c++/85021] "Missing #include" hints for unrecognized names that would implicitly be in "std"

2018-04-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85021 --- Comment #3 from David Malcolm --- Author: dmalcolm Date: Fri Apr 6 15:46:04 2018 New Revision: 259179 URL: https://gcc.gnu.org/viewcvs?rev=259179&root=gcc&view=rev Log: C++: suggest missing headers for implicit use of "std" (PR c++/85021)

[Bug c++/85021] "Missing #include" hints for unrecognized names that would implicitly be in "std"

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

[Bug c/85260] [8 Regression] python-2.7.14 miscompiled at -O3

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

[Bug c++/84269] More suggestions for missing #include

2018-04-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84269 --- Comment #13 from David Malcolm --- Author: dmalcolm Date: Fri Apr 6 17:36:33 2018 New Revision: 259184 URL: https://gcc.gnu.org/viewcvs?rev=259184&root=gcc&view=rev Log: C++: more std header hints; filter on C++ dialect (PR c++/84269) This

[Bug c++/84269] More suggestions for missing #include

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

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

2018-04-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85110 --- Comment #4 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2018-04/msg00342.html

[Bug c++/85948] poor location info for arguments and format strings (printf format warning)

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

[Bug c++/56856] -Wformat warnings don't show location *within* format string in C++ FE

2018-10-04 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56856 David Malcolm changed: What|Removed |Added CC||jg at jguk dot org --- Comment #9 from D

[Bug c++/56856] -Wformat warnings don't show location *within* format string in C++ FE

2018-10-04 Thread dmalcolm at gcc dot gnu.org
|ASSIGNED Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #10 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2018-10/msg00269.html

[Bug ipa/86395] add support of -fopt-info-inline in inliner

2018-10-04 Thread dmalcolm at gcc dot gnu.org
gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #3 from David Malcolm --- I'm looking at this.

[Bug c++/56856] -Wformat warnings don't show location *within* format string in C++ FE

2018-10-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56856 --- Comment #11 from David Malcolm --- Author: dmalcolm Date: Fri Oct 5 19:02:17 2018 New Revision: 264887 URL: https://gcc.gnu.org/viewcvs?rev=264887&root=gcc&view=rev Log: Support string locations for C++ in -Wformat (PR c++/56856) -Wformat

[Bug c/52952] Wformat location info is bad (wrong column number)

2018-10-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 Bug 52952 depends on bug 56856, which changed state. Bug 56856 Summary: -Wformat warnings don't show location *within* format string in C++ FE https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56856 What|Removed |Ad

[Bug c++/56856] -Wformat warnings don't show location *within* format string in C++ FE

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

[Bug c/52952] Wformat location info is bad (wrong column number)

2018-10-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #52 from David Malcolm --- (In reply to Manuel López-Ibáñez from comment #51) > 1. C++ does not work C++ should be fixed for gcc 9 by r264887.

[Bug c/52952] Wformat location info is bad (wrong column number)

2018-10-06 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #53 from David Malcolm --- (In reply to Manuel López-Ibáñez from comment #51) > 2. Locations within strings expanded from a macro (2) should also be fixed for gcc 9 by r264887: /tmp/foo.c: In function ‘foo’: /tmp/foo.c:2:25: warning

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-09 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 --- Comment #1 from David Malcolm --- linemap_position_for_line_and_column(line_maps*, line_map_ordinary const*, unsigned int, unsigned int) at libcpp/line-map.c:848 is: linemap_assert (ORDINARY_MAP_STARTING_LINE_NUMBER (ord_map) <= line); I w

[Bug c++/84993] Combination of fieldnames and accessor suggestions for misspelled private fields

2018-10-11 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84993 --- Comment #3 from David Malcolm --- Author: dmalcolm Date: Thu Oct 11 19:03:33 2018 New Revision: 265056 URL: https://gcc.gnu.org/viewcvs?rev=265056&root=gcc&view=rev Log: C++: suggestions for misspelled private members (PR c++/84993) PR c++/

[Bug c++/84993] Combination of fieldnames and accessor suggestions for misspelled private fields

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

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-17 Thread dmalcolm at gcc dot gnu.org
||2018-10-17 Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #3 from David Malcolm --- (In reply to Renlin Li from comment #2) > I checked that, the ICE starts from r264887. Tha

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-17 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 David Malcolm changed: What|Removed |Added CC||rguenth at gcc dot gnu.org --- Comment #

[Bug preprocessor/87630] [9 Regression] ICE in linemap_position_for_line_and_column, at libcpp/line-map.c:842

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

[Bug c++/87643] [9 regression] ICE at dbxout.c:508:1: linemap_position_for_line_and_column starting with r264887

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

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-18 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 David Malcolm changed: What|Removed |Added CC||seurer at gcc dot gnu.org --- Comment #5

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-18 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 --- Comment #6 from David Malcolm --- Author: dmalcolm Date: Thu Oct 18 15:44:39 2018 New Revision: 265271 URL: https://gcc.gnu.org/viewcvs?rev=265271&root=gcc&view=rev Log: Fix ICE in substring-handling building 502.gcc_r (PR 87562) In r264887

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

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

[Bug c++/87721] [9 Regression] ICE in linemap_position_for_line_and_column at gcc/libcpp/line-map.c:842 since r265271

2018-10-24 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #1 from David Malcolm --- Confirmed; thanks. I'm able to reproduce this and am working on reducing it.

[Bug c++/87721] [9 Regression] ICE in linemap_position_for_line_and_column at gcc/libcpp/line-map.c:842 since r265271

2018-10-24 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87721 --- Comment #2 from David Malcolm --- Reduced reproducer: # define DBG_ERROR(dbg_logger, format, args...) if (1){\ char dbg_buffer[256]; \ __builtin_snprintf(dbg_buffer, sizeof(dbg_buffer)-1,\ __FILE__":%5d: " format , __LINE__ , ## args)

[Bug middle-end/87757] weird underlining and colors in sprintf warnings for unterminated arrays

2018-10-26 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87757 David Malcolm changed: What|Removed |Added CC||aldyh at gcc dot gnu.org --- Comment #2

[Bug middle-end/87757] weird underlining and colors in sprintf warnings for unterminated arrays

2018-10-26 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87757 --- Comment #3 from David Malcolm --- prior to that gimplify.c code, the location of the argument is correctly set, as: t.c:8:31: note: 8 | __builtin_sprintf (e, "%s", &a[0]); | ^

[Bug middle-end/87757] weird underlining and colors in sprintf warnings for unterminated arrays

2018-10-26 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87757 --- Comment #4 from David Malcolm --- Aldy: do you remember why the arguments need to have the same location as the call itself? The first case works because during gimplify_arg it's wrapped in a NOP_EXPR which is casting from char[] to char *,

[Bug c++/56856] -Wformat warnings don't show location *within* format string in C++ FE

2018-10-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=56856 --- Comment #13 from David Malcolm --- Author: dmalcolm Date: Mon Oct 29 23:44:10 2018 New Revision: 265609 URL: https://gcc.gnu.org/viewcvs?rev=265609&root=gcc&view=rev Log: Folding and check_function_arguments This patch eliminates the argloc

[Bug tree-optimization/87562] [9 Regression] ICE in in linemap_position_for_line_and_column, at libcpp/line-map.c:848

2018-10-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87562 --- Comment #9 from David Malcolm --- Author: dmalcolm Date: Mon Oct 29 23:58:34 2018 New Revision: 265611 URL: https://gcc.gnu.org/viewcvs?rev=265611&root=gcc&view=rev Log: Fix ICE in get_substring_ranges_for_loc on __FILE__ (PR c++/87721) PR

[Bug c++/87721] [9 Regression] ICE in linemap_position_for_line_and_column at gcc/libcpp/line-map.c:842 since r265271

2018-10-29 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87721 --- Comment #4 from David Malcolm --- Author: dmalcolm Date: Mon Oct 29 23:58:34 2018 New Revision: 265611 URL: https://gcc.gnu.org/viewcvs?rev=265611&root=gcc&view=rev Log: Fix ICE in get_substring_ranges_for_loc on __FILE__ (PR c++/87721) PR

[Bug c++/87721] [9 Regression] ICE in linemap_position_for_line_and_column at gcc/libcpp/line-map.c:842 since r265271

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

[Bug c++/87504] inconsistent diagnostic style between C and C++ for binary operators

2018-11-05 Thread dmalcolm at gcc dot gnu.org
||2018-11-05 Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Candidate patch for C++ part of this: https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00303

[Bug c++/43486] Preserve variable-use locations

2018-11-05 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org --- Comment #14 from David Malcolm --- Candidate followup patch: https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00304.html

[Bug c++/43064] improve location and text of diagnostics in constructor initializer lists

2018-11-05 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43064 --- Comment #9 from David Malcolm --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00304.html

[Bug ipa/86395] add support of -fopt-info-inline in inliner

2018-11-07 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86395 David Malcolm changed: What|Removed |Added Keywords||patch URL|

[Bug ipa/86395] add support of -fopt-info-inline in inliner

2018-11-08 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86395 --- Comment #5 from David Malcolm --- Author: dmalcolm Date: Thu Nov 8 15:38:30 2018 New Revision: 265920 URL: https://gcc.gnu.org/viewcvs?rev=265920&root=gcc&view=rev Log: ipa-inline.c/tree-inline.c: port from fprintf to dump API (PR ipa/86395

[Bug ipa/87955] [9 Regression] ICE in cl_target_option_print_diff at gcc/options-save.c:3803 since r265920

2018-11-09 Thread dmalcolm at gcc dot gnu.org
||2018-11-09 Assignee|unassigned at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Sorry about that. Confirmed; I'm working on a fix.

[Bug c++/87850] Add fix-it hint for "invalid conversion from 'X' to 'X*'"

2018-11-09 Thread 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 --- Candidate patch: https://gcc.gnu.org/ml/gcc-patches/2018-11/msg00802.html

[Bug ipa/87955] [9 Regression] ICE in cl_target_option_print_diff at gcc/options-save.c:3803 since r265920

2018-11-10 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87955 David Malcolm changed: What|Removed |Added Keywords||patch URL|

[Bug other/19165] (Natural) language independent error / warning classification

2018-11-12 Thread dmalcolm at gcc dot gnu.org
gcc dot gnu.org --- Comment #24 from David Malcolm --- Candidate patch for JSON output: https://gcc.gnu.org/ml/gcc-patches/2018-11/msg01038.html

[Bug ipa/87955] [9 Regression] ICE in cl_target_option_print_diff at gcc/options-save.c:3803 since r265920

2018-11-13 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87955 --- Comment #3 from David Malcolm --- Author: dmalcolm Date: Tue Nov 13 15:59:57 2018 New Revision: 266079 URL: https://gcc.gnu.org/viewcvs?rev=266079&root=gcc&view=rev Log: Fix ICE with -fopt-info-inline (PR ipa/87955) PR ipa/87955 reports a p

[Bug ipa/87955] [9 Regression] ICE in cl_target_option_print_diff at gcc/options-save.c:3803 since r265920

2018-11-13 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87955 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug tree-optimization/88015] [9 Regression] ICE in dump_printf_loc, at dumpfile.c:1287

2018-11-14 Thread dmalcolm at gcc dot gnu.org
at gcc dot gnu.org |dmalcolm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from David Malcolm --- Confirmed, though for some reason I needed to add -floop-parallelize-all to the flags in comment #0 to trigger it.

[Bug tree-optimization/88015] [9 Regression] ICE in dump_printf_loc, at dumpfile.c:1287

2018-11-14 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88015 --- Comment #2 from David Malcolm --- Actually, I'm still not able to reproduce this, but the fix seems obvious; am working on it.

[Bug tree-optimization/88015] [9 Regression] ICE in dump_printf_loc, at dumpfile.c:1287

2018-11-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88015 --- Comment #5 from David Malcolm --- Author: dmalcolm Date: Thu Nov 15 14:01:29 2018 New Revision: 266184 URL: https://gcc.gnu.org/viewcvs?rev=266184&root=gcc&view=rev Log: graphite: add missing dump_enabled_p checks (PR tree-optimization/88015

[Bug tree-optimization/88015] [9 Regression] ICE in dump_printf_loc, at dumpfile.c:1287

2018-11-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88015 David Malcolm changed: What|Removed |Added Status|ASSIGNED|WAITING --- Comment #6 from David Malcol

[Bug other/19165] (Natural) language independent error / warning classification

2018-11-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19165 --- Comment #25 from David Malcolm --- Author: dmalcolm Date: Thu Nov 15 14:32:41 2018 New Revision: 266186 URL: https://gcc.gnu.org/viewcvs?rev=266186&root=gcc&view=rev Log: Machine-readable diagnostic output (PR other/19165) This patch implem

[Bug other/19165] (Natural) language independent error / warning classification

2018-11-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19165 David Malcolm changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug c++/88036] bogus "was not declared in this scope; did you mean ...?" fix-it when the declaration was ill-formed

2018-11-15 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88036 David Malcolm changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug tree-optimization/88015] [9 Regression] ICE in dump_printf_loc, at dumpfile.c:1287

2018-11-16 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88015 --- Comment #8 from David Malcolm --- Thanks!

<    5   6   7   8   9   10   11   12   13   14   >