https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80842
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
Status|NEW
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80852
--- Comment #1 from Marc Glisse ---
I don't think the test on x%2 is relevant, gcc eliminates it without problem. I
thought gcc had some code to recognize 1+2+3+...+n and similar patterns, but
apparently not this one.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80841
--- Comment #2 from Jason Bell ---
Thanks that's a good reduced example. I've changed it slightly so it works
with constexpr input.
//#
template
struct A {};
template
struct B {};
template
struct B>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80842
--- Comment #3 from Richard Biener ---
Hmm, goes CONSTANT -> UNDEFINED. We've been here before I think.
Visiting statement:
_7 = (int) f.7_6;
which is likely UNDEFINED
Lattice value changed to UNDEFINED. Adding SSA edges to worklist.
marking s
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80808
--- Comment #2 from Jakub Jelinek ---
Smaller testcase that still works at -O1 and fails at -O2 (+ -march=armv7-a
-mfpu=vfpv3-d16 -mfloat-abi=hard in both cases):
static __attribute__ ((noinline, noclone)) unsigned
mpihelp_divrem (unsigned *qp,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80808
--- Comment #3 from Andrew Pinski ---
Can you try adding "flags" as a clobber on those inline-asm. I suspect that
might causing something here.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80830
Richard Biener changed:
What|Removed |Added
Keywords||ice-on-valid-code
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80831
Richard Biener changed:
What|Removed |Added
Keywords||ice-on-valid-code
Priority|P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80832
Richard Biener changed:
What|Removed |Added
Keywords||documentation
Severity|normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80833
Richard Biener changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78204
Martin Liška changed:
What|Removed |Added
Target Milestone|--- |8.0
--- Comment #5 from Martin Liška ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80852
--- Comment #2 from Raphael C ---
You are quite right. This code shows the same issue:
int foo(int num) {
int a = 0;
for (int x = 0; x < num; x+=2) {
a += x;
}
return a;
}
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80851
DB changed:
What|Removed |Added
CC||db0451 at gmail dot com
--- Comment #1 from DB ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80808
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67054
DB changed:
What|Removed |Added
CC||db0451 at gmail dot com
--- Comment #3 from DB ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80841
--- Comment #3 from Daniel Krügler ---
(In reply to Jason Bell from comment #2)
> Thanks that's a good reduced example. I've changed it slightly so it works
> with constexpr input.
But that is just adding additional complexity: The constexpr de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80822
--- Comment #1 from Jakub Jelinek ---
What do you get with OMP_PLACES=threads OMP_DISPLAY_ENV=verbose (with both
libgomp and Intel libomp)?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80822
--- Comment #2 from Jakub Jelinek ---
Note, the OpenMP description of OMP_PLACES=threads is:
"Each place corresponds to a single hardware thread on the target machine."
which is what GCC implements. Perhaps ICC implements the same, but orders in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80853
Bug ID: 80853
Summary: [6/7/8 Regression] OpenMP ICE in build_outer_var_ref
with array reduction
Product: gcc
Version: 7.1.1
Status: UNCONFIRMED
Severity: norma
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80853
Jakub Jelinek changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80854
Bug ID: 80854
Summary: hot path is slowed down when the cold return path is
merged into it
Product: gcc
Version: 7.1.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80841
--- Comment #4 from Jason Bell ---
(In reply to Daniel Krügler from comment #3)
> (In reply to Jason Bell from comment #2)
> > Thanks that's a good reduced example. I've changed it slightly so it works
> > with constexpr input.
>
> But that is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80830
Nathan Sidwell changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80850
Thomas Koenig changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=74563
Maciej W. Rozycki changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80809
--- Comment #3 from Jakub Jelinek ---
Created attachment 41400
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41400&action=edit
gcc8-pr80809.patch
Fix for the incorrect implicit determination. The bug was that we turned the
(implicit in t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80830
--- Comment #5 from Nathan Sidwell ---
Author: nathan
Date: Mon May 22 12:05:41 2017
New Revision: 248329
URL: https://gcc.gnu.org/viewcvs?rev=248329&root=gcc&view=rev
Log:
gcc/testsuite/
PR c++/80830
* g++.dg/lookup/frie
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80835
--- Comment #2 from Jonathan Wakely ---
You've reported this against libstdc++ but there's no way the library code can
possibly transform p->load(std::memory_order_acquire).ptr into loading a
subobject. std::atomic::load has no idea what you plan
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80855
Bug ID: 80855
Summary: [nvptx] missing sorry("target cannot support label
values"
Product: gcc
Version: 8.0
Status: UNCONFIRMED
Severity: normal
Prior
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80855
--- Comment #1 from Tom de Vries ---
Created attachment 41401
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41401&action=edit
tentative patch
Using this tentative patch, we get:
...
test.c: In function ‘main’:
test.c:9:11: sorry, unimplem
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79759
--- Comment #2 from Tom Honermann ---
This looks to be directly related to the following reports:
- Bug 80746 - [concepts] ICE evaluating constraints for concepts with dependent
template parameters
- Bug 67147 - [concepts] ICE on checking concept
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80817
--- Comment #3 from Joost VandeVondele
---
If I compile with -m32
gcc -std=c++11 -m32 -S -O3 test.cpp
I get
.cfi_startproc
subl$12, %esp
.cfi_def_cfa_offset 16
movl16(%esp), %ecx
fildq (%ecx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80856
Bug ID: 80856
Summary: ICE from template local overload resolution
Product: gcc
Version: 7.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80759
--- Comment #13 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #10 from Daniel Santos ---
[...]
> Anyway, if you can test it again for me and let me know what you think I would
> appreciate it. I've got some other code formatting cha
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80856
Markus Trippelsdorf changed:
What|Removed |Added
Keywords||ice-on-valid-code
Stat
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80725
--- Comment #3 from Andreas Krebbel ---
process_address_1 does not do any address reloading because it only checks the
first constraint letter whether it is an extra address constraint or not. In
our case ("a,ZR") it is a register constraint. Hen
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80857
Bug ID: 80857
Summary: slow compare_exchange_weak with unintegral type
Product: gcc
Version: 7.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80856
Markus Trippelsdorf changed:
What|Removed |Added
CC||jason at gcc dot gnu.org
--- Comme
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80858
Bug ID: 80858
Summary: When trying to copy std::unordered_map illegally,
error message doesn't tell what's wrong
Product: gcc
Version: 7.1.0
Status: UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80759
--- Comment #14 from ro at CeBiTec dot Uni-Bielefeld.DE ---
> --- Comment #9 from Daniel Santos ---
[...]
>> sure, though there's no need at all (except for the .struct part) to do
>> the testing on Solaris. I believe there are ready-made Solar
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80759
--- Comment #15 from Rainer Orth ---
Created attachment 41404
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41404&action=edit
Switch ms-sysv to more regular dg functions
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80835
--- Comment #3 from Peter Cordes ---
(In reply to Jonathan Wakely from comment #2)
> You've reported this against libstdc++
I had to take a guess at the right component, based on a couple other
std::atomic bugs I looked at. Apparently I picked
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80817
--- Comment #4 from Alexander Monakov ---
On 32-bit x86 manipulating 64-bit integers, let alone atomically, is going to
be inconvenient. The emitted code could have been shorter, instead of
movl(%esp), %eax
movl4(%esp),
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79801
Pat Haugen changed:
What|Removed |Added
CC||pthaugen at gcc dot gnu.org
--- Comment #1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80116
--- Comment #3 from Marek Polacek ---
A testcase:
#define SWAP(x, y) \
tmp = x; \
x = y; \
y = tmp
int x, y, tmp;
void
fn1 (void)
{
if (x)
SWAP(x, y); // warn
}
void
fn2 (void)
{
SWAP(x, y);
}
void
fn3 (void)
{
if (x)
{
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80835
Jonathan Wakely changed:
What|Removed |Added
Component|libstdc++ |c++
Severity|normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80858
--- Comment #1 from Jonathan Wakely ---
(In reply to sgunderson from comment #0)
> Using gcc version 7.1.0 (Debian 7.1.0-5) (but the error goes back to at
> least 4.8, and amazingly, also in Clang), on this piece of code, simplified
> from a much
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80803
--- Comment #2 from Bill Schmidt ---
I've verified that this only happens with a bootstrapped compiler. A one-pass
build does not produce the problem. The output from "cat net/check-testlog"
for such a build is:
PASS
PASS: net
Ian, I am not
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78379
Thomas Koenig changed:
What|Removed |Added
Attachment #40120|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78379
--- Comment #32 from Thomas Koenig ---
Created attachment 41406
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41406&action=edit
Additional files for the previous patch
Here are the new files for the patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80766
--- Comment #11 from janus at gcc dot gnu.org ---
Author: janus
Date: Mon May 22 17:08:24 2017
New Revision: 248341
URL: https://gcc.gnu.org/viewcvs?rev=248341&root=gcc&view=rev
Log:
2017-05-22 Janus Weil
PR fortran/80766
* re
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70268
Tom Rini changed:
What|Removed |Added
CC||tom.rini at gmail dot com
--- Comment #8 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80858
--- Comment #2 from sgunderson at bigfoot dot com ---
Yes, I mean that the error message isn't clear (and it's basically the same
error message in 4.8, so no regression).
I don't think I understand the difficulties involved. Doesn't the error com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80858
Jonathan Wakely changed:
What|Removed |Added
Keywords||diagnostic
Status|UNCONFIR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80803
--- Comment #3 from Bill Schmidt ---
Looks like this arises from this code in libgo/Makefile.in:
if $(SHELL) $(srcdir)/testsuite/gotest --goarch=$(GOARCH)
--goos=$(GO\
OS) --basedir=$(srcdir) --srcdir=$(srcdir)/go/$(@D) --pkgpath="$(@D
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80803
--- Comment #4 from Bill Schmidt ---
And gotest is just a bash script, so "something that it invokes" is the
problem...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80853
--- Comment #2 from Jakub Jelinek ---
Author: jakub
Date: Mon May 22 18:51:54 2017
New Revision: 248344
URL: https://gcc.gnu.org/viewcvs?rev=248344&root=gcc&view=rev
Log:
PR middle-end/80853
* omp-low.c (lower_reduction_clauses):
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80809
--- Comment #4 from Jakub Jelinek ---
Author: jakub
Date: Mon May 22 18:54:05 2017
New Revision: 248345
URL: https://gcc.gnu.org/viewcvs?rev=248345&root=gcc&view=rev
Log:
PR middle-end/80809
* gimplify.c (omp_add_variable): For G
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80809
--- Comment #5 from Jakub Jelinek ---
Author: jakub
Date: Mon May 22 18:54:54 2017
New Revision: 248346
URL: https://gcc.gnu.org/viewcvs?rev=248346&root=gcc&view=rev
Log:
PR middle-end/80809
* omp-low.c (finish_taskreg_remap): Ne
an 8 byte load, not a function-call + CMPXCHG16B.
void cas_compiler(atomic_uint_fast64_t *p) {
*p |= 3; // separate 32-bit loads before loop
}
#gcc8 snapshot 20170522 -m32, and clang does the same
... intro, including including pushing 4 regs, two of which aren't used
:(
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80474
Eric Botcazou changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|2017-04-21 00
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80833
--- Comment #6 from Peter Cordes ---
(In reply to Richard Biener from comment #5)
> There's some related bugs. I think there is no part of the compiler that
> specifically tries to avoid store forwarding issues.
Ideally the compiler would keep
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80835
--- Comment #5 from Peter Cordes ---
Previous godbolt link was supposed to be: https://godbolt.org/g/78kIAl
which includes the CAS functions.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80859
Bug ID: 80859
Summary: Performance Problems with OpenMP 4.5 support
Product: gcc
Version: 6.3.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80860
Bug ID: 80860
Summary: AIX Bootstrap failure
Product: gcc
Version: 8.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: bootstrap
Assignee:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80861
Bug ID: 80861
Summary: ARM (VFPv3): Inefficient float-to-char conversion goes
through memory
Product: gcc
Version: 8.0
Status: UNCONFIRMED
Severity: normal
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80834
--- Comment #3 from Michael Meissner ---
It looks like -fvect-cost-model=unlimited seems to re-enable vectorization with
-mcpu=power9. It also enables some vectorizaton with -mcpu=power7.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80718
--- Comment #3 from Michael Meissner ---
Author: meissner
Date: Mon May 22 22:44:45 2017
New Revision: 248352
URL: https://gcc.gnu.org/viewcvs?rev=248352&root=gcc&view=rev
Log:
[gcc]
2017-05-22 Michael Meissner
PR target/80718
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80803
Ian Lance Taylor changed:
What|Removed |Added
CC||ian at airs dot com
--- Comment #5 fr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66313
--- Comment #15 from Dmitry Babokin ---
The bug is almost 2 years old. I consider it's quite important, as false
positives make UBSAN not usable on any large codebases.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80858
--- Comment #4 from sgunderson at bigfoot dot com ---
I think this should work as reduction:
struct Empty {
};
template
struct A
{
A &operator=(const A&)
{
T t(3);
return *this;
}
}
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=80848
Alexey Neyman changed:
What|Removed |Added
CC||stilor at att dot net
--- Comment #1 fro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80848
--- Comment #2 from Alexey Neyman ---
It seems to be a duplicate of
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037 that has a proposed patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80809
--- Comment #6 from Jakub Jelinek ---
Fixed on the trunk so far.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80725
--- Comment #4 from Jakub Jelinek ---
1), if it works, looks easiest to me. 3) if Vlad would prefer that way.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80037
Alexey Neyman changed:
What|Removed |Added
CC||stilor at att dot net
--- Comment #2 fro
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80837
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80840
Richard Biener changed:
What|Removed |Added
Priority|P3 |P2
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80843
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |8.0
80 matches
Mail list logo
