diff options
| author | Andrew Pinski <quic_apinski@quicinc.com> | 2025-07-07 12:16:54 -0700 |
|---|---|---|
| committer | Andrew Pinski <quic_apinski@quicinc.com> | 2025-07-16 08:03:10 -0700 |
| commit | fbc849d9c3872a05a308724f4009c18685b5af9c (patch) | |
| tree | 6dc6b6a9b14cba064400e3a073e98e560d6567df /gcc | |
| parent | f8563adf99468f61a21fdb447c96f71733310d9d (diff) | |
| download | gcc-fbc849d9c3872a05a308724f4009c18685b5af9c.zip gcc-fbc849d9c3872a05a308724f4009c18685b5af9c.tar.gz gcc-fbc849d9c3872a05a308724f4009c18685b5af9c.tar.bz2 | |
ifconv: Small improvement to fold_build_cond_expr; lhs and rhs being the same.
This is a small compile time optimization, as match and simplify will generate
the same thing but with rhs and lhs being the same we can return early instead
of having to go through match and simplify. This might not show up that much
at this point but can/will show up after my patch for PR 119920 where we factor
out common code between the 2 sides of the if statement while in if-conv.
Bootstrapped and tested on x86_64-linux-gnu.
gcc/ChangeLog:
* tree-if-conv.cc (fold_build_cond_expr): Return early if lhs and rhs
are the same.
Signed-off-by: Andrew Pinski <quic_apinski@quicinc.com>
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/tree-if-conv.cc | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/gcc/tree-if-conv.cc b/gcc/tree-if-conv.cc index d2b9f9f..366e959 100644 --- a/gcc/tree-if-conv.cc +++ b/gcc/tree-if-conv.cc @@ -494,6 +494,10 @@ fold_or_predicates (location_t loc, tree c1, tree c2) static tree fold_build_cond_expr (tree type, tree cond, tree rhs, tree lhs) { + /* Short cut the case where both rhs and lhs are the same. */ + if (operand_equal_p (rhs, lhs)) + return rhs; + /* If COND is comparison r != 0 and r has boolean type, convert COND to SSA_NAME to accept by vect bool pattern. */ if (TREE_CODE (cond) == NE_EXPR) |