On 10/30/2017 11:51 PM, Sergei Shtylyov wrote:
From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
Date: Sat, 28 Oct 2017 19:10:08 +0200
Add a jump target so that a bit of exception handling can be better reused
at the end of this function.
Have you actually tried to see if there's any change in the resulting
object code? I've looked at ARM gcc 4.8.5's output and it didn't really change
-- gcc was able to optimize these repetitive assignments pretty well...
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
---
drivers/net/ethernet/renesas/ravb_main.c | 32
++++++++++++++++----------------
1 file changed, 16 insertions(+), 16 deletions(-)
Diffstat also speaks about the futility of this patch.
Ah, you've added empty lines where there was none... Please don't do this.
I'll ACK the v2 patch, given my comments are addressed.
[...]
MBR, Sergei
