https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81897
Jeffrey A. Law changed:
What|Removed |Added
Summary|[6/7/8 Regression] spurious |[6/7 Regression] spurious
||law at redhat dot com
Resolution|--- |INVALID
--- Comment #20 from Jeffrey A. Law ---
Based on c#18.
||law at redhat dot com
||law at redhat dot com
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #8 from Jeffrey A. Law ---
Fixed by Martin's patch on the trunk.
||law at redhat dot com
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #8 from Jeffrey A. Law ---
Should be fixed by Richard's patch on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79224
Jeffrey A. Law changed:
What|Removed |Added
Summary|[7/8 Regression] Large |[7 Regression] Large C-Ray
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83641
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83654
Jeffrey A. Law changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81611
--- Comment #14 from Jeffrey A. Law ---
MEM_REF (as opposed to TARGET_MEM_REF) should be able to handle any simple
SSA_NAME + CONSTANT_OFFSET which are all we're really concerned with here. THe
target's addressing modes don't really enter the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83620
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #5
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61118
--- Comment #21 from Jeffrey A. Law ---
No problem Eric. I'm monitoring on behalf of Florian who'd really like to see
this fixed for gcc-8.
Actually just noticed it still wasn't showing up in the queries. It didn't
have a target milestone set
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=21161
Jeffrey A. Law changed:
What|Removed |Added
Target Milestone|--- |8.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81611
--- Comment #12 from Jeffrey A. Law ---
You know, I wonder if we're missing something bigger here.
ISTM we're potentially missing CSEs in memory addresses as well as forward
propagation opportunities in MEM_REF expressions.
I strongly suspect
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61118
--- Comment #19 from Jeffrey A. Law ---
Note you lost the regression marker when this was made a duplicate of 21161.
So it's unlikely anyone would have looked at it until the next release cycle.
My understanding from Florian is that at least
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83565
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #27
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83641
--- Comment #3 from Jeffrey A. Law ---
So the issue here is when we have a noreturn function we use a push/pop
sequence to probe the top of the stack.
The generic dwarf2 CFI bits interpret the pop as restoring the value of the
popped register.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83641
--- Comment #2 from Jeffrey A. Law ---
Note this is specific to x86/x86_64 noreturn functions. No other targets are
potentially affected.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83641
Jeffrey A. Law changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83641
Jeffrey A. Law changed:
What|Removed |Added
Priority|P3 |P1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83438
--- Comment #9 from Jeffrey A. Law ---
My bisections keep landing on:
commit bb173647d8221f86812f4e98942960b894e9e972 (HEAD, refs/bisect/bad)
Author: hubicka
Date: Thu Nov 23 15:56:28 2017 +
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81897
--- Comment #8 from Jeffrey A. Law ---
This really looks like something tree-ssa-uninit.c ought to be handling too.
[ Just to be clear we should fix both tree-ssa-uninit.c the cfgcleanup. ]
We have a PHI with a default definition on the RHS:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81897
--- Comment #7 from Jeffrey A. Law ---
So just a note. We *could* pick this up without waiting on Aldy's work.
After the second DOM pass we're failing to merge a pair of blocks because there
are still SSA_NAMEs queued for renaming. If we were
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82666
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82764
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79224
Jeffrey A. Law changed:
What|Removed |Added
CC||aldyh at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79405
Jeffrey A. Law changed:
What|Removed |Added
CC||aldyh at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82682
--- Comment #7 from Jeffrey A. Law ---
One thought would be to realize that the copy/extend sequence seen in this case
is special. It's going to be used when the input for the extension can't be
used in an extension (ie %esi in this case). ie,
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #8 from Jeffrey A. Law ---
Fixed by Wilco's patch on the trunk.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #8 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82682
--- Comment #6 from Jeffrey A. Law ---
Of course we can't propagate because %esi doesn't have a QImode equivalent.
So to back up a little bit. The costing changes twiddled the register
assignments. In particular reg93 gets allocated into %esi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81611
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82682
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #5
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #6 from Jeffrey A. Law ---
Per c#5.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #6 from Jeffrey A. Law ---
Per c#5.
||law at redhat dot com
Resolution|--- |WORKSFORME
--- Comment #8 from Jeffrey A. Law ---
Given c#7.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #12 from Jeffrey A. Law ---
Fixed by Segher's patch on the trunk.
,
||law at redhat dot com
--- Comment #2 from Jeffrey A. Law ---
Aldy, want to run with this one? I think the libcpp patch itself is obvious.
You just need to wire up a testcase...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81949
Jeffrey A. Law changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83477
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83438
--- Comment #8 from Jeffrey A. Law ---
Doesn't look to be the same correctness issue I'm tracking right now as I get a
mis-compare with and without those changes. Sigh.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83438
--- Comment #7 from Jeffrey A. Law ---
It could well be the same problem. The fix for the codegen bug I'm tracking
affects relax_sh.c from 435.gromacs. I'm still investigating.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83438
--- Comment #5 from Jeffrey A. Law ---
Richi,
I've got a code correctness issue I'm looking at with those changes. If you
could pass along the .dom2 and .dom3 dumps for the 435.gromacs benchmark I
could probably scan them for the issue without
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80520
Jeffrey A. Law changed:
What|Removed |Added
CC||rguenth at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83477
--- Comment #4 from Jeffrey A. Law ---
I see it. Fix spinning while I sleep.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83477
--- Comment #3 from Jeffrey A. Law ---
It's another VR_UNDEFINED sneaking through.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80198
--- Comment #12 from Jeffrey A. Law ---
Given that B and A have the same cost to compute, DOM, per design decision,
leaves them alone. It makes no attempt to canonicalize on one or the other.
While we know there's an equivalence between A and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83438
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83460
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83460
--- Comment #6 from Jeffrey A. Law ---
Actually the 79095-4 test is testing that we do loop distribution and
propagation of constants into the the memset call and that after propagation we
realize there's a bogus path that we actually can't
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83239
--- Comment #20 from Jeffrey A. Law ---
Yes. Very much want to keep this open -- I've got a patch for the missed
optimization, but need to recover the tests I'd written and somehow lost before
submitting.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83460
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
||law at redhat dot com
Resolution|--- |DUPLICATE
--- Comment #1 from Jeffrey A. Law ---
ALmost certainly a DUP.
*** This bug has been marked as a duplicate of bug 82965 ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965
Jeffrey A. Law changed:
What|Removed |Added
CC||ro at gcc dot gnu.org
--- Comment #1
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #9 from Jeffrey A. Law ---
This was fixed back in July:
commit f0f5171608d68c3cb3aa6aa43d64814d4f9d67d5
Author: krebbel <krebbel@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Tue Jul
||law at redhat dot com
--- Comment #4 from Jeffrey A. Law ---
MPX/CHKP is likely to be deprecated. Moving to P4.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #6 from Jeffrey A. Law ---
Markus has retuned the div/mod costing model and I've committed Sebastian's
change to use -mtune=generic on this test.
||2017-12-16
CC||law at redhat dot com
Ever confirmed|0 |1
--- Comment #2 from Jeffrey A. Law ---
Still happens on the trunk r255606.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law ---
I went ahead and committed Jan's patch to the trunk.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #3 from Jeffrey A. Law ---
This was fixed on the trunk by Jan back in Oct:
commit 4b57298d4731a37c50ab145bc766a926a98cebf0
Author: hubicka <hubicka@138bc75d-0d04-0410-961f-82ee72b05
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81240
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law ---
Fixed by Markus's patch on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78394
--- Comment #7 from Jeffrey A. Law ---
-O0 has none of the analysis necessary and I believe you get no warnings at
all.
A minimum of -Og is needed, but -Og is inherently going to give many false
positives.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78394
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #5
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639
Bug 24639 depends on bug 69026, which changed state.
Bug 69026 Summary: dwarf2out.c:4295 warning:
‘finder[...]addr_table_entry_struct_union::label’ may be used uninitialized
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69026
What
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #3 from Jeffrey A. Law ---
Per c#2.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61409
--- Comment #29 from Jeffrey A. Law ---
Nevermind my last comment. Totally wrong. This is fixed on the trunk,
totally and completely by Aldy's changes.
The trunk changed in that it can thread the jump now because we'll be under
growth
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61409
--- Comment #28 from Jeffrey A. Law ---
So this is "fixed" on the trunk.
The trunk now has the ability to track statements that will likely become dead
code as a result of jump threading. That's enough to get the provided samples
under the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639
Bug 24639 depends on bug 42145, which changed state.
Bug 42145 Summary: bogus "may be used uninitialized" (a || b converted to a|b)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42145
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=42145
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=20968
Bug 20968 depends on bug 36550, which changed state.
Bug 36550 Summary: Wrong "may be used uninitialized" warning (conditional PHIs)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36550
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639
Bug 24639 depends on bug 36550, which changed state.
Bug 36550 Summary: Wrong "may be used uninitialized" warning (conditional PHIs)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36550
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=36550
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81165
Jeffrey A. Law changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19794
Bug 19794 depends on bug 81165, which changed state.
Bug 81165 Summary: [8 Regression] Regression in GCC-8.0.0's optimizer
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81165
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83410
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80520
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83410
--- Comment #5 from Jeffrey A. Law ---
So to record my current thoughts.
There's two competing needs here. We sometimes want to thread as the
simplifications can enable vectorization. Other times we do not want to thread
because threading
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83410
--- Comment #4 from Jeffrey A. Law ---
And of course there's cases where we depend on threading similar situations to
enable vectorization. I loathe the idea of checking for graphite to determine
whether or not to thread these edges, it feels
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83410
Jeffrey A. Law changed:
What|Removed |Added
Priority|P3 |P2
--- Comment #3 from Jeffrey A. Law
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83410
--- Comment #2 from Jeffrey A. Law ---
Not failing for me. Oh wait. This depends on graphite? I don't typically
build with graphite. Let me restart...
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82123
Jeffrey A. Law changed:
What|Removed |Added
CC||vincent-gcc at vinc17 dot net
---
||law at redhat dot com
Resolution|--- |DUPLICATE
--- Comment #9 from Jeffrey A. Law ---
So the fix for pr82123 will fix the original bug report, which isn't a surprise
because at the core they're both problems with a lack of context sensitive
||law at redhat dot com
Resolution|--- |DUPLICATE
--- Comment #3 from Jeffrey A. Law ---
I'm going mark this as a DUP of 82123. Ultimately the problem here is a lack
of good range information in gimple-ssa-sprintf which is addressed by the same
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82123
--- Comment #4 from Jeffrey A. Law ---
*** Bug 81592 has been marked as a duplicate of this bug. ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83298
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83383
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83298
Jeffrey A. Law changed:
What|Removed |Added
CC||babokin at gmail dot com
--- Comment
,
||law at redhat dot com
--- Comment #3 from Jeffrey A. Law ---
Adding David who knows the linemap bits.
David -- Mike has also posted a patch. See gcc-patches archives Dec 1.
|NEW
Last reconfirmed||2017-12-11
CC||law at redhat dot com
Ever confirmed|0 |1
--- Comment #3 from Jeffrey A. Law ---
Can't see how ppc-spe can be P1 :-) BUt will certainly
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=21182
--- Comment #27 from Jeffrey A. Law ---
So for gcc-8.
UNAIL_REGS -O3 41 stack references
UNAIL_REGS -O2 41 stack referenecs
DNAIL_REGS -O3 3 stack references
DNAIL_REGS -O2 3 stack references
In the DNAIL_REGS case I think all three
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80747
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #5
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #1 from Jeffrey A. Law ---
I think this was fixed by Eric B's fixes to bb-reorder a couple months ago.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #2 from Jeffrey A. Law ---
This was fixed by Eric's Oct 27 change to bb-reorder on the trunk.
||law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83239
--- Comment #14 from Jeffrey A. Law ---
You *really* don't want to do this with a match.pd pattern. Recovering the
arith-with-overflow optimizations is virtually impossible in the general case.
All that really needs to be done here is
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #1 from Jeffrey A. Law ---
The combiner now removes the redundant extension on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66726
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #2 from Jeffrey A. Law ---
Eric fixed this as part of the compare-elim revamp last year.
701 - 800 of 3054 matches
Mail list logo