RE: [PATCH] of_mdio: merge branch tails in of_phy_register_fixed_link()

2017-08-15 Thread David Laight
From: David Miller > Sent: 14 August 2017 04:09 > From: Sergei Shtylyov > Date: Sun, 13 Aug 2017 00:03:06 +0300 > > > Looks like gcc isn't always able to figure out that 3 *if* branches in > > of_phy_register_fixed_link() calling fixed_phy_register() at their ends > > are similar enough and thu

Re: [PATCH] of_mdio: merge branch tails in of_phy_register_fixed_link()

2017-08-13 Thread David Miller
From: Sergei Shtylyov Date: Sun, 13 Aug 2017 00:03:06 +0300 > Looks like gcc isn't always able to figure out that 3 *if* branches in > of_phy_register_fixed_link() calling fixed_phy_register() at their ends > are similar enough and thus can be merged. The "manual" merge saves 40 > bytes of the

[PATCH] of_mdio: merge branch tails in of_phy_register_fixed_link()

2017-08-12 Thread Sergei Shtylyov
Looks like gcc isn't always able to figure out that 3 *if* branches in of_phy_register_fixed_link() calling fixed_phy_register() at their ends are similar enough and thus can be merged. The "manual" merge saves 40 bytes of the object code (AArch64 gcc 4.8.5), and still saves 12 bytes even if gcc