Re: [PATCH 0/5] Reduce needless value copies in the analyzer and a few hot middle-end paths

[Andrew MacLeod]

On 10/13/26 9:52 AM, Richard Biener wrote:
On Tue, Oct 13, 2026 at 3:29 PM Martin Jambor <mjam...@suse.cz> wrote:
On Fri, Oct 09 2026, best xmg wrote:

diff --git a/gcc/range-op.cc b/gcc/range-op.cc
index 350548951f4..3a0e6ed05d1 100644
--- a/gcc/range-op.cc
+++ b/gcc/range-op.cc
@@ -2585,10 +2585,10 @@ operator_div::wi_fold (irange &r, tree type,
                      const wide_int &lh_lb, const wide_int &lh_ub,
                      const wide_int &rh_lb, const wide_int &rh_ub) const
  {
-  const wide_int dividend_min = lh_lb;
-  const wide_int dividend_max = lh_ub;
-  const wide_int divisor_min = rh_lb;
-  const wide_int divisor_max = rh_ub;
+  const wide_int &dividend_min = lh_lb;
+  const wide_int &dividend_max = lh_ub;
+  const wide_int &divisor_min = rh_lb;
+  const wide_int &divisor_max = rh_ub;
If we have a const reference alias why keep it and not replace it with the
also const reference RHS?  As written it doesn't make much sense;
pushing down the declarations to where they are used might make the intent
more clear.

Why not just rename the 4 parameters. that seems like it would be pretty 
clear.

Andrew