https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64242
--- Comment #17 from Jeffrey A. Law ---
Or just emit a blockage insn to avoid the undesirable code motion.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=24639
Bug 24639 depends on bug 79768, which changed state.
Bug 79768 Summary: `-Wmaybe-uninitialized' false positive with optimisation
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79768
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79768
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #3 from Jeffrey A. Law ---
Fixed by Vlad's patch on the trunk.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #6 from Jeffrey A. Law ---
Fixed on trunk by Wilco's patch.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87539
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law ---
Fixed by Vlad's patch on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88246
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #7
||2018-11-27
CC||law at redhat dot com
Ever confirmed|0 |1
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #6 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58370
Bug 58370 depends on bug 58397, which changed state.
Bug 58397 Summary: Please add host_hooks for NetBSD to make precompiled headers
work
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58397
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87011
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87468
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87468
--- Comment #6 from Jeffrey A. Law ---
More correctly, jump threading is duplicating a switch in which there's an
outgoing edge marked as EDGE_IGNORE. That duplicate has EDGE_IGNORE set, but
isn't in the cleanup queue. Thus it leaks into the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52171
--- Comment #16 from Jeffrey A. Law ---
I think Qing did a fair amount of tuning on this work. If we're not handling a
particular case it's likely because it wasn't generally profitable.
||law at redhat dot com
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law ---
Should be fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87813
--- Comment #8 from Jeffrey A. Law ---
Aldy -- fixing that is a TODO for stage3.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70831
Jeffrey A. Law changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #1 from Jeffrey A. Law ---
Fixed on the trunk.
||2018-10-30
CC||law at redhat dot com
Assignee|unassigned at gcc dot gnu.org |law at redhat dot com
Ever confirmed|0 |1
--- Comment #2 from Jeffrey A. Law ---
I thought I already had a bug
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87664
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80155
--- Comment #44 from Jeffrey A. Law ---
I'd be very hesitant to make the cost model target specific. It goes against
core design goals of gimple.
Conceptually I believe we should be optimizing as much as possible on gimple
and that issues such
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #8 from Jeffrey A. Law ---
Patch committed on the trunk
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #8 from Jeffrey A. Law ---
I committed Jan's patch to the trunk.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #7 from Jeffrey A. Law ---
Fixed by Giuliano's patch I've installed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61409
--- Comment #31 from Jeffrey A. Law ---
No, it's marked as a regression against gcc-6. One of two things needs to
happen for this to be closed.
1. Aldy's work would have to be backported to gcc-6.
or
2. gcc-6 goes out of support
I don't
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87387
Jeffrey A. Law changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86540
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #2 from Jeffrey A. Law ---
Fixed by Bernd's patch on the trunk.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #9 from Jeffrey A. Law ---
Should be fixed on trunk now.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #6 from Jeffrey A. Law ---
Should be fixed on trunk now.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #4 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86010
--- Comment #9 from Jeffrey A. Law ---
We don't merge the two memset calls in the second example. It's of arguable
value IMHO.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87110
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87110
--- Comment #3 from Jeffrey A. Law ---
It's different, but related.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87110
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #2
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #23 from Jeffrey A. Law ---
Fixed by Martin's patch on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81958
--- Comment #3 from Jeffrey A. Law ---
I'd suggest leaving as a single bug for the Wuninitialized issue. There may be
other approaches to fixing it that are worth exploring. If we split into
multiple bugs for the various hunks of work DOM
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86996
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #9 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86897
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86897
--- Comment #2 from Jeffrey A. Law ---
Yea, the fix for this was in my tree when I was testing. The DOM changes
compromise the test. I'll push my fix to the trunk in a couple hours when I'm
back online.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #2 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86772
Bug 86772 depends on bug 86795, which changed state.
Bug 86795 Summary: mn10300 port needs updating for CVE-2017-5753
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86795
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86732
--- Comment #8 from Jeffrey A. Law ---
I wouldn't object to that. In fact I thought we kicked that around along with
an option to remove path leading to the undefined behavior completely. But
it's not something I'm likely to work on.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86732
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86732
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #5
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #2 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86772
Bug 86772 depends on bug 86790, which changed state.
Bug 86790 Summary: m68k port needs updating for CVE-2017-5753
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86790
What|Removed |Added
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #2 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86772
Bug 86772 depends on bug 86784, which changed state.
Bug 86784 Summary: H8 port needs updating for CVE-2017-5753
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86784
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69224
--- Comment #13 from Jeffrey A. Law ---
Agreed. I don't see a lot of value in backporting this fix to the release
branches. One could argue that decision means this should move to CLOSED as
it's been fixed for gcc-8 and the trunk.
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86010
--- Comment #4 from Jeffrey A. Law ---
Actually, alignment considerations are really just for head trimming. When
trimming the tail we can be more aggressive as residuals are usually handled
reasonably efficiently.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=86010
--- Comment #3 from Jeffrey A. Law ---
It's not an off by 1 error, but a conscious decision to not pass odd addresses
into the mem* routines, which are typically doing to emit crappy code for badly
aligned data.
We correctly see that 7 bytes of
||law at redhat dot com
Resolution|--- |INVALID
--- Comment #1 from Jeffrey A. Law ---
No, .align 128 is right AFAICT.
.align on the m68k is a byte alignment and requires its argument to be a power
of 2.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=55860
--- Comment #6 from Jeffrey A. Law ---
Doesn't matter much to me either way. I review all the jump threading PRs
every release.
-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: law at redhat dot com
Target Milestone: ---
As outlined in this discussion of one of Kugan's patches, DOM does not handle
statements like
x = COND ? val1 : val2;
When COND has a known result.
The discussion
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79986
--- Comment #11 from Jeffrey A. Law ---
I believe it's done. So all the CHKP/MPX bugs can be CLOSED/WONTFIX.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66240
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84342
--- Comment #12 from Jeffrey A. Law ---
I'm not familiar enough with the ccfsm bits to know if there's something we
ought to be doing generically to improve CC handling further. I think
downgrading to P2 certainly makes sense though.
However,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85964
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #13
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85478
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #11
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84923
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #4 from Jeffrey A. Law ---
Fixed on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77485
Bug 77485 depends on bug 33562, which changed state.
Bug 33562 Summary: [6 Regression] aggregate DSE disabled
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33562
What|Removed |Added
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=33562
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85259
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
Target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85301
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #7
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85301
--- Comment #5 from Jeffrey A. Law ---
FWIW, there's another similar bug where DOM doesn't do a particularly good job
at tracking the state of objects implied the results of logical operations
which in turn causes missed optimizations. I've
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85186
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #6
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79958
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84919
Jeffrey A. Law changed:
What|Removed |Added
Target Milestone|8.0 |9.0
--- Comment #13 from Jeffrey A.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84933
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84919
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #5 from Jeffrey A. Law ---
Fixed by Vlad's commit on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70359
--- Comment #36 from Jeffrey A. Law ---
WRT the division removal. That seems so profitable that a slight increase in
codesize is warranted. So if we fix the other issue and the source of the
remaining codesize regressions is the removal of
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #12 from Jeffrey A. Law ---
Fixed by Martin's patch on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84682
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84670
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #5
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84515
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #4
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84646
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #2
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70359
--- Comment #27 from Jeffrey A. Law ---
Yea, we went through similar issues with Alex's work recently. After that work
landed I went through all the BZs that I could find with autoinc
{pre,post}{inc,dec} that BZ could find in the hopes that his
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57534
Jeffrey A. Law changed:
What|Removed |Added
Target Milestone|6.5 |9.0
--- Comment #30 from Jeffrey A.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57534
--- Comment #28 from Jeffrey A. Law ---
BTW, ISTM that we need Bin to chime in on the complexity of improving this in
IVOPTS -- ie, is it gcc-8 or gcc-9 material. If the latter, then we should
adjust the target milestone.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57534
--- Comment #27 from Jeffrey A. Law ---
WRT c#21. There is a good paper from Click on an integrated GVN/GCM/reassoc.
"Combining Analyses, Combining Optimizations" It was published in PLDI. I've
probably got a copy here if you want it.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66433
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66433
Jeffrey A. Law changed:
What|Removed |Added
CC||law at redhat dot com
--- Comment #3
||law at redhat dot com
Resolution|--- |FIXED
--- Comment #7 from Jeffrey A. Law ---
Fixed by Ramana's patch on the trunk.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57534
--- Comment #18 from Jeffrey A. Law ---
A couple more notes. It could also well be the case that reassociating in a
way that encourages lea could be good for x86, but bad for other targets.
I also suspect this is closely related to other BZs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=57534
--- Comment #17 from Jeffrey A. Law ---
It could well end up being a case where we need to look to see if the
expressions are likely to CSE to determine which is better.
I'm not sure if reassoc has that kind of capability.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83496
--- Comment #33 from Jeffrey A. Law ---
No. THe one I'm currently chasing is not 84545. I'm chasing a ton of ICEs due
to debug insns appearing in places we didn't expect.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83496
--- Comment #30 from Jeffrey A. Law ---
Just a note. I'm tracking a separate problem with delay slot filling that
looks like it's related to handling of debug insns. I doubt it's the same
problem, but if you stumble over it, be aware I'm
||law at redhat dot com
at gcc dot gnu.org |law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=21161
Jeffrey A. Law changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61118
--- Comment #23 from Jeffrey A. Law ---
Note that if we fix the abnormal handler to target the block after the setjmp
rather than the setjmp itself all the problems magically disappear. That's
actually a more accurate CFG and arguably the right
||law at redhat dot com
Resolution|DUPLICATE |---
--- Comment #6 from Jeffrey A. Law ---
The warning on a2 is clearly bogus. ISTM we ought to get a warning on fd, but
don't because it's associated DECL_RTL is not set.
It almost feels like instead
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61118
Jeffrey A. Law changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|DUPLICATE
501 - 600 of 3054 matches
Mail list logo