Re: [PATCH v2] module: LLVMLinux: Remove unused function warning from __param_check macro

2014-03-07 Thread Behan Webster
On 03/07/14 18:17, Joe Perches wrote: On Fri, 2014-03-07 at 18:10 -0800, beh...@converseincode.com wrote: This code makes a compile time type check that is optimized away. Clang complains that it generates an unused function. [] diff --git a/include/linux/moduleparam.h b/include/linux/modulepa

Re: [PATCH v2] module: LLVMLinux: Remove unused function warning from __param_check macro

2014-03-07 Thread Joe Perches
On Fri, 2014-03-07 at 18:10 -0800, beh...@converseincode.com wrote: > This code makes a compile time type check that is optimized away. Clang > complains that it generates an unused function. [] > diff --git a/include/linux/moduleparam.h b/include/linux/moduleparam.h [] > @@ -346,7 +346,7 @@ static

[PATCH v2] module: LLVMLinux: Remove unused function warning from __param_check macro

2014-03-07 Thread behanw
From: Mark Charlebois This code makes a compile time type check that is optimized away. Clang complains that it generates an unused function. I believe GCC won't complain for a static inline fuction but would if it was just a static function. Adding the unused attribute to the function definiti