public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug ipa/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox
[not found] <bug-88702-4@http.gcc.gnu.org/bugzilla/>
@ 2020-12-01 10:47 ` cvs-commit at gcc dot gnu.org
2020-12-01 11:14 ` marxin at gcc dot gnu.org
` (2 subsequent siblings)
3 siblings, 0 replies; 4+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-12-01 10:47 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702
--- Comment #12 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:
https://gcc.gnu.org/g:03eb09292ef228d1d12b5168cdd748583b1f992a
commit r11-5605-g03eb09292ef228d1d12b5168cdd748583b1f992a
Author: Martin Liska <mliska@suse.cz>
Date: Fri Aug 28 10:26:13 2020 +0200
Add if-chain to switch conversion pass.
gcc/ChangeLog:
PR tree-optimization/14799
PR ipa/88702
* Makefile.in: Add gimple-if-to-switch.o.
* dbgcnt.def (DEBUG_COUNTER): Add new debug counter.
* passes.def: Include new pass_if_to_switch pass.
* timevar.def (TV_TREE_IF_TO_SWITCH): New timevar.
* tree-pass.h (make_pass_if_to_switch): New.
* tree-ssa-reassoc.c (struct operand_entry): Move to the header.
(dump_range_entry): Move to header file.
(debug_range_entry): Likewise.
(no_side_effect_bb): Make it global.
* tree-switch-conversion.h (simple_cluster::simple_cluster):
Add inline for couple of functions in order to prevent error
about multiple defined symbols.
* gimple-if-to-switch.cc: New file.
* tree-ssa-reassoc.h: New file.
gcc/testsuite/ChangeLog:
PR tree-optimization/14799
PR ipa/88702
* gcc.dg/tree-ssa/pr96480.c: Disable if-to-switch conversion.
* gcc.dg/tree-ssa/reassoc-32.c: Likewise.
* g++.dg/tree-ssa/if-to-switch-1.C: New test.
* gcc.dg/tree-ssa/if-to-switch-1.c: New test.
* gcc.dg/tree-ssa/if-to-switch-2.c: New test.
* gcc.dg/tree-ssa/if-to-switch-3.c: New test.
* gcc.dg/tree-ssa/if-to-switch-4.c: New test.
* gcc.dg/tree-ssa/if-to-switch-5.c: New test.
* gcc.dg/tree-ssa/if-to-switch-6.c: New test.
* gcc.dg/tree-ssa/if-to-switch-7.c: New test.
* gcc.dg/tree-ssa/if-to-switch-8.c: New test.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug ipa/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox
[not found] <bug-88702-4@http.gcc.gnu.org/bugzilla/>
2020-12-01 10:47 ` [Bug ipa/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox cvs-commit at gcc dot gnu.org
@ 2020-12-01 11:14 ` marxin at gcc dot gnu.org
2020-12-02 7:21 ` [Bug tree-optimization/88702] " cvs-commit at gcc dot gnu.org
2020-12-02 7:38 ` marxin at gcc dot gnu.org
3 siblings, 0 replies; 4+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-12-01 11:14 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702
--- Comment #13 from Martin Liška <marxin at gcc dot gnu.org> ---
All right, this test-case will need one more small tweak.
I've got a patch candidate.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug tree-optimization/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox
[not found] <bug-88702-4@http.gcc.gnu.org/bugzilla/>
2020-12-01 10:47 ` [Bug ipa/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox cvs-commit at gcc dot gnu.org
2020-12-01 11:14 ` marxin at gcc dot gnu.org
@ 2020-12-02 7:21 ` cvs-commit at gcc dot gnu.org
2020-12-02 7:38 ` marxin at gcc dot gnu.org
3 siblings, 0 replies; 4+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2020-12-02 7:21 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702
--- Comment #14 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Martin Liska <marxin@gcc.gnu.org>:
https://gcc.gnu.org/g:c961e94901eb793b1a18d431a1acf7f682eaf04f
commit r11-5642-gc961e94901eb793b1a18d431a1acf7f682eaf04f
Author: Martin Liska <mliska@suse.cz>
Date: Tue Dec 1 12:18:46 2020 +0100
if-to-switch: Support chain with 2 BBs.
As seen in the test-case, even 2 BBs can handle interesting
cases covered by a jump table or a bit-test.
gcc/ChangeLog:
PR tree-optimization/88702
* gimple-if-to-switch.cc (pass_if_to_switch::execute):
Require at least 2 BBs.
* gimple-if-to-switch.cc (find_conditions): Require
equal precision for low and high of a range.
gcc/testsuite/ChangeLog:
PR tree-optimization/88702
* gcc.dg/tree-ssa/if-to-switch-9.c: New test.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug tree-optimization/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox
[not found] <bug-88702-4@http.gcc.gnu.org/bugzilla/>
` (2 preceding siblings ...)
2020-12-02 7:21 ` [Bug tree-optimization/88702] " cvs-commit at gcc dot gnu.org
@ 2020-12-02 7:38 ` marxin at gcc dot gnu.org
3 siblings, 0 replies; 4+ messages in thread
From: marxin at gcc dot gnu.org @ 2020-12-02 7:38 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702
Martin Liška <marxin at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #15 from Martin Liška <marxin at gcc dot gnu.org> ---
Implemented on the current master.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-12-02 7:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <bug-88702-4@http.gcc.gnu.org/bugzilla/>
2020-12-01 10:47 ` [Bug ipa/88702] [8/9/10/11 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox cvs-commit at gcc dot gnu.org
2020-12-01 11:14 ` marxin at gcc dot gnu.org
2020-12-02 7:21 ` [Bug tree-optimization/88702] " cvs-commit at gcc dot gnu.org
2020-12-02 7:38 ` marxin at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).