Re: [PATCH] LLVMLinux: Remove warning about returning an uninitialized variable

From: Behan Webster
Date: Sat Mar 22 2014 - 11:46:04 EST


On 03/22/14 03:01, Arnd Bergmann wrote:
On Saturday 22 March 2014, behanw@xxxxxxxxxxxxxxxxxx wrote:
diff --git a/include/asm-generic/cmpxchg-local.h b/include/asm-generic/cmpxchg-local.h
index d8d4c89..4c41bb8 100644
--- a/include/asm-generic/cmpxchg-local.h
+++ b/include/asm-generic/cmpxchg-local.h
@@ -41,6 +41,7 @@ static inline unsigned long __cmpxchg_local_generic(volatile void *ptr,
break;
default:
wrong_size_cmpxchg(ptr);
+ prev = 0;
}
raw_local_irq_restore(flags);
return prev;

How about using __unreachable() instead to annotate the fact that you won't
get here?
Yours is a _much_ better idea. Will fix.

Thanks,

Behan

--
Behan Webster
behanw@xxxxxxxxxxxxxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/