Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
In the following code x clearly is used uninitialized
#include
int main(void)
{
while (1)
{
int x;
printf
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81712
Jan Smets changed:
What|Removed |Added
CC||jan.smets at nokia dot com
--- Comment #7
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
--- Comment #17 from Jan Smets ---
Patch confirmed to work on the original code. Thanks.
Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
During compilation of libstdc++-v3 with -mlong-calls
Occurs on mips O32, N32, 64.
In both GCC5 and 6.
Target I used is a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79915
--- Comment #1 from Jan Smets ---
Sorry, copy/pasted incorrect libtool compile, it's the one of
libstdc++-v3/src/c++98/strstream.cc
Also, occurs at any optimisation level.
libtool: compile: /jasmets/git/tools/toolchains/gcc6/gcc-builddir/./gcc/
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79915
--- Comment #2 from Jan Smets ---
Reduced test case, only occurs on the vxworks port, not on linux.
namespace a {
template class c;
class d {
public:
virtual ~d();
};
template class e : d {};
template class c : virtual e {};
class g : c {
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79915
Jan Smets changed:
What|Removed |Added
Target|mips|mips64-linux-gnuabi64
Summary|ICE
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79915
--- Comment #4 from Jan Smets ---
Created attachment 40910
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40910&action=edit
testcase pr79951
mips64-linux-gnuabi64-gcc -xc++ pr79951.i -o /dev/null -S -mlong-calls
-mno-abicalls
++
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Created attachment 40959
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40959&action=edit
testcase
I have a unlucky code pattern that resulted in 10592 bytes stac
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80032
--- Comment #5 from Jan Smets ---
I think suggested patch might generate bad code. (but hard to track down on my
embedded target, it's stuck at a point where I don't have exceptions/backtraces
available yet).
Priority: P3
Component: preprocessor
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
-Wmisleading-indentation is basically disabled when compile jobs are
distributed using distcc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80032
--- Comment #9 from Jan Smets ---
The alternative patch to gimplify.c seems to run fine.
Thanks
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80032
--- Comment #12 from Jan Smets ---
Thanks. I've also been using the patch on the 6 branch and it seems to work
fine as well.
-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
On MIPS, O32
GCC 6.3.1 (does not happen on GCC5, but there it's not similarly optimized by
ipa-cp)
I wasn't yet successful at producing a minimized testcase.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
--- Comment #2 from Jan Smets ---
Created attachment 41245
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41245&action=edit
testcase pr80474
-O2 -fno-reorder-blocks -march=mips2 -fno-inline-small-functions
MIPS O32
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
--- Comment #4 from Jan Smets ---
For the reduced testcase :
$ mips64-linux-gnuabi64-gcc --version
mips64-linux-gnuabi64-gcc (Ubuntu 6.3.0-12ubuntu2) 6.3.0 20170406
mips64-linux-gnuabi64-gcc -O2 -fno-reorder-blocks -march=mips2
-fno-inline-sm
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
--- Comment #5 from Jan Smets ---
We get 'good' code on 7.1/trunk since
2016-04-29 Patrick Palka
tree-ssa-threadedge.c (simplify_control_stmt_condition): Split out into ...
simplify_control_stmt_condition_1): ... here. Rec
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
--- Comment #7 from Jan Smets ---
My apologies, somehow I dropped the -mno-abicalls along the road.
$ git status
HEAD detached at gcc-6_3_0-release
configure --target=mips64-linux-gnuabi64 --enable-languages=c (with
binutils/gmp/etc all alread
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
--- Comment #11 from Jan Smets ---
Thanks Eric
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Following ICE is seen :
x.cpp: In function 'void a()':
x.cpp:3: internal compiler error: in subspan, at input.h:69
3 | #define DB_PRINTF(str, fmt, args...) db_printf(
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96935
--- Comment #1 from Jan Smets ---
Proper backtrace (10.2)
x.cpp: In function ‘void a()’:
x.cpp:3: internal compiler error: in subspan, at input.h:69
3 | #define DB_PRINTF(str, fmt, args...) db_printf(indent_len, 50, fmt,
str, ##args)
|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96935
--- Comment #3 from Jan Smets ---
A bisect resulted in this commit :
commit 0d48e8779c6a9ac88f5efd1b4a2d40f43ef75faf
Author: David Malcolm
Date: Fri Oct 5 19:02:17 2018 +
Support string locations for C++ in -Wformat (PR c++/56856)
Priority: P3
Component: preprocessor
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Target: x86_64-linux-gnu
Configured with: /usr/src/gcc/configure --build=x86_64-linux-gnu
--disable-multilib --enable-languages=c,c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96391
Jan Smets changed:
What|Removed |Added
CC||jan.smets at nokia dot com
--- Comment #5
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96940
--- Comment #1 from Jan Smets ---
Likely duplicate of Bug 96391
(https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96391)
That one has a testcase for i686-w64-mingw32
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96940
--- Comment #2 from Jan Smets ---
This is the workaround I currently have. It avoids calling min_location().
diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 90111e4c786..f49019e81d0 100644
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -11005,8 +11
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Created attachment 49203
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49203&action=edit
testcase
Following testcase produces invalid code in 10.1, 10.2. Works with 9.3.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96993
--- Comment #3 from Jan Smets ---
The testcase has a sign-compare warning that we've traditionally been ignoring
given the ancient codebase.
warning: operand of ‘?:’ changes signedness from ‘int’ to ‘long unsigned int’
due to unsignedness of ot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96993
--- Comment #5 from Jan Smets ---
Hi Andrew
I agree that __builtin_offsetof would of good use here. However, I believe this
code predates the availability of the builtin (was written around the time we
were using gcc3.4)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96935
--- Comment #11 from Jan Smets ---
Thanks. Fix verified.
Severity: normal
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Created attachment 49256
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49256&acti
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97164
--- Comment #9 from Jan Smets ---
Thanks for the quick resolution everyone. Our codebase apparenty has a handful
of these issues.
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Created attachment 49458
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49458&action=edit
testcase
Following testcase produces a compile warning in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97617
--- Comment #2 from Jan Smets ---
Is it maybe a possibility to report the (possible) false positives with
something like -Waggressive-loop-optimizations=2 ?
Would that only require a skip of single_exit() in
do_warn_aggressive_loop_optimization
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97617
--- Comment #3 from Jan Smets ---
Sorry, I was too quickly in my wording to "skip single_exit()", of course that
edge is still required.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96391
--- Comment #10 from Jan Smets ---
I have a couple of changes in my own tree. I had a couple of different issues
and I don't recall exactly what change was for what specifically.
I locally have a revert of 0d48e8779c6a9ac88f5efd1b4a2d40f43ef75fa
Severity: normal
Priority: P3
Component: middle-end
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
int strcmpTEST(
const char * s1,
const char * s2)
{
while (*s1++ == *s2++)
if (*(s1-1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97902
--- Comment #2 from Jan Smets ---
Apologies, I omitted the -O1 / -O2
$ docker run --privileged --rm -it -v /tmp:/tmp gcc:10.2 bash -c "gcc -c
/tmp/test4.c -S -o - -O2 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer
| grep rbp"
$ docker r
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97902
Jan Smets changed:
What|Removed |Added
Status|RESOLVED|VERIFIED
--- Comment #6 from Jan Smets ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97902
--- Comment #13 from Jan Smets ---
H.J, There are still some very basic backtrace implementations that rely on
frame pointers. (No DWARF based things or any other forms of 'assistance'). A
missing stack frame means the "previous" function is not
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
As discussed in PR 97902 we'd need an option that disables all optimizations
that may affect the generation of a frame pointer.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97902
--- Comment #15 from Jan Smets ---
Thanks. See 98018.
Priority: P3
Component: sanitizer
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
CC: dodji at gcc dot gnu.org, dvyukov at gcc dot gnu.org,
jakub at gcc dot gnu.org, kcc at gcc dot gnu.org, marxin at
gcc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=103730
--- Comment #2 from Jan Smets ---
PEBKAC. Thanks for clarifying.
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
Created attachment 50840
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=50840&action=edit
scalar_storage_order te
Version: 11.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
This test case reports a override-init error when doing a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100671
--- Comment #1 from Jan Smets ---
Another one we've had problems with is quite similar. The example below is a
void function trying to return a value.
#if 1
/* NULL defined in a system header file
=> warning in "one shot" compilation.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100653
--- Comment #3 from Jan Smets ---
Is there some way there can be warned against such invalid usages? Because
these things are really hard to see on a 'macro' level.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70871
--- Comment #6 from Jan Smets ---
Sounds good. I appreciate the follow-up, thanks you for that.
: normal
Priority: P3
Component: target
Assignee: unassigned at gcc dot gnu.org
Reporter: jan.smets at nokia dot com
Target Milestone: ---
I noticed that the .file directive prints differently on targets (mips vs x86)
when (dwarf) debug symbols are enabled
50 matches
Mail list logo