[Bug middle-end/4210] should not warn in dead code

2020-05-05 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=4210 --- Comment #35 from Manuel López-Ibáñez --- (In reply to Niels Möller from comment #32) > 1. There's similar code in c_fully_fold_internal, in gcc/c/c-fold.c, close > to line 400. But that code does *not* emit any warning for the example > above,

[Bug c/93432] variable is used uninitialized, but gcc shows no warning

2020-06-08 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93432 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/96121] Uninitialized variable copying not diagnosed

2020-07-13 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Keywords||easyhack --- Comment #6 from Manuel López-Ibáñez --- Probably a dup of bug 19808. There is an unfinished patch there that does the check in the FE. That would side-step issues with A being empty (you just

[Bug c/93432] variable is used uninitialized, but gcc shows no warning

2020-07-13 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93432 Manuel López-Ibáñez changed: What|Removed |Added Keywords||easyhack --- Comment #5 from Manue

[Bug preprocessor/49973] Column numbers count multibyte characters as multiple columns

2020-07-14 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=49973 Manuel López-Ibáñez changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[Bug c/61579] -Wwrite-strings does not behave as a warning option

2020-07-22 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61579 --- Comment #8 from Manuel López-Ibáñez --- (In reply to David Brown from comment #7) > Could "-Wwrite-strings" be split into two options? The warning could remain > (and become part of -Wall for C as well as C++) if the compiler can spot and >

[Bug tree-optimization/18501] [8/9/10/11 Regression] Missing 'used uninitialized' warning (CCP)

2020-07-29 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=18501 Manuel López-Ibáñez changed: What|Removed |Added CC||felix at piedallu dot me --- Comme

[Bug middle-end/24639] [meta-bug] bug to track all Wuninitialized issues

2020-07-29 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639 Bug 24639 depends on bug 96368, which changed state. Bug 96368 Summary: False negative with maybe-uninitialized with goto https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96368 What|Removed |Added --

[Bug c/96368] False negative with maybe-uninitialized with goto

2020-07-29 Thread manu at gcc dot gnu.org
|--- |DUPLICATE CC||manu at gcc dot gnu.org --- Comment #1 from Manuel López-Ibáñez --- I'm pretty sure this is PR18501. pktio_read() creates a PHI node and prevents CCP deleting the undefined value. *** This bug has been mark

[Bug middle-end/96300] missing -Wuninitialized reading a struct member by a conditionally assigned pointer

2020-07-29 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Status|UNCONFIRMED |NEW Last reconfirmed||2020-07-29 --- Comment #2 from Manuel López-Ibáñez --- This seems some kind of weird missed optimization because this variant does warn: struct S { int i

[Bug c/96368] False negative with maybe-uninitialized with goto

2020-07-29 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96368 Manuel López-Ibáñez changed: What|Removed |Added Status|VERIFIED|RESOLVED --- Comment #3 from Manue

[Bug c/96629] spurious maybe uninitialized variable warning with difficult control-flow analysis

2020-09-03 Thread manu at gcc dot gnu.org
|variable warning with |uninitialized variable |branches at -O1 and higher |warning with difficult ||control-flow analysis CC||manu at gcc dot gnu.org --- Comment #2 from Manuel

[Bug middle-end/95848] missing -Wuninitialized passing structs by value (VOPS)

2020-09-03 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95848 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/94905] Bogus warning -Werror=maybe-uninitialized

2020-09-03 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94905 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/94905] [10 Regression] Bogus warning -Werror=maybe-uninitialized

2020-09-03 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94905 Manuel López-Ibáñez changed: What|Removed |Added Summary|Bogus warning |[10 Regression] Bogus

[Bug tree-optimization/94774] Uninitialized variable retval in function try_substitute_return_value

2020-09-03 Thread manu at gcc dot gnu.org
|--- |FIXED CC||manu at gcc dot gnu.org --- Comment #4 from Manuel López-Ibáñez --- Seems fixed.

[Bug middle-end/24639] [meta-bug] bug to track all Wuninitialized issues

2020-09-03 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639 Bug 24639 depends on bug 94774, which changed state. Bug 94774 Summary: Uninitialized variable retval in function try_substitute_return_value https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94774 What|Removed |Add

[Bug c++/83591] -Wduplicated-branches fires in system headers in template instantiation

2020-09-04 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83591 --- Comment #8 from Manuel López-Ibáñez --- (In reply to Tony E Lewis from comment #7) > Manuel López-Ibáñez: are you happy that all underlying issues are resolved > and this can be closed? Sure.

[Bug c++/66099] _Pragma diagnostic 'ignored' in macro with strict-overflow not suppressing warning fully with -Werror

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66099 Manuel López-Ibáñez changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug middle-end/92330] New: Wstrict-overflow documentation does not say that it is deprecated and has no effect

2019-11-02 Thread manu at gcc dot gnu.org
Severity: normal Priority: P3 Component: middle-end Assignee: unassigned at gcc dot gnu.org Reporter: manu at gcc dot gnu.org CC: msebor at gcc dot gnu.org, rguenth at gcc dot gnu.org Target Milestone: --- According to GCC 8, Wstrict-overflow is

[Bug c++/55881] #pragma GCC diagnostic ignored ignored when inlining

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55881 Manuel López-Ibáñez changed: What|Removed |Added Last reconfirmed|2013-01-07 00:00:00 |2019-11-2 --- Comment #8 from Manu

[Bug tree-optimization/91890] [10 Regression] -Warray-bounds warning testing glibc not suppressed by pragma

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91890 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug middle-end/92330] Wstrict-overflow documentation does not say that it is deprecated and has no effect

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92330 --- Comment #1 from Manuel López-Ibáñez --- Actually, it is not even deprecated. There are still a bunch of Wstrict-overflow warnings, just some of them got removed. Is there a way to tell which ones are still active and update the documentati

[Bug tree-optimization/91890] [10 Regression] -Warray-bounds warning testing glibc not suppressed by pragma

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91890 --- Comment #4 from Manuel López-Ibáñez --- I'm 100% convinced this has nothing to do with locations and all to do with how -Warray-bounds and -Wstringop-overflow= interact. Change the ignored for error, char one[50]; char two[50]; void test_s

[Bug tree-optimization/91890] [10 Regression] -Warray-bounds warning testing glibc not suppressed by pragma

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91890 --- Comment #5 from Manuel López-Ibáñez --- 333 Warray-bounds 334 LangEnabledBy(C ObjC C++ LTO ObjC++) 335 ; in common.opt This seems wrong, the second argument ", Wall" is missing. Moreover, this probably should be an Alias for some -Warray-

[Bug c++/90058] False Positive in undefined-sanitizer only with GCC8

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90058 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/89976] missing uninitialized warning for uninitialized struct member (VOPs)

2019-11-02 Thread manu at gcc dot gnu.org
||2019-11-02 CC||manu at gcc dot gnu.org Depends on||49754, 79658 Summary|missing uninitialized |missing uninitialized |warning: laundering via |warning for

[Bug tree-optimization/89202] missing -Wnonnull-dereference or -Wuninitialized for a certain bug

2019-11-02 Thread manu at gcc dot gnu.org
||2019-11-02 CC||manu at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Manuel López-Ibáñez --- For the Wuninit, this is PR18501

[Bug middle-end/24639] [meta-bug] bug to track all Wuninitialized issues

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639 Bug 24639 depends on bug 89192, which changed state. Bug 89192 Summary: -Wuninitialized doesn't warn about a vector initialization with uninitialized field https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89192 What|Removed

[Bug c++/19808] miss a warning about uninitialized member usage in member initializer list in constructor

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19808 Manuel López-Ibáñez changed: What|Removed |Added CC||Hi-Angel at yandex dot ru --- Comm

[Bug c++/89192] -Wuninitialized doesn't warn about a vector initialization with uninitialized field

2019-11-02 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Resolution|--- |DUPLICATE --- Comment #4 from Manuel López-Ibáñez --- Duplicated *** This bug has been marked as a duplicate of bug 19808 ***

[Bug middle-end/88175] Showing header file instead of source code line for uninitialized variable

2019-11-02 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88175 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c/91820] missing error diagnosis of '&' in initialization

2019-12-16 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Resolution|--- |INVALID --- Comment #6 from Manuel López-Ibáñez --- Not a bug per comment #5

[Bug c/91839] missing error diagnosis for undeclared identifier

2019-12-16 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Resolution|--- |WONTFIX --- Comment #4 from Manuel López-Ibáñez --- (In reply to tangyixuan from comment #3) > Error handler should not stop at the first error and report the errors as > many as possible. This

[Bug middle-end/92118] warning with [-Wmaybe-uninitialized] in -O1

2019-12-16 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Resolution|--- |DUPLICATE --- Comment #1 from Manuel López-Ibáñez --- When reporting a Wuninitialized bug please check PR24639. 99% of the cases are duplicated of an existing report. *** This bug has been marked as a

[Bug c++/43361] missing uninitialized warning without optimization (-O0) (PHI in always_executed basic block)

2019-12-16 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=43361 Manuel López-Ibáñez changed: What|Removed |Added CC||tangyixuan at mail dot dlut.edu.cn

[Bug c/92210] no warning for invariable used in loop condition

2019-12-16 Thread manu at gcc dot gnu.org
|UNCONFIRMED |NEW Last reconfirmed||2019-12-16 CC||manu at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Manuel López-Ibáñez --- This one would be useful to have and

[Bug c/92209] Imprecise column number for -Wstrict-prototypes

2019-12-16 Thread manu at gcc dot gnu.org
|UNCONFIRMED |NEW Last reconfirmed||2019-12-16 CC||manu at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Manuel López-Ibáñez --- The key here is to add the fix-hit hint

[Bug c/92392] -Wignored-qualifiers points to wrong location and doesn't mention which qualifiers

2019-12-16 Thread manu at gcc dot gnu.org
CC||manu at gcc dot gnu.org Summary|-Wignored-qualifiers points |-Wignored-qualifiers points |to diff location|to wrong location and ||doesn't mention

[Bug c/92479] missing warnings for unreachable codes with -Wunreachable-code

2019-12-16 Thread manu at gcc dot gnu.org
||2019-12-16 CC||manu at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #5 from Manuel López-Ibáñez --- This should be suspended until someone comes up with a proposal for reviving -Wunreachable

[Bug c/92826] Impossible to silence warning: non-standard suffix on floating constant [-Wpedantic]

2019-12-16 Thread manu at gcc dot gnu.org
Status|UNCONFIRMED |NEW Last reconfirmed||2019-12-16 CC||manu at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #6 from Manuel López-Ibáñez --- The fact that GCC points to

[Bug c/92826] Impossible to silence warning: non-standard suffix on floating constant [-Wpedantic]

2019-12-16 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92826 --- Comment #7 from Manuel López-Ibáñez --- (In reply to kim.walisch from comment #0) > GCC should definitely not warn when using constants from . GCC > should also provide an option to disable these warnings (e.g. > -Wno-non-standard-suffix). Pe

[Bug c++/80635] std::optional and bogus -Wmaybe-uninitialized warning

2019-12-17 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 --- Comment #34 from Manuel López-Ibáñez --- (In reply to Martin Sebor from comment #15) > I think the following smaller test case independent of libstdc++ captures > the same issue as the bigger test case in comment #4. Again, declaring f() > n

[Bug c++/80635] std::optional and bogus -Wmaybe-uninitialized warning

2019-12-17 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80635 --- Comment #35 from Manuel López-Ibáñez --- In any case, looking at the uninit dump with -fdump-tree-all-all-lineno it appears that GCC knows the block where the warning is triggered is never executed: ;; basic block 13, loop depth 0, count 0

[Bug middle-end/323] optimized code gives strange floating point results

2020-02-07 Thread manu at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=323 --- Comment #216 from Manuel López-Ibáñez --- (In reply to Vincent Lefèvre from comment #215) > According to https://gcc.gnu.org/bugzilla/page.cgi?id=fields.html#bug_status > the possible status are UNCONFIRMED, CONFIRMED and IN_PROGRESS. I think t

[Bug libstdc++/56202] SIGFPE (division by zero) in std::binomial_distribution

2013-02-04 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56202 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug libstdc++/56202] SIGFPE (division by zero) in std::binomial_distribution

2013-02-04 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56202 --- Comment #7 from Manuel López-Ibáñez 2013-02-04 17:29:41 UTC --- I don't understand the check for __e. If you continue, then neither __t nor __x change, the only difference is that you sample a new __e. But __e doesn't have any effect in th

[Bug libstdc++/56202] SIGFPE (division by zero) in std::binomial_distribution

2013-02-04 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56202 --- Comment #8 from Manuel López-Ibáñez 2013-02-04 17:35:00 UTC --- I would understand something like: const double __e = -std::log(1.0 - __aurng()); if (__t == __x) { return __x; } el

[Bug libstdc++/56202] SIGFPE (division by zero) in std::binomial_distribution

2013-02-04 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56202 --- Comment #10 from Manuel López-Ibáñez 2013-02-04 18:58:18 UTC --- (In reply to comment #9) > You are right, but then I don't understand why we should compute __e *before* > checking __t == __x, per your first patch (I think I managed to confu

[Bug libstdc++/56202] SIGFPE (division by zero) in std::binomial_distribution

2013-02-04 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56202 --- Comment #11 from Manuel López-Ibáñez 2013-02-04 19:00:22 UTC --- (In reply to comment #10) > (In reply to comment #9) > > You are right, but then I don't understand why we should compute __e > > *before* > > checking __t == __x, per your fi

[Bug middle-end/54582] gap in FORTIFY checking of buffer lengths

2013-02-06 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54582 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug middle-end/54582] gap in FORTIFY checking of buffer lengths

2013-02-06 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54582 --- Comment #8 from Manuel López-Ibáñez 2013-02-06 13:39:32 UTC --- (In reply to comment #7) > Because object sizes are finalized only during the objsz pass, after lots of > optimization passes. Note, as I said earlier, what matters most is tha

[Bug middle-end/56231] warning traces have bogus line information when using LTO

2013-02-07 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56231 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c/56399] Adding int to float generates incorrect results

2013-02-19 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56399 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/56418] errors do not show the types (makes it hard to debug)

2013-02-21 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56418 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug sanitizer/56454] need to rename attribute no_address_safety_analysis to no_sanitize_address

2013-02-26 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56454 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug sanitizer/56454] need to rename attribute no_address_safety_analysis to no_sanitize_address

2013-02-26 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56454 --- Comment #10 from Manuel López-Ibáñez 2013-02-26 11:19:34 UTC --- (In reply to comment #9) > Clang accepts the old name silently. > I am not sure about "forever", > clang will probably start printing a "deprecated" warning at some point. W

[Bug tree-optimization/49234] [4.5/4.6/4.7/4.8 Regression] -Wstrict-overflow gives obviously unwarranted warning

2013-02-28 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49234 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/56497] comparison is always true due to limited range of data type

2013-03-01 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56497 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug middle-end/56589] Array bounds violation is very end-user unfriendly

2013-03-11 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56589 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug tree-optimization/53265] Warn when undefined behavior implies smaller iteration count

2013-03-11 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53265 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug tree-optimization/53265] Warn when undefined behavior implies smaller iteration count

2013-03-11 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53265 --- Comment #12 from Manuel López-Ibáñez 2013-03-11 16:49:25 UTC --- (In reply to comment #10) > (In reply to comment #8) > > Not sure about the warning wording > > What about (... "iteration %E invokes undefined behavior", max)? > > > plus no

[Bug c/56599] very confusing compiler diagnostics (for stupid bug on my part)

2013-03-11 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56599 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

[Bug c++/56403] [4.6/4.7 Regression] internal compiler error: in build_zero_init_1, at cp/init.c:279

2013-03-14 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org --- Comment #12 from Manuel López-Ibáñez 2013-03-14 15:43:28 UTC --- (In reply to comment #11) > I'm not familiar with gcc's backport process; is there anything I can do to > help that along? Is it already in-queue? I guess Jakub i

[Bug c++/56725] extra spaces in error message

2013-03-25 Thread manu at gcc dot gnu.org
||2013-03-25 CC||manu at gcc dot gnu.org Ever Confirmed|0 |1 --- Comment #1 from Manuel López-Ibáñez 2013-03-25 22:55:48 UTC --- This is because the second error is not an error but a note clarifying with

[Bug c++/56725] extra spaces in error message

2013-03-25 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #3 from Manuel López-Ibáñez 2013-03-25 23:07:00 UTC --- BTW, in this case, I find the output of g++ much better than that of clang++ test.cc:7:10: error: no matching function for call to 'callf' return callf (23, 72, ^~~~

[Bug c++/56725] extra spaces in error message

2013-03-25 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #2 from Manuel López-Ibáñez 2013-03-25 23:03:42 UTC --- Note that there are a fair amount of calls like these in the C++ FE, and the use is inconsistent. I guess the indentation predates the use of inform, and this is why there are s

[Bug c++/51440] Improve message and add an option for warning for an unnamed struct member: TYPE has a field FIELD whose type uses the anonymous namespace

2013-03-26 Thread manu at gcc dot gnu.org
||2013-03-26 CC||manu at gcc dot gnu.org Summary|C++ compiler produces |Improve message and add an |warning for an unnamed |option for warning for an |struct member: TYPE has a

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

2013-03-26 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=19165 --- Comment #18 from Manuel López-Ibáñez 2013-03-27 00:59:43 UTC --- (In reply to comment #17) > (In reply to comment #16) > > > > If you have some developer power to spare, it may be worthwhile to try to > > tackle this yourself. Otherwise I a

[Bug c++/56725] extra spaces in error message

2013-03-27 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #5 from Manuel López-Ibáñez 2013-03-27 17:43:02 UTC --- (In reply to comment #4) > Hi Manuel. I wanted to actually handle this per your indications and move on, > but I don't understand why you check the return value of permerror: sh

[Bug c++/56725] extra spaces in error message

2013-03-27 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #7 from Manuel López-Ibáñez 2013-03-27 19:46:57 UTC --- (In reply to comment #6) > I haven't tested anything so far, but stared a bit at the comment before > permerror and figured tgat in case of plain error (vs warning) the function

[Bug c++/56725] extra spaces in error message

2013-03-27 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #8 from Manuel López-Ibáñez 2013-03-27 19:48:21 UTC --- (In reply to comment #7) > manuel@gcc10:~$ ~/test1/195333/build/gcc/cc1plus test.cc -fpermissive > test.cc:4:5: note: initializing argument 3 of ‘int callf(int, int, int > (*

[Bug c++/56725] extra spaces in error message

2013-03-27 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #10 from Manuel López-Ibáñez 2013-03-27 20:24:47 UTC --- No, true means something was reported (error or warning), false means that nothing was reported. The same as for pedwarn and warning. pedwarn also returns true for --pedantic-e

[Bug c++/56725] extra spaces in error message

2013-03-27 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56725 --- Comment #12 from Manuel López-Ibáñez 2013-03-27 20:37:05 UTC --- That was the intention, if the comment or the behaviour does not match this, then I did a mistake implementing it, and it is a bug in my opinion.

[Bug libstdc++/53631] [C++11] is unimplemented

2013-03-28 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53631 Manuel López-Ibáñez changed: What|Removed |Added CC||manu at gcc dot gnu.org

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

2013-03-30 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #15 from Manuel López-Ibáñez 2013-03-30 12:32:08 UTC --- (In reply to comment #13) > I guess the C/C++ FEs could for non-trivial string literals put into a hash > table mapping from locus_t (of ADDR_EXPR around STRING_CST) to the fir

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

2013-03-30 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #16 from Manuel López-Ibáñez 2013-03-30 12:32:57 UTC --- Created attachment 29753 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29753 loc_token_hash

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

2013-03-30 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 Manuel López-Ibáñez changed: What|Removed |Added CC||joseph at codesourcery dot

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

2013-03-30 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #19 from Manuel López-Ibáñez 2013-03-30 13:26:19 UTC --- Unfortunately, there are some stuff like this: BLK size unit size align 8 symtab 0 alias set -1 canonical type 0x770945

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

2013-03-30 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #20 from Manuel López-Ibáñez 2013-03-30 15:02:11 UTC --- (In reply to comment #18) > > * It would be extremely nice to update the testsuite to check the locations > > are > > correct. This is unfortunately a lot of boring work, so i

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

2013-03-30 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #21 from Manuel López-Ibáñez 2013-03-30 15:03:14 UTC --- Created attachment 29755 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29755 add locations and offsets to format warnings My current patch, untested except for a few tes

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

2013-04-01 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #22 from Manuel López-Ibáñez 2013-04-01 14:17:45 UTC --- (In reply to comment #13) > and didn't need to be translated. So, printf ("%.*d"); (the common case) > wouldn't have to be recorded, while printf (R"<<<(%)>>>" "." R"(*)" "d")

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

2013-04-01 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 Manuel López-Ibáñez changed: What|Removed |Added Attachment #29753|0 |1 is obsolete|

[Bug c++/56815] void pointer arithmetic

2013-04-03 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56815 --- Comment #8 from Manuel López-Ibáñez 2013-04-03 09:40:30 UTC --- (In reply to comment #7) > In practice the warning (a pedwarn) is emitted by code shared with the C > front-end, the error (a permerror, thus with -fpermissive it can be demoted

[Bug c++/56815] void pointer arithmetic

2013-04-03 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56815 --- Comment #13 from Manuel López-Ibáñez 2013-04-03 11:53:23 UTC --- (In reply to comment #9) > Ok Manuel, thanks. I'm not completely convinced by the > > else if ((pedantic || warn_pointer_arith) > > which is protecting the permerrors (a -W

[Bug c++/56815] void pointer arithmetic

2013-04-03 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56815 Manuel López-Ibáñez changed: What|Removed |Added CC||jason at gcc dot gnu.org --- Comme

[Bug c++/56815] void pointer arithmetic

2013-04-03 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56815 --- Comment #16 from Manuel López-Ibáñez 2013-04-03 12:12:15 UTC --- BTW, I also see that in c-family/c.opt -Wpointer-arith is not LangEnabledBy(C ObjC C++ ObjC++,Wpedantic). If it was, then -Werror=pedantic will automatically handle -Werror=poi

[Bug c++/56815] void pointer arithmetic

2013-04-04 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56815 --- Comment #22 from Manuel López-Ibáñez 2013-04-04 10:38:06 UTC --- (In reply to comment #21) > Manuel, I'm adding the LangEnabledBy, to match the documentation. Thanks. > > Now, I'm not sure to understand the existing lines (many): > > ped

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

2013-04-05 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52952 --- Comment #25 from Manuel López-Ibáñez 2013-04-05 22:02:26 UTC --- I am currently stuck on the three problems I described above and I cannot figure out a way to fix any of them: * How to reprocess tokens that need to be translated/have prefix

[Bug c++/56856] New: the location of -Wformat warnings point *after* the format string

2013-04-06 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56856 Bug #: 56856 Summary: the location of -Wformat warnings point *after* the format string Classification: Unclassified Product: gcc Version: 4.9.0 Status: UNCONFIRMED

[Bug c++/56856] the location of Wformat warnings points *after* the format string

2013-04-06 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56856 Manuel López-Ibáñez changed: What|Removed |Added Keywords||diagnostic Blocks|

[Bug c++/16564] g++ seems to go into an infinite loop after errors

2013-04-09 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=16564 --- Comment #16 from Manuel López-Ibáñez 2013-04-09 18:04:14 UTC --- (In reply to comment #15) > Manuel, compile_file changed with this commit: > > http://gcc.gnu.org/viewcvs/gcc?view=revision&revision=118362 > > I seem to remember that anoth

[Bug c/56824] pragma GCC diagnostic push/pop regression for GCC diagnostic ignored "-Waggregate-return"

2013-04-10 Thread manu at gcc dot gnu.org
||2013-04-10 CC||manu at gcc dot gnu.org Ever Confirmed|0 |1 --- Comment #3 from Manuel López-Ibáñez 2013-04-10 18:32:00 UTC --- I think the problem is the continue, which moves to the next i--: Index

[Bug c/56824] pragma GCC diagnostic push/pop regression for GCC diagnostic ignored "-Waggregate-return"

2013-04-12 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56824 --- Comment #5 from Manuel López-Ibáñez 2013-04-12 09:12:32 UTC --- Probably the while loop is looping forever in the continue. I don't have time to investigate this, but I am pretty sure that the bug is somewhere there. If you use gdb to invest

[Bug other/36150] colorize output of gcc

2013-04-14 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=36150 Manuel López-Ibáñez changed: What|Removed |Added Resolution|INVALID |FIXED --- Comment #16 from Manuel

[Bug c/56972] Missing "may be used uninitialized" warning for "obvious" uninitialized

2013-04-15 Thread manu at gcc dot gnu.org
||manu at gcc dot gnu.org Resolution||DUPLICATE --- Comment #1 from Manuel López-Ibáñez 2013-04-15 18:46:59 UTC --- Infamous PR18501. With: ~/test1/197214/build/gcc/cc1 -Wall -Wextra -O1 pr56972.c -fdump-tree-all-lineno you can see that

[Bug tree-optimization/18501] [4.7/4.8/4.9 Regression] Missing 'used uninitialized' warning (CCP)

2013-04-15 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=18501 Manuel López-Ibáñez changed: What|Removed |Added CC||ppluzhnikov at google dot

[Bug c/56980] Misleading note

2013-04-16 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56980 --- Comment #1 from Manuel López-Ibáñez 2013-04-16 15:24:46 UTC --- Confirmed, but I seriously doubt it has anything to do with my patch. At the moment of warning we get: (gdb) p debug_tree(type) unit size align 32 symtab 0 a

[Bug c/56980] Misleading note

2013-04-16 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56980 Manuel López-Ibáñez changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|

[Bug c/56980] Misleading note

2013-04-16 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56980 Manuel López-Ibáñez changed: What|Removed |Added Keywords||diagnostic --- Comment #3 from Man

  1   2   3   4   5   6   7   8   9   10   >