https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
--- Comment #11 from CVS Commits ---
The trunk branch has been updated by Andrew Pinski :
https://gcc.gnu.org/g:b06cfb62229f17eca59fa4aabf853d7e17e2327b
commit r14-868-gb06cfb62229f17eca59fa4aabf853d7e17e2327b
Author: Andrew Pinski
Date:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
Andrew Pinski changed:
What|Removed |Added
Attachment #55026|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
--- Comment #9 from Andrew Pinski ---
By the way this does show up in GCC itself.
in worse_state in ipa-pure-const.cc where it does MAX of bools
and for x86's internal_min_issue_delay in insn-automata.cc
The below is the similar code to what
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
--- Comment #8 from Andrew Pinski ---
Created attachment 55027
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55027=edit
testcases
max is optimized with this, max1 was already handled.
min was already handled, min1 is optimized with
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
--- Comment #7 from Andrew Pinski ---
Created attachment 55026
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=55026=edit
Patch which adds what I Mentioned
I still need to add the testcases.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
Richard Biener changed:
What|Removed |Added
Target Milestone|12.3|12.4
--- Comment #6 from Richard
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
--- Comment #5 from Andrew Pinski ---
I think this should not be hard to add to minmax_from_comparison I think.
Though right now we don't call it for the non-constant case but that should be
easy to fix I think.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107888
Andrew Pinski changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |pinskia at gcc dot
gnu.org