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

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

[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