Bug 323845 (MONO81161) - [PATCH] Compiler Ignores PreserveSigAttribute
Summary: [PATCH] Compiler Ignores PreserveSigAttribute
Status: RESOLVED FIXED
Alias: MONO81161
Product: Mono: Compilers
Classification: Mono
Component: C# (show other bugs)
Version: 1.0
Hardware: Other Other
: P3 - Medium : Enhancement
Target Milestone: ---
Assignee: Marek Safar
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-03-16 18:43 UTC by Jonathan Chambers
Modified: 2007-09-15 21:24 UTC (History)
0 users

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Test case (527 bytes, text/plain)
2007-03-16 18:44 UTC, Thomas Wiest 		Details
Patch with test case (3.29 KB, patch)
2007-03-16 20:11 UTC, Thomas Wiest 		Details | Diff
Patch with license statement (3.38 KB, patch)
2007-03-16 20:45 UTC, Thomas Wiest 		Details | Diff
Another test case... (651 bytes, text/plain)
2007-03-19 22:13 UTC, Thomas Wiest 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wiest 2007-09-15 20:32:05 UTC 


---- Reported by joncham@gmail.com 2007-03-16 11:43:52 MST ----

Please fill in this template when reporting a bug, unless you know what you
are doing.
Description of Problem:
Compiler Ignores PreserveSigAttribute. The compiler should set PreserveSig
flag in the MethodImplAttributes of a method if that method is tagged with
the PreserveSigAttribute.

Steps to reproduce the problem:
1. Run attached test case
2. 
3. 

Actual Results:
"No PreserveSig"

Expected Results:
"Has PreserveSig"

How often does this happen? 
Everytime

Additional Information:



---- Additional Comments From joncham@gmail.com 2007-03-16 11:44:14 MST ----

Created an attachment (id=171674)
Test case




---- Additional Comments From joncham@gmail.com 2007-03-16 13:11:35 MST ----

Created an attachment (id=171675)
Patch with test case




---- Additional Comments From joncham@gmail.com 2007-03-16 13:45:20 MST ----

Created an attachment (id=171676)
Patch with license statement




---- Additional Comments From marek.safar@seznam.cz 2007-03-17 15:12:39 MST ----

Fixed in SVN.



---- Additional Comments From joncham@gmail.com 2007-03-19 15:13:45 MST ----

Created an attachment (id=171677)
Another test case...




---- Additional Comments From joncham@gmail.com 2007-03-19 15:14:28 MST ----

Fails when another pseudo attribute is used, particularly
[MethodImpl(MethodImplOptions.InternalCall,MethodCodeType=MethodCodeType.Runtime)]




---- Additional Comments From marek.safar@seznam.cz 2007-03-19 16:15:11 MST ----

Fixed in SVN



---- Additional Comments From joncham@gmail.com 2007-03-19 17:51:31 MST ----

I reverted this patch as it broke the build.



---- Additional Comments From marek.safar@seznam.cz 2007-03-19 20:52:46 MST ----

Fixed in SVN.

Hopefully this version will not cause any errors.

Imported an attachment (id=171674)
Imported an attachment (id=171675)
Imported an attachment (id=171676)
Imported an attachment (id=171677)

Unknown operating system unknown. Setting to default OS "Other".