Patchworkβ Multiple translation unit regression

login
register
about
Submitter Ramsay Jones
Date 2009-07-18 20:26:23
Message ID <4A622FEF.2010704@ramsay1.demon.co.uk>
Download mbox | patch
Permalink /patch/36208/
State New
Headers show

Comments

Ramsay Jones - 2009-07-18 20:26:23
Ramsay Jones wrote:
> Christopher Li wrote:
>> I attach the patch follows. Can you give it a try?
>>
> 
> Yep, this works.
> Tested on cygwin and Linux.
> 
> Thanks!
> 

BTW, I forgot to mention that I changed the test file so that
it would fail before applying the rest of this patch, and
pass afterward.

I've attached the new file. The diff is as follows:

-->8--
-->8--

HTH.

ATB,
Ramsay Jones
extern __inline__ int f(int);

extern __inline__ int
f(int x)
{
        return x;
}

extern int g(int);

extern __inline__ int
g(int x)
{
        return x;
}


/*
 * check-name: extern inline function
 * check-command: sparse $file $file
 * check-description: Extern inline function never emits stand alone copy
 * of the function. It allows multiple such definitions in different file.
 */
Christopher Li - 2009-07-18 21:53:01
On Sat, Jul 18, 2009 at 1:26 PM, Ramsay Jones<ramsay@ramsay1.demon.co.uk> wrote:
>
> BTW, I forgot to mention that I changed the test file so that
> it would fail before applying the rest of this patch, and
> pass afterward.
>
> I've attached the new file. The diff is as follows:

Thanks. Will apply that.

Chris
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/validation/extern-inline.c b/validation/extern-inline.c
index 13af438..4f12ac0 100644
--- a/validation/extern-inline.c
+++ b/validation/extern-inline.c
@@ -17,6 +17,7 @@  g(int x)
 
 /*
  * check-name: extern inline function
+ * check-command: sparse $file $file
  * check-description: Extern inline function never emits stand alone copy
  * of the function. It allows multiple such definitions in different file.
  */