[Bug ipa/88702] [7/8/9/10 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2019-10-31
   Assignee|unassigned at gcc dot gnu.org  |marxin at gcc dot 
gnu.org
   Target Milestone|7.5 |8.5
 Ever confirmed|0   |1

[Bug ipa/88702] [7/8/9/10 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox

[egallager at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702

--- Comment #11 from Eric Gallager  ---
(In reply to Martin Liška from comment #10)
> (In reply to David Malcolm from comment #9)
> > If using a switch is better than a series of tests against constants, would
> > it make sense for the compiler to spot this case, and automatically convert
> > the conditions to a switch?
> 
> Yes, we've got a PR for it somewhere..

bug 46935? or bug 14799?

[Bug ipa/88702] [7/8/9/10 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702

--- Comment #10 from Martin Liška  ---
(In reply to David Malcolm from comment #9)
> If using a switch is better than a series of tests against constants, would
> it make sense for the compiler to spot this case, and automatically convert
> the conditions to a switch?

Yes, we've got a PR for it somewhere..

[Bug ipa/88702] [7/8/9/10 regression] We do terrible job optimizing IsHTMLWhitespace from Firefox

[dmalcolm at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88702

David Malcolm  changed:

   What|Removed |Added

 CC||dmalcolm at gcc dot gnu.org

--- Comment #9 from David Malcolm  ---
If using a switch is better than a series of tests against constants, would it
make sense for the compiler to spot this case, and automatically convert the
conditions to a switch?