public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug tree-optimization/113189] New: `(-X * Y) * -X` can be optimized to `(X * Y) * X`
@ 2024-01-01 9:39 pinskia at gcc dot gnu.org
2024-01-01 9:43 ` [Bug tree-optimization/113189] " pinskia at gcc dot gnu.org
2024-01-01 13:31 ` jsm28 at gcc dot gnu.org
0 siblings, 2 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2024-01-01 9:39 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113189
Bug ID: 113189
Summary: `(-X * Y) * -X` can be optimized to `(X * Y) * X`
Product: gcc
Version: 14.0
Status: UNCONFIRMED
Keywords: missed-optimization, TREE
Severity: enhancement
Priority: P3
Component: tree-optimization
Assignee: unassigned at gcc dot gnu.org
Reporter: pinskia at gcc dot gnu.org
Target Milestone: ---
Take:
```
int f(int X, int Y)
{
return (-X * Y) * -X;
}
double fd(double X, double Y)
{
return (-X * Y) * -X;
}
int g(int X, int Y)
{
return ((-X) << Y) * -X;
}
```
The negative should be optimized away but currently is only optimized at the
RTL level.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug tree-optimization/113189] `(-X * Y) * -X` can be optimized to `(X * Y) * X`
2024-01-01 9:39 [Bug tree-optimization/113189] New: `(-X * Y) * -X` can be optimized to `(X * Y) * X` pinskia at gcc dot gnu.org
@ 2024-01-01 9:43 ` pinskia at gcc dot gnu.org
2024-01-01 13:31 ` jsm28 at gcc dot gnu.org
1 sibling, 0 replies; 3+ messages in thread
From: pinskia at gcc dot gnu.org @ 2024-01-01 9:43 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113189
--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Other examples where the negative can be removed:
```
int foo(int a) {
return -a * (-a * a);
}
int bar(int a) {
return (-a * -a) * a;
}
int foo1(int a, int b, int c) {
return -a * (-b * c);
}
```
bar is handled at the gimple level but foo and foo1 are not.
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Bug tree-optimization/113189] `(-X * Y) * -X` can be optimized to `(X * Y) * X`
2024-01-01 9:39 [Bug tree-optimization/113189] New: `(-X * Y) * -X` can be optimized to `(X * Y) * X` pinskia at gcc dot gnu.org
2024-01-01 9:43 ` [Bug tree-optimization/113189] " pinskia at gcc dot gnu.org
@ 2024-01-01 13:31 ` jsm28 at gcc dot gnu.org
1 sibling, 0 replies; 3+ messages in thread
From: jsm28 at gcc dot gnu.org @ 2024-01-01 13:31 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113189
--- Comment #2 from Joseph S. Myers <jsm28 at gcc dot gnu.org> ---
If Y is INT_MIN and X is -1, removing the negations introduces undefined
behavior in the first example (-(-1) * INT_MIN * -(-1) is valid, -1 * INT_MIN *
-1 has undefined behavior).
For floating types, removing the negations isn't valid if flag_rounding_math.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2024-01-01 13:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-01-01 9:39 [Bug tree-optimization/113189] New: `(-X * Y) * -X` can be optimized to `(X * Y) * X` pinskia at gcc dot gnu.org
2024-01-01 9:43 ` [Bug tree-optimization/113189] " pinskia at gcc dot gnu.org
2024-01-01 13:31 ` jsm28 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).